added missing return on error

[wxWidgets.git] / src / os2 / settings.cpp

diff --git a/src/os2/settings.cpp b/src/os2/settings.cpp
index 3dee795eca02fd34142122bbdf887bda35184b0d..b2c3b1c7f3fb4a4af8aecf4f017fab589dc9d66a 100644 (file)
--- a/src/os2/settings.cpp
+++ b/src/os2/settings.cpp
@@ -24,18 +24,183 @@
 #include "wx/window.h"
 #include "wx/os2/private.h"
 
 #include "wx/window.h"
 #include "wx/os2/private.h"
 

-// TODO: see ::SystemParametersInfo for all sorts of Windows settings.
-// Different args are required depending on the id. How does this differ
-// from GetSystemMetric, and should it? Perhaps call it GetSystemParameter
-// and pass an optional void* arg to get further info.
-// Should also have SetSystemParameter.
-// Also implement WM_WININICHANGE (NT) / WM_SETTINGCHANGE (Win95)
-wxColour wxSystemSettings::GetSystemColour(int index)
+// the module which is used to clean up wxSystemSettings data (this is a
+// singleton class so it can't be done in the dtor)
+class wxSystemSettingsModule : public wxModule

 {
 {

-    // TODO
-    return wxColour();
+    friend class wxSystemSettings;
+public:
+    virtual bool OnInit();
+    virtual void OnExit();
+
+private:
+    DECLARE_DYNAMIC_CLASS(wxSystemSettingsModule)
+
+    static wxArrayString   sm_optionNames;
+    static wxArrayString   sm_optionValues;
+};
+
+// ----------------------------------------------------------------------------
+// global data
+// ----------------------------------------------------------------------------
+
+static wxFont *gs_fontDefault = NULL;
+
+// ============================================================================
+// implementation
+// ============================================================================
+
+// ----------------------------------------------------------------------------
+// wxSystemSettingsModule
+// ----------------------------------------------------------------------------
+
+IMPLEMENT_DYNAMIC_CLASS(wxSystemSettingsModule, wxModule)
+
+wxArrayString wxSystemSettingsModule::sm_optionNames;
+wxArrayString wxSystemSettingsModule::sm_optionValues;
+
+bool wxSystemSettingsModule::OnInit()
+{
+    return TRUE;

 }
 
 }
 

+void wxSystemSettingsModule::OnExit()
+{
+    sm_optionNames.Clear();
+    sm_optionValues.Clear();
+    delete gs_fontDefault;
+}
+
+wxColour wxSystemSettings::GetSystemColour(
+  int                               nIndex
+)
+{
+    COLORREF                        vRef;
+    wxColour                        vCol;
+    switch (nIndex)
+    {
+        //
+        // PM actually has values for these
+        //
+        case wxSYS_COLOUR_WINDOW:
+            vRef = (ULONG)::WinQuerySysColor( HWND_DESKTOP
+                                             ,SYSCLR_WINDOW
+                                             ,0L
+                                            );
+            vCol.Set( GetRValue(vRef)
+                     ,GetGValue(vRef)
+                     ,GetBValue(vRef)
+                    );
+            break;
+
+        case wxSYS_COLOUR_WINDOWFRAME:
+            vRef = (ULONG)::WinQuerySysColor( HWND_DESKTOP
+                                             ,SYSCLR_WINDOWFRAME
+                                             ,0L
+                                            );
+            vCol.Set( GetRValue(vRef)
+                     ,GetGValue(vRef)
+                     ,GetBValue(vRef)
+                    );
+            break;
+
+        case wxSYS_COLOUR_MENUTEXT:
+            vRef = (ULONG)::WinQuerySysColor( HWND_DESKTOP
+                                             ,SYSCLR_MENUTEXT
+                                             ,0L
+                                            );
+            vCol.Set( GetRValue(vRef)
+                     ,GetGValue(vRef)
+                     ,GetBValue(vRef)
+                    );
+            break;
+
+        case wxSYS_COLOUR_BTNFACE:
+            vRef = (ULONG)::WinQuerySysColor( HWND_DESKTOP
+                                             ,SYSCLR_BUTTONDEFAULT
+                                             ,0L
+                                            );
+            vCol.Set( GetRValue(vRef)
+                     ,GetGValue(vRef)
+                     ,GetBValue(vRef)
+                    );
+            break;
+
+        case wxSYS_COLOUR_BTNSHADOW:
+            vRef = (ULONG)::WinQuerySysColor( HWND_DESKTOP
+                                             ,SYSCLR_BUTTONMIDDLE
+                                             ,0L
+                                            );
+            vCol.Set( GetRValue(vRef)
+                     ,GetGValue(vRef)
+                     ,GetBValue(vRef)
+                    );
+            break;
+
+        case wxSYS_COLOUR_BTNHIGHLIGHT:
+            vRef = (ULONG)::WinQuerySysColor( HWND_DESKTOP
+                                             ,SYSCLR_BUTTONLIGHT
+                                             ,0L
+                                            );
+            vCol.Set( GetRValue(vRef)
+                     ,GetGValue(vRef)
+                     ,GetBValue(vRef)
+                    );
+            break;
+
+        //
+        // We'll have to just give values to these
+        //
+        case wxSYS_COLOUR_LISTBOX:
+        case wxSYS_COLOUR_CAPTIONTEXT:
+            return(*wxWHITE);
+            break;
+
+        case wxSYS_COLOUR_WINDOWTEXT:
+        case wxSYS_COLOUR_INACTIVECAPTIONTEXT:
+        case wxSYS_COLOUR_BTNTEXT:
+        case wxSYS_COLOUR_INFOTEXT:
+            vCol = (*wxBLACK);
+            break;
+
+        //
+        // We should customize these to look like other ports
+        //
+
+        case wxSYS_COLOUR_ACTIVECAPTION:
+        case wxSYS_COLOUR_ACTIVEBORDER:
+        case wxSYS_COLOUR_HIGHLIGHT:
+            vCol = (*wxBLUE);
+            break;
+
+        case wxSYS_COLOUR_SCROLLBAR:
+        case wxSYS_COLOUR_BACKGROUND:
+        case wxSYS_COLOUR_INACTIVECAPTION:
+        case wxSYS_COLOUR_MENU:
+        case wxSYS_COLOUR_INACTIVEBORDER:
+        case wxSYS_COLOUR_APPWORKSPACE:
+        case wxSYS_COLOUR_HIGHLIGHTTEXT:
+        case wxSYS_COLOUR_GRAYTEXT:
+        case wxSYS_COLOUR_3DDKSHADOW:
+        case wxSYS_COLOUR_3DLIGHT:
+        case wxSYS_COLOUR_INFOBK:
+            vCol = (*wxLIGHT_GREY);
+            break;
+
+        default:
+            vRef = (ULONG)::WinQuerySysColor( HWND_DESKTOP
+                                             ,SYSCLR_WINDOW
+                                             ,0L
+                                            );
+            vCol.Set( GetRValue(vRef)
+                     ,GetGValue(vRef)
+                     ,GetBValue(vRef)
+                    );
+            break;
+    }
+    return(vCol);
+} // end of wxSystemSettings::GetSystemColour
+

 wxFont wxSystemSettings::GetSystemFont(int index)
 {
     // TODO
 wxFont wxSystemSettings::GetSystemFont(int index)
 {
     // TODO


@@ -63,8 +228,10 @@ wxFont wxSystemSettings::GetSystemFont(int index)
             break;
         }
     }
             break;
         }
     }

+    if(wxSWISS_FONT)
+         return *wxSWISS_FONT;

 
 

-    return wxFont();
+    return wxNullFont;

 }
 
 // Get a system metric, e.g. scrollbar size
 }
 
 // Get a system metric, e.g. scrollbar size


@@ -189,3 +356,66 @@ int wxSystemSettings::GetSystemMetric(int index)
     return 0;
 }
 
     return 0;
 }
 

+// Option functions (arbitrary name/value mapping)
+void wxSystemSettings::SetOption(
+  const wxString&                   rsName
+, const wxString&                   rsValue
+)
+{
+    int                             nIdx = wxSystemSettingsModule::sm_optionNames.Index( rsName
+                                                                                        ,FALSE
+                                                                                       );
+
+    if (nIdx == wxNOT_FOUND)
+    {
+        wxSystemSettingsModule::sm_optionNames.Add(rsName);
+        wxSystemSettingsModule::sm_optionValues.Add(rsValue);
+    }
+    else
+    {
+        wxSystemSettingsModule::sm_optionNames[nIdx] = rsName;
+        wxSystemSettingsModule::sm_optionValues[nIdx] = rsValue;
+    }
+}
+
+void wxSystemSettings::SetOption(
+  const wxString&                   rsName
+, int                               nValue
+)
+{
+    wxString                        sValStr;
+
+    sValStr.Printf(wxT("%d"), nValue);
+    SetOption( rsName
+              ,sValStr
+             );
+} // end of
+
+wxString wxSystemSettings::GetOption(
+  const wxString&                   rsName
+)
+{
+    int                             nIdx = wxSystemSettingsModule::sm_optionNames.Index( rsName
+                                                                                        ,FALSE
+                                                                                       );
+
+    if (nIdx == wxNOT_FOUND)
+        return wxEmptyString;
+    else
+        return wxSystemSettingsModule::sm_optionValues[nIdx];
+} // end of
+
+int wxSystemSettings::GetOptionInt(
+  const wxString&                   rsName
+)
+{
+    return wxAtoi(GetOption(rsName));
+} // end of
+
+bool wxSystemSettings::HasOption(
+  const wxString&                   rsName
+)
+{
+    return (wxSystemSettingsModule::sm_optionNames.Index(rsName, FALSE) != wxNOT_FOUND);
+} // end of wxSystemSettings::HasOption
+