reworked wxBuildOptions code to work even w/o function inlining

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@22576 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

include/wx/app.h

@@ -261,8 +261,8 @@ public:
     // check that the wxBuildOptions object (constructed in the application
     // itself, usually the one from IMPLEMENT_APP() macro) matches the build
     // options of the library and abort if it doesn't
-    static bool CheckBuildOptions(const wxBuildOptions& buildOptions);
+    static bool CheckBuildOptions(const char *optionsSignature,
-
+                                  const char *componentName);
 
     // implementation only from now on
     // -------------------------------
@@ -628,7 +628,8 @@ public:
 #define IMPLEMENT_APP_NO_MAIN(appname)                                      \
     wxAppConsole *wxCreateApp()                                             \
     {                                                                       \
-        wxAppConsole::CheckBuildOptions(wxBuildOptions());                  \
+        wxAppConsole::CheckBuildOptions(WX_BUILD_OPTIONS_SIGNATURE,         \
+                                        wxT("your program"));               \
         return new appname;                                                 \
     }                                                                       \
     wxAppInitializer                                                        \

include/wx/build.h

@@ -1,7 +1,7 @@
 ///////////////////////////////////////////////////////////////////////////////
 // Name:        wx/build.h
-// Purpose:     wxBuildOptions class declaration
+// Purpose:     Runtime build options checking
-// Author:      Vadim Zeitlin
+// Author:      Vadim Zeitlin, Vaclav Slavik
 // Modified by:
 // Created:     07.05.02
 // RCS-ID:      $Id$
@@ -14,44 +14,48 @@
 
 #include "wx/version.h"
 
-class WXDLLIMPEXP_BASE wxAppConsole;
-
 // ----------------------------------------------------------------------------
-// wxBuildOptions
+// WX_BUILD_OPTIONS_SIGNATURE
 // ----------------------------------------------------------------------------
 
-class wxBuildOptions
+#define __WX_BO_STRINGIZE(x)  #x
-{
+
-public:
+#if (wxMINOR_VERSION % 2) == 0
-    // the ctor must be inline to get the compilation settings of the code
+    #define __WX_BO_VERSION(x,y,z) \
-    // which included this header
+        __WX_BO_STRINGIZE(x) "." __WX_BO_STRINGIZE(y)
-    wxBuildOptions()
-    {
-        // debug/release
-#ifdef __WXDEBUG__
-        m_isDebug = TRUE;
 #else
-        m_isDebug = FALSE;
+    #define __WX_BO_VERSION(x,y,z) \
+        __WX_BO_STRINGIZE(x) "." __WX_BO_STRINGIZE(y) "." __WX_BO_STRINGIZE(z)
 #endif
 
-        // version: we don't test the micro version as hopefully changes
+#ifdef __WXDEBUG__
-        // between 2 micro versions don't result in fatal compatibility
+    #define __WX_BO_DEBUG "debug"
-        // problems
+#else
-        m_verMaj = wxMAJOR_VERSION;
+    #define __WX_BO_DEBUG "no debug"
-        m_verMin = wxMINOR_VERSION;
+#endif
-    }
 
-private:
+#if wxUSE_UNICODE
-    // the version
+    #define __WX_BO_UNICODE "Unicode"
-    int m_verMaj,
+#else
-        m_verMin;
+    #define __WX_BO_UNICODE "ANSI"
+#endif
     
-    // compiled with __WXDEBUG__?
+// This macro is passed as argument to wxConsoleApp::CheckBuildOptions()
-    bool m_isDebug;
+#define WX_BUILD_OPTIONS_SIGNATURE \
+    __WX_BO_VERSION(wxMAJOR_VERSION, wxMINOR_VERSION, wxRELEASE_NUMBER) \
+    " (" __WX_BO_DEBUG "," __WX_BO_UNICODE ")"
 
-    // actually only CheckBuildOptions() should be our friend but well...
+
-    friend class wxAppConsole;
+
-};
+// Use this macro to check build options. Adding it to a file in DLL will
+// ensure that the DLL checks build options in same way IMPLEMENT_APP() does.
+#define WX_CHECK_BUILD_OPTIONS(libName)                                 \
+    static bool wxCheckBuildOptions()                                   \
+    {                                                                   \
+        wxAppConsole::CheckBuildOptions(WX_BUILD_OPTIONS_SIGNATURE,     \
+                                        libName);                       \
+        return true;                                                    \
+    };                                                                  \
+    static bool gs_buildOptionsCheck = wxCheckBuildOptions();
 
 #endif // _WX_BUILD_H_
-

src/common/appbase.cpp

@@ -357,37 +357,23 @@ bool wxAppConsole::OnCmdLineError(wxCmdLineParser& parser)
 // ----------------------------------------------------------------------------
 
 /* static */
-bool wxAppConsole::CheckBuildOptions(const wxBuildOptions& opts)
+bool wxAppConsole::CheckBuildOptions(const char *optionsSignature,
+                                     const char *componentName)
 {
-#define wxCMP(what)   (what == opts.m_ ## what)
+#if 0 // can't use wxLogTrace, not up and running yet
-
+    printf("checking build options object '%s' (ptr %p) in '%s'\n",
-    bool
+             optionsSignature, optionsSignature, componentName);
-#ifdef __WXDEBUG__
-    isDebug = TRUE;
-#else
-    isDebug = FALSE;
 #endif
 
-    int verMaj = wxMAJOR_VERSION,
+    if ( strcmp(optionsSignature, WX_BUILD_OPTIONS_SIGNATURE) != 0 )
-        verMin = wxMINOR_VERSION;
-
-    if ( !(wxCMP(isDebug) && wxCMP(verMaj) && wxCMP(verMin)) )
     {
+        wxString lib = wxString::FromAscii(WX_BUILD_OPTIONS_SIGNATURE);
+        wxString prog = wxString::FromAscii(optionsSignature);
+        wxString progName = wxString::FromAscii(componentName);
         wxString msg;
-        wxString libDebug, progDebug;
         
-        if (isDebug)
+        msg.Printf(_T("Mismatch between the program and library build versions detected.\nThe library used %s, and %s used %s."),
-            libDebug = wxT("debug");
+                   lib.c_str(), progName.c_str(), prog.c_str());
-        else
-            libDebug = wxT("no debug");
-
-        if (opts.m_isDebug)
-            progDebug = wxT("debug");
-        else
-            progDebug = wxT("no debug");
-        
-        msg.Printf(_T("Mismatch between the program and library build versions detected.\nThe library used %d.%d (%s), and your program used %d.%d (%s)."),
-                   verMaj, verMin, libDebug.c_str(), opts.m_verMaj, opts.m_verMin, progDebug.c_str());
         
         wxLogFatalError(msg);