Ignore:
Timestamp:
Aug 17, 2012, 7:16:23 AM (13 years ago)
Author:
[email protected]
Message:

[Qt] QNX build fails due to ctype usage in system headers
https://bugs.webkit.org/show_bug.cgi?id=93849

Patch by Milian Wolff <[email protected]> on 2012-08-17
Reviewed by Simon Hausmann.

Move the check for whether DisallowCType should be active or not
to the DisallowCType.h header. This way, we can update the list
of platforms or OSes which do not work with this header in a
central place. All users can now safely include the header
and do not need to place custom guards around it.

Source/JavaScriptCore:

  • config.h:

Source/WebCore:

  • config.h:

Source/WTF:

  • config.h:
  • wtf/DisallowCType.h:
File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/config.h

    r113798 r125895  
    7070#endif
    7171
    72 // this breaks compilation of <QFontDatabase>, at least, so turn it off for now
    73 // Also generates errors on wx on Windows and QNX, because these functions
    74 // are used from wx and QNX headers.
    75 #if !PLATFORM(QT) && !PLATFORM(WX) && !OS(QNX)
    7672#include <wtf/DisallowCType.h>
    77 #endif
    7873
    7974#if COMPILER(MSVC)
Note: See TracChangeset for help on using the changeset viewer.