diff --git a/changes.txt b/changes.txt index 808328dd..5c874101 100644 --- a/changes.txt +++ b/changes.txt @@ -9,6 +9,7 @@ Changes in ogl-es (not yet released - will be merged with trunk at some point) -------------------------- Changes in 1.9 (not yet released) +- Slightly changed close window handling on X11 (optimized and avoids problems on some shells). Thanks @TheBrokenRail for a patch (was part of patch #322). - Add getActiveColor functions to IGUIStaticText and IGUIButton (get currently used color). - Add IGUIEnvironment::addToDeletionQueue to allow save removal of gui elements while iterating over them (like the same named function in ISceneManager). - IGUIEnvironment::drawAll has now a parameter to allow disabling automatic resize to screensize. Makes it easier to use partial screens with full alignment support. diff --git a/source/Irrlicht/CIrrDeviceLinux.cpp b/source/Irrlicht/CIrrDeviceLinux.cpp index 34f0b2cd..af54946a 100644 --- a/source/Irrlicht/CIrrDeviceLinux.cpp +++ b/source/Irrlicht/CIrrDeviceLinux.cpp @@ -89,13 +89,12 @@ namespace Atom X_ATOM_NETWM_MAXIMIZE_VERT; Atom X_ATOM_NETWM_MAXIMIZE_HORZ; Atom X_ATOM_NETWM_STATE; + + Atom X_ATOM_WM_DELETE_WINDOW; }; namespace irr { - -const char wmDeleteWindow[] = "WM_DELETE_WINDOW"; - //! constructor CIrrDeviceLinux::CIrrDeviceLinux(const SIrrlichtCreationParameters& param) : CIrrDeviceStub(param), @@ -489,9 +488,8 @@ bool CIrrDeviceLinux::createWindow() XMapRaised(XDisplay, XWindow); CreationParams.WindowId = (void*)XWindow; - Atom wmDelete; - wmDelete = XInternAtom(XDisplay, wmDeleteWindow, True); - XSetWMProtocols(XDisplay, XWindow, &wmDelete, 1); + X_ATOM_WM_DELETE_WINDOW = XInternAtom(XDisplay, "WM_DELETE_WINDOW", True); + XSetWMProtocols(XDisplay, XWindow, &X_ATOM_WM_DELETE_WINDOW, 1); if (CreationParams.Fullscreen) { XSetInputFocus(XDisplay, XWindow, RevertToParent, CurrentTime); @@ -1046,8 +1044,7 @@ bool CIrrDeviceLinux::run() case ClientMessage: { - char *atom = XGetAtomName(XDisplay, event.xclient.message_type); - if (*atom == *wmDeleteWindow) + if (static_cast(event.xclient.data.l[0]) == X_ATOM_WM_DELETE_WINDOW && X_ATOM_WM_DELETE_WINDOW != None) { os::Printer::log("Quit message received.", ELL_INFORMATION); Close = true; @@ -1060,7 +1057,6 @@ bool CIrrDeviceLinux::run() irrevent.UserEvent.UserData2 = static_cast(event.xclient.data.l[1]); postEventFromUser(irrevent); } - XFree(atom); } break; diff --git a/source/Irrlicht/Makefile b/source/Irrlicht/Makefile index c967d247..d637b553 100644 --- a/source/Irrlicht/Makefile +++ b/source/Irrlicht/Makefile @@ -94,15 +94,15 @@ endif ifdef EMSCRIPTEN CXXFLAGS += -std=gnu++11 -U__STRICT_ANSI__ ifndef NDEBUG - CFLAGS := -DPNG_THREAD_UNSAFE_OK -DPNG_NO_MMX_CODE -DPNG_NO_MNG_FEATURES + CFLAGS := -DPNG_THREAD_UNSAFE_OK -DPNG_NO_MMX_CODE -DPNG_NO_MNG_FEATURES -DPNG_ARM_NEON_OPT=0 else - CFLAGS := -O3 -DPNG_THREAD_UNSAFE_OK -DPNG_NO_MMX_CODE -DPNG_NO_MNG_FEATURES + CFLAGS := -O3 -DPNG_THREAD_UNSAFE_OK -DPNG_NO_MMX_CODE -DPNG_NO_MNG_FEATURES -DPNG_ARM_NEON_OPT=0 endif ifdef WASM CXXFLAGS += -s WASM=1 endif else - CFLAGS := -O3 -fexpensive-optimizations -DPNG_THREAD_UNSAFE_OK -DPNG_NO_MMX_CODE -DPNG_NO_MNG_FEATURES + CFLAGS := -O3 -fexpensive-optimizations -DPNG_THREAD_UNSAFE_OK -DPNG_NO_MMX_CODE -DPNG_NO_MNG_FEATURES -DPNG_ARM_NEON_OPT=0 endif sharedlib sharedlib_osx: CXXFLAGS += -fPIC