From e5c0b16a78bff758f5167cc2afe3d0012b5cbb86 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Tue, 23 Feb 1999 22:49:26 +0000 Subject: [PATCH] wxLogXXX() functions called near app termiantion shouldn't crash git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@1765 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- src/msw/app.cpp | 905 +++++++++++++++++++++++++----------------------- 1 file changed, 466 insertions(+), 439 deletions(-) diff --git a/src/msw/app.cpp b/src/msw/app.cpp index ba4a42af25..055aa2f3b1 100644 --- a/src/msw/app.cpp +++ b/src/msw/app.cpp @@ -9,32 +9,40 @@ // Licence: wxWindows license ///////////////////////////////////////////////////////////////////////////// +// =========================================================================== +// declarations +// =========================================================================== + +// --------------------------------------------------------------------------- +// headers +// --------------------------------------------------------------------------- + #ifdef __GNUG__ -#pragma implementation "app.h" + #pragma implementation "app.h" #endif // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" #if defined(__BORLANDC__) -#pragma hdrstop + #pragma hdrstop #endif #ifndef WX_PRECOMP - #include "wx/frame.h" - #include "wx/app.h" - #include "wx/utils.h" - #include "wx/gdicmn.h" - #include "wx/pen.h" - #include "wx/brush.h" - #include "wx/cursor.h" - #include "wx/icon.h" - #include "wx/palette.h" - #include "wx/dc.h" - #include "wx/dialog.h" - #include "wx/msgdlg.h" - #include "wx/intl.h" - #include "wx/dynarray.h" + #include "wx/frame.h" + #include "wx/app.h" + #include "wx/utils.h" + #include "wx/gdicmn.h" + #include "wx/pen.h" + #include "wx/brush.h" + #include "wx/cursor.h" + #include "wx/icon.h" + #include "wx/palette.h" + #include "wx/dc.h" + #include "wx/dialog.h" + #include "wx/msgdlg.h" + #include "wx/intl.h" + #include "wx/dynarray.h" #endif #include "wx/msw/private.h" @@ -53,14 +61,17 @@ #endif // wxUSE_THREADS #if wxUSE_WX_RESOURCES - #include "wx/resource.h" + #include "wx/resource.h" #endif -// To UG: there's no point in putting this #if here -// if you don't do the same for the Ole calls further down. -// Also, OLE is used not just for drag and drop (it's used by automatn.cpp). -// #if wxUSE_DRAG_AND_DROP -#if !defined(__GNUWIN32__) && !defined(__SC__) && !defined(__SALFORDC__) +// OLE is used for drag-and-drop, clipboard, OLE Automation... +#if defined(__GNUWIN32__) || defined(__SC__) || defined(__SALFORDC__) + #undef wxUSE_OLE + + #define wxUSE_OLE 0 +#endif // broken compilers + +#if wxUSE_OLE #include #endif @@ -68,11 +79,15 @@ #include #if (defined(__WIN95__) && !defined(__GNUWIN32__)) || defined(__TWIN32__) - #include + #include #endif #include "wx/msw/msvcrt.h" +// --------------------------------------------------------------------------- +// global variables +// --------------------------------------------------------------------------- + extern char *wxBuffer; extern char *wxOsVersion; extern wxList *wxWinHandleList; @@ -84,7 +99,7 @@ HINSTANCE wxhInstance = 0; static MSG s_currentMsg; wxApp *wxTheApp = NULL; -// @@ why not const? and not static? +// FIXME why not const? and not static? char wxFrameClassName[] = "wxFrameClass"; char wxMDIFrameClassName[] = "wxMDIFrameClass"; char wxMDIChildFrameClassName[] = "wxMDIChildFrameClass"; @@ -103,39 +118,46 @@ HBRUSH wxDisableButtonBrush = (HBRUSH) 0; LRESULT APIENTRY wxWndProc(HWND, UINT, WPARAM, LPARAM); -#if !USE_SHARED_LIBRARY - IMPLEMENT_DYNAMIC_CLASS(wxApp, wxEvtHandler) +#if defined(__WIN95__) && !defined(__TWIN32__) + #define wxUSE_RICHEDIT 1 +#else + #define wxUSE_RICHEDIT 0 +#endif - BEGIN_EVENT_TABLE(wxApp, wxEvtHandler) - EVT_IDLE(wxApp::OnIdle) - EVT_END_SESSION(wxApp::OnEndSession) - EVT_QUERY_END_SESSION(wxApp::OnQueryEndSession) - END_EVENT_TABLE() +#if wxUSE_RICHEDIT + static HINSTANCE gs_hRichEdit = (HINSTANCE) NULL; +#endif + +// =========================================================================== +// implementation +// =========================================================================== + +// --------------------------------------------------------------------------- +// wxApp +// --------------------------------------------------------------------------- + +#if !USE_SHARED_LIBRARY + IMPLEMENT_DYNAMIC_CLASS(wxApp, wxEvtHandler) + + BEGIN_EVENT_TABLE(wxApp, wxEvtHandler) + EVT_IDLE(wxApp::OnIdle) + EVT_END_SESSION(wxApp::OnEndSession) + EVT_QUERY_END_SESSION(wxApp::OnQueryEndSession) + END_EVENT_TABLE() #endif long wxApp::sm_lastMessageTime = 0; -#if defined(__WIN95__) && !defined(__TWIN32__) -#define wxUSE_RICHEDIT 1 -#else -#define wxUSE_RICHEDIT 0 -#endif - -#if wxUSE_RICHEDIT - static HINSTANCE gs_hRichEdit = (HINSTANCE) NULL; -#endif - //// Initialize - bool wxApp::Initialize() { // Some people may wish to use this, but // probably it shouldn't be here by default. #ifdef __WXDEBUG__ -// wxRedirectIOToConsole(); + // wxRedirectIOToConsole(); #endif - wxBuffer = new char[1500]; + wxBuffer = new char[1500]; // FIXME wxClassInfo::InitializeClasses(); @@ -163,38 +185,37 @@ bool wxApp::Initialize() if (gs_hRichEdit == (HINSTANCE) NULL) { - wxMessageBox("Could not initialise Rich Edit DLL"); + wxLogError(_("Could not initialise Rich Edit DLL")); } -#endif +#endif // wxUSE_RICHEDIT -#endif +#endif // __WIN95__ + // for OLE, enlarge message queue to be as large as possible int iMsg = 96; + while (!SetMessageQueue(iMsg) && (iMsg -= 8)); - // for OLE, enlarge message queue to be as large as possible - while (!SetMessageQueue(iMsg) && (iMsg -= 8)); - -/* - DWORD dwOleVer; - dwOleVer = CoBuildVersion(); + /* + DWORD dwOleVer; + dwOleVer = CoBuildVersion(); // check the OLE library version if (rmm != HIWORD(dwOleVer)) { - wxMessageBox("Incorrect version of OLE libraries."); - return FALSE; + wxMessageBox("Incorrect version of OLE libraries."); + return FALSE; } -*/ + */ -#if !defined(__GNUWIN32__) && !defined(__SC__) && !defined(__SALFORDC__) +#if wxUSE_OLE // we need to initialize OLE library if ( FAILED(::OleInitialize(NULL)) ) - wxFatalError(_("Cannot initialize OLE")); + wxLogError(_("Cannot initialize OLE")); #endif #if wxUSE_CTL3D if (!Ctl3dRegister(wxhInstance)) - wxFatalError("Cannot register CTL3D"); + wxLogError("Cannot register CTL3D"); Ctl3dAutoSubclass(wxhInstance); #endif @@ -244,124 +265,124 @@ bool wxApp::Initialize() bool wxApp::RegisterWindowClasses() { -/////////////////////////////////////////////////////////////////////// -// Register the frame window class. - WNDCLASS wndclass; // Structure used to register Windows class. + /////////////////////////////////////////////////////////////////////// + // Register the frame window class. + WNDCLASS wndclass; // Structure used to register Windows class. - wndclass.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; - wndclass.lpfnWndProc = (WNDPROC)wxWndProc; - wndclass.cbClsExtra = 0; - wndclass.cbWndExtra = sizeof( DWORD ); // was 4 - wndclass.hInstance = wxhInstance; - wndclass.hIcon = (HICON) NULL; // wxSTD_FRAME_ICON; - wndclass.hCursor = LoadCursor( (HINSTANCE) NULL, IDC_ARROW ); - wndclass.hbrBackground = (HBRUSH)(COLOR_APPWORKSPACE+1) ; -// wndclass.hbrBackground = GetStockObject( WHITE_BRUSH ); - wndclass.lpszMenuName = NULL; + wndclass.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; + wndclass.lpfnWndProc = (WNDPROC)wxWndProc; + wndclass.cbClsExtra = 0; + wndclass.cbWndExtra = sizeof( DWORD ); // was 4 + wndclass.hInstance = wxhInstance; + wndclass.hIcon = (HICON) NULL; // wxSTD_FRAME_ICON; + wndclass.hCursor = LoadCursor( (HINSTANCE) NULL, IDC_ARROW ); + wndclass.hbrBackground = (HBRUSH)(COLOR_APPWORKSPACE+1) ; + // wndclass.hbrBackground = GetStockObject( WHITE_BRUSH ); + wndclass.lpszMenuName = NULL; #ifdef _MULTIPLE_INSTANCES - sprintf( wxFrameClassName,"wxFrameClass%d", wxhInstance ); + sprintf( wxFrameClassName,"wxFrameClass%d", wxhInstance ); #endif - wndclass.lpszClassName = wxFrameClassName; + wndclass.lpszClassName = wxFrameClassName; - if (!RegisterClass( &wndclass )) - { - // wxFatalError("Can't register Frame Window class"); - } + if (!RegisterClass( &wndclass )) + { + // wxFatalError("Can't register Frame Window class"); + } -/////////////////////////////////////////////////////////////////////// -// Register the MDI frame window class. - WNDCLASS wndclass1; // Structure used to register Windows class. + /////////////////////////////////////////////////////////////////////// + // Register the MDI frame window class. + WNDCLASS wndclass1; // Structure used to register Windows class. - wndclass1.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; - wndclass1.lpfnWndProc = (WNDPROC)wxWndProc; - wndclass1.cbClsExtra = 0; - wndclass1.cbWndExtra = sizeof( DWORD ); // was 4 - wndclass1.hInstance = wxhInstance; - wndclass1.hIcon = (HICON) NULL; // wxSTD_MDIPARENTFRAME_ICON; - wndclass1.hCursor = LoadCursor( (HINSTANCE) NULL, IDC_ARROW ); -// wndclass1.hbrBackground = (HBRUSH)(COLOR_APPWORKSPACE+1) ; - wndclass1.hbrBackground = (HBRUSH) NULL; - wndclass1.lpszMenuName = NULL; + wndclass1.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; + wndclass1.lpfnWndProc = (WNDPROC)wxWndProc; + wndclass1.cbClsExtra = 0; + wndclass1.cbWndExtra = sizeof( DWORD ); // was 4 + wndclass1.hInstance = wxhInstance; + wndclass1.hIcon = (HICON) NULL; // wxSTD_MDIPARENTFRAME_ICON; + wndclass1.hCursor = LoadCursor( (HINSTANCE) NULL, IDC_ARROW ); + // wndclass1.hbrBackground = (HBRUSH)(COLOR_APPWORKSPACE+1) ; + wndclass1.hbrBackground = (HBRUSH) NULL; + wndclass1.lpszMenuName = NULL; - wndclass1.lpszClassName = wxMDIFrameClassName; - if (!RegisterClass( &wndclass1 )) - { -// wxFatalError("Can't register MDI Frame window class"); -// return FALSE; - } + wndclass1.lpszClassName = wxMDIFrameClassName; + if (!RegisterClass( &wndclass1 )) + { + // wxFatalError("Can't register MDI Frame window class"); + // return FALSE; + } -/////////////////////////////////////////////////////////////////////// -// Register the MDI child frame window class. - WNDCLASS wndclass4; // Structure used to register Windows class. + /////////////////////////////////////////////////////////////////////// + // Register the MDI child frame window class. + WNDCLASS wndclass4; // Structure used to register Windows class. - wndclass4.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; - wndclass4.lpfnWndProc = (WNDPROC)wxWndProc; - wndclass4.cbClsExtra = 0; - wndclass4.cbWndExtra = sizeof( DWORD ); // was 4 - wndclass4.hInstance = wxhInstance; - wndclass4.hIcon = (HICON) NULL; // wxSTD_MDICHILDFRAME_ICON; - wndclass4.hCursor = LoadCursor( (HINSTANCE) NULL, IDC_ARROW ); - // TODO: perhaps this should be NULL so that Windows doesn't - // paint the background itself (would OnEraseBackground duplicate - // this?) - wndclass4.hbrBackground = (HBRUSH)(COLOR_WINDOW+1) ; -// wndclass4.hbrBackground = NULL; - wndclass4.lpszMenuName = NULL; - wndclass4.lpszClassName = wxMDIChildFrameClassName; + wndclass4.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; + wndclass4.lpfnWndProc = (WNDPROC)wxWndProc; + wndclass4.cbClsExtra = 0; + wndclass4.cbWndExtra = sizeof( DWORD ); // was 4 + wndclass4.hInstance = wxhInstance; + wndclass4.hIcon = (HICON) NULL; // wxSTD_MDICHILDFRAME_ICON; + wndclass4.hCursor = LoadCursor( (HINSTANCE) NULL, IDC_ARROW ); + // TODO: perhaps this should be NULL so that Windows doesn't + // paint the background itself (would OnEraseBackground duplicate + // this?) + wndclass4.hbrBackground = (HBRUSH)(COLOR_WINDOW+1) ; + // wndclass4.hbrBackground = NULL; + wndclass4.lpszMenuName = NULL; + wndclass4.lpszClassName = wxMDIChildFrameClassName; - if (!RegisterClass( &wndclass4 )) - { -// wxFatalError("Can't register MDI child frame window class"); -// return FALSE; - } + if (!RegisterClass( &wndclass4 )) + { + // wxFatalError("Can't register MDI child frame window class"); + // return FALSE; + } -/////////////////////////////////////////////////////////////////////// -// Register the panel window class. - WNDCLASS wndclass2; // Structure used to register Windows class. - memset(&wndclass2, 0, sizeof(WNDCLASS)); // start with NULL defaults - wndclass2.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; - wndclass2.lpfnWndProc = (WNDPROC)wxWndProc; - wndclass2.cbClsExtra = 0; - wndclass2.cbWndExtra = sizeof( DWORD ); // was 4 - wndclass2.hInstance = wxhInstance; - wndclass2.hIcon = (HICON) NULL; - wndclass2.hCursor = (HCURSOR) NULL; -// wndclass2.hbrBackground = (HBRUSH)(COLOR_BTNFACE+1) ; - wndclass2.hbrBackground = (HBRUSH) GetStockObject( LTGRAY_BRUSH ); - wndclass2.lpszMenuName = NULL; - wndclass2.lpszClassName = wxPanelClassName; - if (!RegisterClass( &wndclass2 )) - { -// wxFatalError("Can't register Panel Window class"); -// return FALSE; - } + /////////////////////////////////////////////////////////////////////// + // Register the panel window class. + WNDCLASS wndclass2; // Structure used to register Windows class. + memset(&wndclass2, 0, sizeof(WNDCLASS)); // start with NULL defaults + wndclass2.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; + wndclass2.lpfnWndProc = (WNDPROC)wxWndProc; + wndclass2.cbClsExtra = 0; + wndclass2.cbWndExtra = sizeof( DWORD ); // was 4 + wndclass2.hInstance = wxhInstance; + wndclass2.hIcon = (HICON) NULL; + wndclass2.hCursor = (HCURSOR) NULL; + // wndclass2.hbrBackground = (HBRUSH)(COLOR_BTNFACE+1) ; + wndclass2.hbrBackground = (HBRUSH) GetStockObject( LTGRAY_BRUSH ); + wndclass2.lpszMenuName = NULL; + wndclass2.lpszClassName = wxPanelClassName; + if (!RegisterClass( &wndclass2 )) + { + // wxFatalError("Can't register Panel Window class"); + // return FALSE; + } -/////////////////////////////////////////////////////////////////////// -// Register the canvas and textsubwindow class name - WNDCLASS wndclass3; // Structure used to register Windows class. - memset(&wndclass3, 0, sizeof(WNDCLASS)); // start with NULL defaults - // Use CS_OWNDC to avoid messing about restoring the context - // for every graphic operation. -// wndclass3.style = CS_HREDRAW | CS_VREDRAW | CS_OWNDC | CS_DBLCLKS ; - // wxWin 2.0 - wndclass3.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; - wndclass3.lpfnWndProc = (WNDPROC)wxWndProc; - wndclass3.cbClsExtra = 0; - wndclass3.cbWndExtra = sizeof( DWORD ); // was 4 - wndclass3.hInstance = wxhInstance; - wndclass3.hIcon = (HICON) NULL; - wndclass3.hCursor = (HCURSOR) NULL; -// wndclass3.hbrBackground = (HBRUSH)(COLOR_WINDOW+1) ; - wndclass3.hbrBackground = (HBRUSH) NULL; - wndclass3.lpszMenuName = NULL; - wndclass3.lpszClassName = wxCanvasClassName; - if (!RegisterClass( &wndclass3)) - { -// wxFatalError("Can't register Canvas class"); -// return FALSE; - } + /////////////////////////////////////////////////////////////////////// + // Register the canvas and textsubwindow class name + WNDCLASS wndclass3; // Structure used to register Windows class. + memset(&wndclass3, 0, sizeof(WNDCLASS)); // start with NULL defaults + // Use CS_OWNDC to avoid messing about restoring the context + // for every graphic operation. + // wndclass3.style = CS_HREDRAW | CS_VREDRAW | CS_OWNDC | CS_DBLCLKS ; + // wxWin 2.0 + wndclass3.style = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS ; + wndclass3.lpfnWndProc = (WNDPROC)wxWndProc; + wndclass3.cbClsExtra = 0; + wndclass3.cbWndExtra = sizeof( DWORD ); // was 4 + wndclass3.hInstance = wxhInstance; + wndclass3.hIcon = (HICON) NULL; + wndclass3.hCursor = (HCURSOR) NULL; + // wndclass3.hbrBackground = (HBRUSH)(COLOR_WINDOW+1) ; + wndclass3.hbrBackground = (HBRUSH) NULL; + wndclass3.lpszMenuName = NULL; + wndclass3.lpszClassName = wxCanvasClassName; + if (!RegisterClass( &wndclass3)) + { + // wxFatalError("Can't register Canvas class"); + // return FALSE; + } - return TRUE; + return TRUE; } //// Convert Windows to argc, argv style @@ -443,116 +464,119 @@ void wxApp::ConvertToStandardCommandArgs(char* lpCmdLine) void wxApp::CleanUp() { - //// COMMON CLEANUP + //// COMMON CLEANUP - // flush the logged messages if any - wxLog *pLog = wxLog::GetActiveTarget(); - if ( pLog != NULL && pLog->HasPendingMessages() ) - pLog->Flush(); + // flush the logged messages if any and install a 'safer' log target: the + // default one (wxLogGui) can't be used after the resources are freed just + // below and the user suppliedo ne might be even more unsafe (using any + // wxWindows GUI function is unsafe starting from now) + wxLog::DontCreateOnDemand(); - // One last chance for pending objects to be cleaned up - wxTheApp->DeletePendingObjects(); + // this will flush the old messages if any + delete wxLog::SetActiveTarget(new wxLogStderr); - wxModule::CleanUpModules(); + // One last chance for pending objects to be cleaned up + wxTheApp->DeletePendingObjects(); + + wxModule::CleanUpModules(); #if wxUSE_WX_RESOURCES - wxCleanUpResourceSystem(); + wxCleanUpResourceSystem(); -// wxDefaultResourceTable->ClearTable(); + // wxDefaultResourceTable->ClearTable(); #endif - // Indicate that the cursor can be freed, - // so that cursor won't be deleted by deleting - // the bitmap list before g_globalCursor goes out - // of scope (double deletion of the cursor). - wxSetCursor(wxNullCursor); - delete g_globalCursor; + // Indicate that the cursor can be freed, so that cursor won't be deleted + // by deleting the bitmap list before g_globalCursor goes out of scope + // (double deletion of the cursor). + wxSetCursor(wxNullCursor); + delete g_globalCursor; - wxDeleteStockObjects() ; + wxDeleteStockObjects() ; - // Destroy all GDI lists, etc. - wxDeleteStockLists(); + // Destroy all GDI lists, etc. + wxDeleteStockLists(); - delete wxTheColourDatabase; - wxTheColourDatabase = NULL; + delete wxTheColourDatabase; + wxTheColourDatabase = NULL; - wxBitmap::CleanUpHandlers(); + wxBitmap::CleanUpHandlers(); - delete[] wxBuffer; - wxBuffer = NULL; + delete[] wxBuffer; + wxBuffer = NULL; - //// WINDOWS-SPECIFIC CLEANUP + //// WINDOWS-SPECIFIC CLEANUP - wxSetKeyboardHook(FALSE); + wxSetKeyboardHook(FALSE); #ifdef __WIN95__ #if wxUSE_RICHEDIT - if (gs_hRichEdit != (HINSTANCE) NULL) - FreeLibrary(gs_hRichEdit); + if (gs_hRichEdit != (HINSTANCE) NULL) + FreeLibrary(gs_hRichEdit); #endif #endif #if wxUSE_PENWINDOWS - wxCleanUpPenWin(); + wxCleanUpPenWin(); #endif - if (wxSTD_FRAME_ICON) - DestroyIcon(wxSTD_FRAME_ICON); - if (wxSTD_MDICHILDFRAME_ICON) - DestroyIcon(wxSTD_MDICHILDFRAME_ICON); - if (wxSTD_MDIPARENTFRAME_ICON) - DestroyIcon(wxSTD_MDIPARENTFRAME_ICON); + if (wxSTD_FRAME_ICON) + DestroyIcon(wxSTD_FRAME_ICON); + if (wxSTD_MDICHILDFRAME_ICON) + DestroyIcon(wxSTD_MDICHILDFRAME_ICON); + if (wxSTD_MDIPARENTFRAME_ICON) + DestroyIcon(wxSTD_MDIPARENTFRAME_ICON); - if (wxDEFAULT_FRAME_ICON) - DestroyIcon(wxDEFAULT_FRAME_ICON); - if (wxDEFAULT_MDICHILDFRAME_ICON) - DestroyIcon(wxDEFAULT_MDICHILDFRAME_ICON); - if (wxDEFAULT_MDIPARENTFRAME_ICON) - DestroyIcon(wxDEFAULT_MDIPARENTFRAME_ICON); + if (wxDEFAULT_FRAME_ICON) + DestroyIcon(wxDEFAULT_FRAME_ICON); + if (wxDEFAULT_MDICHILDFRAME_ICON) + DestroyIcon(wxDEFAULT_MDICHILDFRAME_ICON); + if (wxDEFAULT_MDIPARENTFRAME_ICON) + DestroyIcon(wxDEFAULT_MDIPARENTFRAME_ICON); - if ( wxDisableButtonBrush ) - ::DeleteObject( wxDisableButtonBrush ) ; + if ( wxDisableButtonBrush ) + ::DeleteObject( wxDisableButtonBrush ) ; -#if !defined(__GNUWIN32__) && !defined(__SC__) && !defined(__SALFORDC__) - ::OleUninitialize(); +#if wxUSE_OLE + ::OleUninitialize(); #endif #if wxUSE_CTL3D - Ctl3dUnregister(wxhInstance); + Ctl3dUnregister(wxhInstance); #endif - if (wxWinHandleList) - delete wxWinHandleList ; + if (wxWinHandleList) + delete wxWinHandleList ; - wxClassInfo::CleanUpClasses(); + wxClassInfo::CleanUpClasses(); - delete wxTheApp; - wxTheApp = NULL; + delete wxTheApp; + wxTheApp = NULL; #if (defined(__WXDEBUG__) && wxUSE_MEMORY_TRACING) || wxUSE_DEBUG_CONTEXT - // At this point we want to check if there are any memory - // blocks that aren't part of the wxDebugContext itself, - // as a special case. Then when dumping we need to ignore - // wxDebugContext, too. - if (wxDebugContext::CountObjectsLeft() > 0) - { - wxLogDebug("There were memory leaks."); - wxDebugContext::Dump(); - wxDebugContext::PrintStatistics(); - } -// wxDebugContext::SetStream(NULL, NULL); + // At this point we want to check if there are any memory + // blocks that aren't part of the wxDebugContext itself, + // as a special case. Then when dumping we need to ignore + // wxDebugContext, too. + if (wxDebugContext::CountObjectsLeft() > 0) + { + wxLogDebug("There were memory leaks."); + wxDebugContext::Dump(); + wxDebugContext::PrintStatistics(); + } + // wxDebugContext::SetStream(NULL, NULL); #endif - // do it as the very last thing because everything else can log messages - wxLog::DontCreateOnDemand(); - delete wxLog::SetActiveTarget(NULL); + // do it as the very last thing because everything else can log messages + delete wxLog::SetActiveTarget(NULL); } #if !defined(_WINDLL) || (defined(_WINDLL) && defined(WXMAKINGDLL)) -// temporarily disable this warning +// temporarily disable this warning which would be generated in release builds +// because of __try #ifdef __VISUALC__ #pragma warning(disable: 4715) // not all control paths return a value #endif // Visual C++ @@ -569,10 +593,10 @@ int wxEntry(WXHINSTANCE hInstance, // deallocated memory which may be used to simulate low-memory condition) wxCrtSetDbgFlag(_CRTDBG_LEAK_CHECK_DF); - // take everything into a try-except block in release build - // FIXME other compilers must support Win32 SEH (structured exception - // handling) too, just find the appropriate keyword in their docs! - // Please note that it's _not_ the same as C++ exceptions! + // take everything into a try-except block in release build + // FIXME other compilers must support Win32 SEH (structured exception + // handling) too, just find the appropriate keyword in their docs! + // Please note that it's _not_ the same as C++ exceptions! #if !defined(__WXDEBUG__) && defined(__VISUALC__) #define CATCH_PROGRAM_EXCEPTIONS @@ -581,82 +605,89 @@ int wxEntry(WXHINSTANCE hInstance, #undef CATCH_PROGRAM_EXCEPTIONS #endif - wxhInstance = (HINSTANCE) hInstance; + wxhInstance = (HINSTANCE) hInstance; - if (!wxApp::Initialize()) - return 0; + if (!wxApp::Initialize()) + return 0; - // create the application object or ensure that one already exists - if (!wxTheApp) - { - // The app may have declared a global application object, but we recommend - // the IMPLEMENT_APP macro is used instead, which sets an initializer - // function for delayed, dynamic app object construction. - wxCHECK_MSG( wxApp::GetInitializerFunction(), 0, - "No initializer - use IMPLEMENT_APP macro." ); + // create the application object or ensure that one already exists + if (!wxTheApp) + { + // The app may have declared a global application object, but we recommend + // the IMPLEMENT_APP macro is used instead, which sets an initializer + // function for delayed, dynamic app object construction. + wxCHECK_MSG( wxApp::GetInitializerFunction(), 0, + "No initializer - use IMPLEMENT_APP macro." ); - wxTheApp = (*wxApp::GetInitializerFunction()) (); - } + wxTheApp = (*wxApp::GetInitializerFunction()) (); + } - wxCHECK_MSG( wxTheApp, 0, "You have to define an instance of wxApp!" ); + wxCHECK_MSG( wxTheApp, 0, "You have to define an instance of wxApp!" ); - // save the WinMain() parameters - wxTheApp->ConvertToStandardCommandArgs(lpCmdLine); - wxTheApp->m_nCmdShow = nCmdShow; + // save the WinMain() parameters + wxTheApp->ConvertToStandardCommandArgs(lpCmdLine); + wxTheApp->m_nCmdShow = nCmdShow; - // GUI-specific initialisation. In fact on Windows we don't have any, - // but this call is provided for compatibility across platforms. - wxTheApp->OnInitGui() ; + // GUI-specific initialisation. In fact on Windows we don't have any, + // but this call is provided for compatibility across platforms. + wxTheApp->OnInitGui() ; - int retValue = 0; + int retValue = 0; - if ( wxTheApp->OnInit() ) - { - if ( enterLoop ) - { - retValue = wxTheApp->OnRun(); - } - else - // We want to initialize, but not run or exit immediately. - return 1; - } - //else: app initialization failed, so we skipped OnRun() + if ( wxTheApp->OnInit() ) + { + if ( enterLoop ) + { + retValue = wxTheApp->OnRun(); + } + else + // We want to initialize, but not run or exit immediately. + return 1; + } + //else: app initialization failed, so we skipped OnRun() - wxWindow *topWindow = wxTheApp->GetTopWindow(); - if ( topWindow ) - { - // Forcibly delete the window. - if ( topWindow->IsKindOf(CLASSINFO(wxFrame)) || - topWindow->IsKindOf(CLASSINFO(wxDialog)) ) - { - topWindow->Close(TRUE); - wxTheApp->DeletePendingObjects(); - } - else - { - delete topWindow; - wxTheApp->SetTopWindow(NULL); - } - } + wxWindow *topWindow = wxTheApp->GetTopWindow(); + if ( topWindow ) + { + // Forcibly delete the window. + if ( topWindow->IsKindOf(CLASSINFO(wxFrame)) || + topWindow->IsKindOf(CLASSINFO(wxDialog)) ) + { + topWindow->Close(TRUE); + wxTheApp->DeletePendingObjects(); + } + else + { + delete topWindow; + wxTheApp->SetTopWindow(NULL); + } + } - wxTheApp->OnExit(); + wxTheApp->OnExit(); - wxApp::CleanUp(); + wxApp::CleanUp(); - return retValue; + return retValue; #ifdef CATCH_PROGRAM_EXCEPTIONS - } - __except ( EXCEPTION_EXECUTE_HANDLER ) { - /* - if ( wxTheApp ) - wxTheApp->OnFatalException(); - */ + } + __except ( EXCEPTION_EXECUTE_HANDLER ) { + /* + if ( wxTheApp ) + wxTheApp->OnFatalException(); + */ - ::ExitProcess(3); // the same exit code as abort() + // using wxLog would be unsafe here + ::MessageBox(NULL, + _("Unrecoverable program error detected: " + " the application will terminate."), + _("Fatal Error"), + MB_APPLMODAL | MB_ICONSTOP | MB_OK); - // NOTREACHED - } + ::ExitProcess(3); // the same exit code as abort() + + // NOTREACHED + } #endif // CATCH_PROGRAM_EXCEPTIONS } @@ -671,41 +702,36 @@ int wxEntry(WXHINSTANCE hInstance, int wxEntry(WXHINSTANCE hInstance) { - wxhInstance = (HINSTANCE) hInstance; - wxApp::Initialize(); + wxhInstance = (HINSTANCE) hInstance; + wxApp::Initialize(); - // The app may have declared a global application object, but we recommend - // the IMPLEMENT_APP macro is used instead, which sets an initializer function - // for delayed, dynamic app object construction. - - if (!wxTheApp) - { - if (!wxApp::GetInitializerFunction()) + // The app may have declared a global application object, but we recommend + // the IMPLEMENT_APP macro is used instead, which sets an initializer function + // for delayed, dynamic app object construction. + if (!wxTheApp) { - MessageBox(NULL, "No initializer - use IMPLEMENT_APP macro.", "wxWindows Error", MB_APPLMODAL | MB_ICONSTOP | MB_OK); - return 0; + wxCHECK_MSG( wxApp::GetInitializerFunction(), 0, + "No initializer - use IMPLEMENT_APP macro." ); + + wxTheApp = (* wxApp::GetInitializerFunction()) (); } - wxTheApp = (* wxApp::GetInitializerFunction()) (); - } + wxCHECK_MSG( wxTheApp, 0, "You have to define an instance of wxApp!" ); - if (!wxTheApp) { - MessageBox(NULL, "You have to define an instance of wxApp!", "wxWindows Error", MB_APPLMODAL | MB_ICONSTOP | MB_OK); - return 0; - } + wxTheApp->argc = 0; + wxTheApp->argv = NULL; - wxTheApp->argc = 0; - wxTheApp->argv = NULL; + wxTheApp->OnInitGui(); - wxTheApp->OnInitGui(); + wxTheApp->OnInit(); - wxTheApp->OnInit(); + wxWindow *topWindow = wxTheApp->GetTopWindow(); + if ( topWindow && topWindow->GetHWND()) + { + topWindow->Show(TRUE); + } - if (wxTheApp->GetTopWindow() && wxTheApp->GetTopWindow()->GetHWND()) { - wxTheApp->GetTopWindow()->Show(TRUE); - } - - return 1; + return 1; } #endif // _WINDLL @@ -715,43 +741,43 @@ wxAppInitializerFunction wxApp::m_appInitFn = (wxAppInitializerFunction) NULL; wxApp::wxApp() { - m_topWindow = NULL; - wxTheApp = this; - m_className = ""; - m_wantDebugOutput = TRUE ; - m_appName = ""; - argc = 0; - argv = NULL; + m_topWindow = NULL; + wxTheApp = this; + m_className = ""; + m_wantDebugOutput = TRUE ; + m_appName = ""; + argc = 0; + argv = NULL; #ifdef __WXMSW__ - m_printMode = wxPRINT_WINDOWS; + m_printMode = wxPRINT_WINDOWS; #else - m_printMode = wxPRINT_POSTSCRIPT; + m_printMode = wxPRINT_POSTSCRIPT; #endif - m_exitOnFrameDelete = TRUE; - m_auto3D = TRUE; + m_exitOnFrameDelete = TRUE; + m_auto3D = TRUE; } wxApp::~wxApp() { - // Delete command-line args - int i; - for (i = 0; i < argc; i++) - { - delete[] argv[i]; - } - delete[] argv; + // Delete command-line args + int i; + for (i = 0; i < argc; i++) + { + delete[] argv[i]; + } + delete[] argv; } bool wxApp::Initialized() { #ifndef _WINDLL - if (GetTopWindow()) - return TRUE; - else - return FALSE; + if (GetTopWindow()) + return TRUE; + else + return FALSE; #endif #ifdef _WINDLL // Assume initialized if DLL (no way of telling) - return TRUE; + return TRUE; #endif } @@ -794,7 +820,7 @@ bool wxApp::DoMessage() // leave out WM_COMMAND messages: too dangerous, sometimes // the message will be processed twice if ( !wxIsWaitingForThread() || - s_currentMsg.message != WM_COMMAND ) + s_currentMsg.message != WM_COMMAND ) { s_aSavedMessages.Add(s_currentMsg); } @@ -857,23 +883,23 @@ bool wxApp::DoMessage() int wxApp::MainLoop() { - m_keepGoing = TRUE; + m_keepGoing = TRUE; - while ( m_keepGoing ) - { - #if wxUSE_THREADS - wxMutexGuiLeaveOrEnter(); - #endif // wxUSE_THREADS - - while ( !::PeekMessage(&s_currentMsg, 0, 0, 0, PM_NOREMOVE) && - ProcessIdle() ) + while ( m_keepGoing ) { +#if wxUSE_THREADS + wxMutexGuiLeaveOrEnter(); +#endif // wxUSE_THREADS + + while ( !::PeekMessage(&s_currentMsg, 0, 0, 0, PM_NOREMOVE) && + ProcessIdle() ) + { + } + + DoMessage(); } - DoMessage(); - } - - return s_currentMsg.wParam; + return s_currentMsg.wParam; } // Returns TRUE if more time is needed. @@ -888,12 +914,12 @@ bool wxApp::ProcessIdle() void wxApp::ExitMainLoop() { - m_keepGoing = FALSE; + m_keepGoing = FALSE; } bool wxApp::Pending() { - return (::PeekMessage(&s_currentMsg, 0, 0, 0, PM_NOREMOVE) != 0) ; + return (::PeekMessage(&s_currentMsg, 0, 0, 0, PM_NOREMOVE) != 0) ; } void wxApp::Dispatch() @@ -908,33 +934,33 @@ void wxApp::Dispatch() */ bool wxApp::ProcessMessage(WXMSG *Msg) { - MSG *msg = (MSG *)Msg; + MSG *msg = (MSG *)Msg; - HWND hWnd; + HWND hWnd; - // Try translations first; find the youngest window with - // a translation table. - for (hWnd = msg->hwnd; hWnd != (HWND) NULL; hWnd = ::GetParent(hWnd)) - { - wxWindow *wnd = wxFindWinFromHandle((WXHWND) hWnd); - if (wnd) + // Try translations first; find the youngest window with + // a translation table. + for (hWnd = msg->hwnd; hWnd != (HWND) NULL; hWnd = ::GetParent(hWnd)) { - if (wnd->MSWTranslateMessage(Msg)) - return TRUE; + wxWindow *wnd = wxFindWinFromHandle((WXHWND) hWnd); + if (wnd) + { + if (wnd->MSWTranslateMessage(Msg)) + return TRUE; + } } - } - // Anyone for a non-translation message? Try youngest descendants first. - for (hWnd = msg->hwnd; hWnd != (HWND) NULL; hWnd = ::GetParent(hWnd)) - { - wxWindow *wnd = wxFindWinFromHandle((WXHWND) hWnd); - if (wnd) + // Anyone for a non-translation message? Try youngest descendants first. + for (hWnd = msg->hwnd; hWnd != (HWND) NULL; hWnd = ::GetParent(hWnd)) { - if (wnd->MSWProcessMessage(Msg)) - return TRUE; + wxWindow *wnd = wxFindWinFromHandle((WXHWND) hWnd); + if (wnd) + { + if (wnd->MSWProcessMessage(Msg)) + return TRUE; + } } - } - return FALSE; + return FALSE; } void wxApp::OnIdle(wxIdleEvent& event) @@ -986,43 +1012,43 @@ bool wxApp::SendIdleEvents() // Send idle event to window and all subwindows bool wxApp::SendIdleEvents(wxWindow* win) { - bool needMore = FALSE; + bool needMore = FALSE; - wxIdleEvent event; - event.SetEventObject(win); - win->GetEventHandler()->ProcessEvent(event); + wxIdleEvent event; + event.SetEventObject(win); + win->GetEventHandler()->ProcessEvent(event); - if (event.MoreRequested()) - needMore = TRUE; + if (event.MoreRequested()) + needMore = TRUE; - wxNode* node = win->GetChildren().First(); - while (node) - { - wxWindow* win = (wxWindow*) node->Data(); - if (SendIdleEvents(win)) - needMore = TRUE; + wxNode* node = win->GetChildren().First(); + while (node) + { + wxWindow* win = (wxWindow*) node->Data(); + if (SendIdleEvents(win)) + needMore = TRUE; - node = node->Next(); - } - return needMore ; + node = node->Next(); + } + return needMore ; } void wxApp::DeletePendingObjects() { - wxNode *node = wxPendingDelete.First(); - while (node) - { - wxObject *obj = (wxObject *)node->Data(); + wxNode *node = wxPendingDelete.First(); + while (node) + { + wxObject *obj = (wxObject *)node->Data(); - delete obj; + delete obj; - if (wxPendingDelete.Member(obj)) - delete node; + if (wxPendingDelete.Member(obj)) + delete node; - // Deleting one object may have deleted other pending - // objects, so start from beginning of list again. - node = wxPendingDelete.First(); - } + // Deleting one object may have deleted other pending + // objects, so start from beginning of list again. + node = wxPendingDelete.First(); + } } void wxApp::OnEndSession(wxCloseEvent& WXUNUSED(event)) @@ -1071,30 +1097,30 @@ int wxApp::GetComCtl32Version() const if (! theProc) { // not found, must be 4.00 - version = 400; + version = 400; } else { - // The following symbol are unique to 4.71 - // DllInstall - // FlatSB_EnableScrollBar FlatSB_GetScrollInfo FlatSB_GetScrollPos - // FlatSB_GetScrollProp FlatSB_GetScrollRange FlatSB_SetScrollInfo - // FlatSB_SetScrollPos FlatSB_SetScrollProp FlatSB_SetScrollRange - // FlatSB_ShowScrollBar - // _DrawIndirectImageList _DuplicateImageList - // InitializeFlatSB - // UninitializeFlatSB - // we could check for any of these - I chose DllInstall - FARPROC theProc = ::GetProcAddress(theModule, "DllInstall"); - if (! theProc) - { - // not found, must be 4.70 - version = 470; - } - else - { // found, must be 4.71 - version = 471; - } + // The following symbol are unique to 4.71 + // DllInstall + // FlatSB_EnableScrollBar FlatSB_GetScrollInfo FlatSB_GetScrollPos + // FlatSB_GetScrollProp FlatSB_GetScrollRange FlatSB_SetScrollInfo + // FlatSB_SetScrollPos FlatSB_SetScrollProp FlatSB_SetScrollRange + // FlatSB_ShowScrollBar + // _DrawIndirectImageList _DuplicateImageList + // InitializeFlatSB + // UninitializeFlatSB + // we could check for any of these - I chose DllInstall + FARPROC theProc = ::GetProcAddress(theModule, "DllInstall"); + if (! theProc) + { + // not found, must be 4.70 + version = 470; + } + else + { // found, must be 4.71 + version = 471; + } } } return version; @@ -1102,28 +1128,29 @@ int wxApp::GetComCtl32Version() const void wxExit() { - wxApp::CleanUp(); - FatalAppExit(0, "Fatal error: exiting"); + wxLogError(_("Fatal error: exiting")); + + wxApp::CleanUp(); } // Yield to incoming messages bool wxYield() { - MSG msg; - // We want to go back to the main message loop - // if we see a WM_QUIT. (?) - while (PeekMessage(&msg, NULL, 0, 0, PM_NOREMOVE) && msg.message != WM_QUIT) - { - if ( !wxTheApp->DoMessage() ) - break; - } + MSG msg; + // We want to go back to the main message loop + // if we see a WM_QUIT. (?) + while (PeekMessage(&msg, NULL, 0, 0, PM_NOREMOVE) && msg.message != WM_QUIT) + { + if ( !wxTheApp->DoMessage() ) + break; + } - return TRUE; + return TRUE; } HINSTANCE wxGetInstance() { - return wxhInstance; + return wxhInstance; } void wxSetInstance(HINSTANCE hInst) @@ -1134,5 +1161,5 @@ void wxSetInstance(HINSTANCE hInst) // For some reason, with MSVC++ 1.5, WinMain isn't linked in properly // if in a separate file. So include it here to ensure it's linked. #if (defined(__VISUALC__) && !defined(__WIN32__)) || (defined(__GNUWIN32__) && !defined(__TWIN32__)) - #include "main.cpp" +#include "main.cpp" #endif