Fix some minor warnings in extensions#7514
Conversation
Might be the only case of -Wstrict-prototypes being used.
Current releases of distutils (2.7 & 3.5, at least) import customize_compiler into the build_ext namespace, so changing the original doesn't actually have any effect.
This prevents any macro-has-been-redefined warnings.
|
Also fixed warnings about format specifiers and doubly-defined macros. |
QuLogic
modified the milestones:
2.0.1 (next bug fix release),
2.1 (next point release)
Not sure why it works locally but not on Travis.
QuLogic
changed the title
NelleV
changed the title
|
I've approved this, but this is typically things that should not go into 2.x at this point. |
|
Admittedly, it got a bit larger than I expected (I could have sworn that |
|
from https://stackoverflow.com/questions/8132399/how-to-printf-uint64-t it looks like it is a difference between compilers that requires |
|
Ah yes, gcc 6.2.1; I always forget it's that much newer (especially as Travis becomes older and older.) |
QuLogic
changed the title
QuLogic
modified the milestones:
2.0.1 (next bug fix release),
2.0 (style change major release)
3 participants
Fix prototypes, an unused variable and some new misleading indentation warnings on GCC 6.