Navigation Menu

Skip to content

Commit

Permalink
Fixed bug 4159 - Windows headers are included after packing alignment…
Browse files Browse the repository at this point in the history
… change

lectem

The SDL_syswm.h header includes the windows.h header after including begin_code.h which changes the structure packing alignment.

It seems this is not safe as suggested by the following warning :
warning C4121: 'JOBOBJECT_IO_RATE_CONTROL_INFORMATION_NATIVE_V2': alignment of a member was sensitive to packing
  • Loading branch information
slouken committed May 8, 2018
1 parent 0bade9b commit c04dca0
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions include/SDL_syswm.h
Expand Up @@ -33,12 +33,6 @@
#include "SDL_video.h"
#include "SDL_version.h"

#include "begin_code.h"
/* Set up for C function definitions, even when using C++ */
#ifdef __cplusplus
extern "C" {
#endif

/**
* \file SDL_syswm.h
*
Expand Down Expand Up @@ -110,6 +104,12 @@ typedef void *EGLSurface;
#include "SDL_egl.h"
#endif

#include "begin_code.h"
/* Set up for C function definitions, even when using C++ */
#ifdef __cplusplus
extern "C" {
#endif

/**
* These are the various supported windowing subsystems
*/
Expand Down

0 comments on commit c04dca0

Please sign in to comment.