solemnwarning/wxwidgets
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Initialize wxAppConsoleBase members in their declaration

Browse source 
No real changes, just initialize member variables immediately in the
declaration instead of doing it in the ctor in a different file.
This commit is contained in:
Vadim Zeitlin 2023-03-06 18:49:54 +00:00
parent 347d7fcccb
commit b4256aa4e2
2 changed files with 5 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

10
include/wx/app.h
View file

@ -502,13 +502,13 @@ protected:
m_appDisplayName, // app display name ("My Application")
m_className; // class name
// the class defining the application behaviour, nullptr initially and created
// by GetTraits() when first needed
wxAppTraits *m_traits;
// allows customizing the application behaviour, created by GetTraits()
// when first needed
wxAppTraits *m_traits = nullptr;
// the main event loop of the application (may be null if the loop hasn't
// been started yet or has already terminated)
wxEventLoopBase *m_mainLoop;
wxEventLoopBase *m_mainLoop = nullptr;
// pending events management vars:
@ -529,7 +529,7 @@ protected:
#endif
// flag modified by Suspend/ResumeProcessingOfPendingEvents()
bool m_bDoPendingEventProcessing;
bool m_bDoPendingEventProcessing = true;
friend class WXDLLIMPEXP_FWD_BASE wxEvtHandler;

4
src/common/appbase.cpp
View file

@ -134,10 +134,6 @@ wxDEFINE_TIED_SCOPED_PTR_TYPE(wxEventLoopBase)
wxAppConsoleBase::wxAppConsoleBase()
{
m_traits = nullptr;
m_mainLoop = nullptr;
m_bDoPendingEventProcessing = true;
ms_appInstance = reinterpret_cast<wxAppConsole *>(this);
#ifdef __WXDEBUG__