[issue14387] Include\accu.h incompatible with Windows.h

Jeff Robbins report at bugs.python.org
Thu Mar 22 18:50:30 CET 2012


Jeff Robbins <jeffr at livedata.com> added the comment:

The easiest fix would be to rename the "small" field in accu.h (and of course in any file that references that field.)  I have no idea who uses this file.

Perhaps the 2nd easiest fix would be to put this after the #include of Windows.h

#ifdef small
#undef small
#endif

As long as any extension also including Windows.h included Windows.h BEFORE Python.h, this should fix the problem for an extension builder.

A further thought is to put

#define WIN32_LEAN_AND_MEAN

right before the #include Windows.h

This doesn't fix the extension problem in that every #include of Windows.h would need to do this (SQLITE doesn't right now) but if all Python needs from Windows.h is in the "WIN32_LEAN_AND_MEAN" subset, it seems like a good thing in general, no?

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue14387>
_______________________________________


More information about the Python-bugs-list mailing list