Ok patch was rejected. BUT I removed it from that header and it works perfectly. If that flag was in the header C::B use 99% CPU if I have no default.conf....
I originally removed that flag from the header, but left it there when updating to Ceniza's newest release of the splash screen class. It is a default parameter, so it was trivial to specify the correct parameter in
app.cpp. This way, we do not have a patched 3rd party component (which would cause problems every time we update it, again). We need not and we should not patch that header.
The flag is, however,
not related to the 99% CPU bug (this one was fixed by Yiannis independently). The flag was related to the message box lockout and later to the update event crash.