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

Fix a false positive OverflowError in Python 3.x when value above 0x7… #7568

Merged
merged 1 commit into from
Apr 26, 2016

Conversation

mlamarre
Copy link
Contributor

…FFFFFF are passed to a function accepting "unsigned int".

This is a port of a fix in pyprimtype.swg from which several code snippets where copy pasted into swig/pyfragments.swg in the past.

Please see SWIG changes log (2015-12-23) for more details:

http://www.swig.org/Release/CHANGES.current

[...]

2015-12-23: ahnolds [Python] Fixes for conversion of signed and unsigned integer types

[...]

…FFFFFF are passed to a function accepting "unsigned int".

This a port of a fix in pyprimtype.swg from which several code snippets where copy pasted into swig/pyfragments.swg.

Please see SWIG changes log (2015-12-23) for more details:

http://www.swig.org/Release/CHANGES.current

2015-12-23: ahnolds  [Python] Fixes for conversion of signed and unsigned integer types ...
@charris
Copy link
Member

charris commented Apr 22, 2016

@wfspotz Comment?

@charris charris added this to the 1.12.0 release milestone Apr 22, 2016
@mlamarre
Copy link
Contributor Author

This is the commit in SWIG with the fix that I brought back:
swig/swig@ba01182

@charris charris merged commit 1b6831b into numpy:master Apr 26, 2016
@charris
Copy link
Member

charris commented Apr 26, 2016

OK, I'll leave it up to you folks. Thanks @mlamarre .

@mlamarre mlamarre deleted the fix_swig_py3_overflow branch April 26, 2016 14:14
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.

None yet

2 participants