Skip to content

Support unsigned long values for Dynamic #1214

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

singpolyma
Copy link

Fixes ambiguous calls when compiling with Xcode

@singpolyma singpolyma force-pushed the support-unsigned-long branch from 441f720 to c913af3 Compare April 14, 2025 20:48
@barisyild
Copy link
Contributor

It looks nice but may break backwards compatibility.

@singpolyma
Copy link
Author

why would it break compatibility? anything using an unsigned long (or equivalent) before would fail to compile

@barisyild
Copy link
Contributor

why would it break compatibility? anything using an unsigned long (or equivalent) before would fail to compile

Isn't this change likely to affect the behavior of existing projects?

@singpolyma
Copy link
Author

It shouldn't affect the behaviour of anything existing, except that some that would not compile before will compile now.

Fixes ambiguous calls when compiling with Xcode
@singpolyma singpolyma force-pushed the support-unsigned-long branch from c913af3 to ffbb7ed Compare April 15, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants