Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adapter::Reslice: fix handling of complex types #2891

Closed
wants to merge 1 commit into from

Conversation

jdtournier
Copy link
Member

Not sure whether this is required, but it caused trouble for me at some point. I suggest leaving as a draft for now until the problem comes up again...

Copy link

github-actions bot commented May 2, 2024

clang-tidy review says "All clean, LGTM! 👍"

@Lestropie
Copy link
Member

Suspected duplicate of #2768; I posted a suggested alternative as #2785.

@jdtournier
Copy link
Member Author

Well spotted! OK, I'll close this and #2768 - I prefer your approach in #2785.

@jdtournier jdtournier closed this May 2, 2024
@jdtournier jdtournier deleted the adapter_reslice_complex_types branch May 2, 2024 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants