Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't use multi-character constants for portability reasons

Browse Source 
Instead of using numeric constants for MinGW and character constants
otherwise, just always use the former as this code can also be compiled
with non-MinGW gcc when building under Cygwin and this results in the
same warnings as for MinGW.

Additionally, it's not guaranteed that other MSW compilers actually
support multi-character constants at all, even if they don't warn about
them, so just avoid them completely.
This commit is contained in:
Vadim Zeitlin
2017-11-11 15:30:22 +01:00
parent c98fa0ca98
commit c443b4c264
1 changed files with 5 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/msw/mediactrl_qt.cpp
View File

@@ -101,14 +101,11 @@ typedef struct ComponentInstanceRecord * ComponentInstance;
#define MovieController ComponentInstance #define MovieController ComponentInstance
#ifndef URLDataHandlerSubType #ifndef URLDataHandlerSubType
#if defined(__MINGW32__) // Under Mac this would be defined as 'url ' and 'eyes' multi-character
// use magic numbers for compilers which complain about multicharacter integers // constants respectively (translate each byte to ASCII to see it), but this is
const OSType URLDataHandlerSubType = 1970433056; // not accepted by non-Mac compilers, so use the numeric constants instead.
const OSType VisualMediaCharacteristic = 1702454643; const OSType URLDataHandlerSubType = 0x75726c20;
#else const OSType VisualMediaCharacteristic = 0x65796573;
const OSType URLDataHandlerSubType = 'url ';
const OSType VisualMediaCharacteristic = 'eyes';
#endif
#endif #endif
struct FSSpec struct FSSpec