streamlining OSX event support first step, see #11805, see #11797

[wxWidgets.git] / src / osx / carbon / settings.cpp

diff --git a/src/osx/carbon/settings.cpp b/src/osx/carbon/settings.cpp
index be6c68ca1b0a369a5fd781c141f61a6c36b1cd47..bc6b13ad73785b2891be80f89aec4f910e808f9e 100644 (file)
--- a/src/osx/carbon/settings.cpp
+++ b/src/osx/carbon/settings.cpp
@@ -19,6 +19,7 @@
 #endif
 
 #include "wx/osx/private.h"
 #endif
 
 #include "wx/osx/private.h"

+#include "wx/fontenum.h"

 
 // ----------------------------------------------------------------------------
 // wxSystemSettingsNative
 
 // ----------------------------------------------------------------------------
 // wxSystemSettingsNative


@@ -75,6 +76,7 @@ wxColour wxSystemSettingsNative::GetColour(wxSystemColour index)
         case wxSYS_COLOUR_CAPTIONTEXT:
         case wxSYS_COLOUR_INFOTEXT:
         case wxSYS_COLOUR_INACTIVECAPTIONTEXT:
         case wxSYS_COLOUR_CAPTIONTEXT:
         case wxSYS_COLOUR_INFOTEXT:
         case wxSYS_COLOUR_INACTIVECAPTIONTEXT:

+        case wxSYS_COLOUR_LISTBOXTEXT:

             resultColor = *wxBLACK;
             break ;
 
             resultColor = *wxBLACK;
             break ;
 


@@ -141,10 +143,12 @@ wxColour wxSystemSettingsNative::GetColour(wxSystemColour index)
         // case wxSYS_COLOUR_MAX:
         default:
             resultColor = *wxWHITE;
         // case wxSYS_COLOUR_MAX:
         default:
             resultColor = *wxWHITE;

-            // wxCHECK_MSG( index >= wxSYS_COLOUR_MAX, false, _T("unknown system colour index") );
+            // wxCHECK_MSG( index >= wxSYS_COLOUR_MAX, false, wxT("unknown system colour index") );

             break ;
     }
 
             break ;
     }
 

+    //wxASSERT(resultColor.IsOk());
+

     return resultColor;
 }
 
     return resultColor;
 }
 


@@ -154,20 +158,25 @@ wxColour wxSystemSettingsNative::GetColour(wxSystemColour index)
 
 wxFont wxSystemSettingsNative::GetFont(wxSystemFont index)
 {
 
 wxFont wxSystemSettingsNative::GetFont(wxSystemFont index)
 {

+    wxFont font;
+

     switch (index)
     {
         case wxSYS_ANSI_VAR_FONT :
         case wxSYS_SYSTEM_FONT :
         case wxSYS_DEVICE_DEFAULT_FONT :
         case wxSYS_DEFAULT_GUI_FONT :
     switch (index)
     {
         case wxSYS_ANSI_VAR_FONT :
         case wxSYS_SYSTEM_FONT :
         case wxSYS_DEVICE_DEFAULT_FONT :
         case wxSYS_DEFAULT_GUI_FONT :

-            return *wxSMALL_FONT ;
+            font = *wxSMALL_FONT ;

             break ;
 
         default :
             break ;
 
         default :

+            font = *wxNORMAL_FONT ;

             break ;
     }
 
             break ;
     }
 

-    return *wxNORMAL_FONT;
+    //wxASSERT(font.IsOk() && wxFontEnumerator::IsValidFacename(font.GetFaceName()));
+
+    return font;

 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------


@@ -243,7 +252,7 @@ int wxSystemSettingsNative::GetMetric(wxSystemMetric index, wxWindow* WXUNUSED(w
 #if wxOSX_USE_CARBON
             return (int)(GetDblTime() * 1000. / 60.);
 #else
 #if wxOSX_USE_CARBON
             return (int)(GetDblTime() * 1000. / 60.);
 #else

-            // default on mac is 30 ticks, we shouldn't really use wxSYS_DCLICK_MSEC anyway 
+            // default on mac is 30 ticks, we shouldn't really use wxSYS_DCLICK_MSEC anyway

             // but rather rely on the 'click-count' by the system delivered in a mouse event
             return 500;
 #endif
             // but rather rely on the 'click-count' by the system delivered in a mouse event
             return 500;
 #endif