Warning fix about assigning unused value + little source cleaning.

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

diff --git a/src/motif/settings.cpp b/src/motif/settings.cpp
index 054bc38821852ae325e1d90801616403db9d7a17..03ab6e5a6e493d07c0e9c09cf2e6aa919f0d152f 100644 (file)
--- a/src/motif/settings.cpp
+++ b/src/motif/settings.cpp
@@ -165,7 +165,7 @@ wxFont wxSystemSettingsNative::GetFont(wxSystemFont index)
     {
         case wxSYS_SYSTEM_FIXED_FONT:
         {
-            return wxFont(12, wxMODERN, wxNORMAL, wxNORMAL, FALSE);
+            return wxFont(12, wxMODERN, wxNORMAL, wxNORMAL, false);
             break;
         }
         case wxSYS_DEVICE_DEFAULT_FONT:
@@ -173,7 +173,7 @@ wxFont wxSystemSettingsNative::GetFont(wxSystemFont index)
         case wxSYS_DEFAULT_GUI_FONT:
         default:
         {
-            return wxFont(12, wxSWISS, wxNORMAL, wxNORMAL, FALSE);
+            return wxFont(12, wxSWISS, wxNORMAL, wxNORMAL, false);
             break;
         }
     }
@@ -182,7 +182,7 @@ wxFont wxSystemSettingsNative::GetFont(wxSystemFont index)
 }
 
 // Get a system metric, e.g. scrollbar size
-int wxSystemSettingsNative::GetMetric(wxSystemMetric index)
+int wxSystemSettingsNative::GetMetric(wxSystemMetric index, wxWindow* WXUNUSED(win))
 {
     int return_value = 0;
 
@@ -191,80 +191,50 @@ int wxSystemSettingsNative::GetMetric(wxSystemMetric index)
         case wxSYS_HSCROLL_Y:
         case wxSYS_VSCROLL_X:
             return 15;
-        case wxSYS_MOUSE_BUTTONS:
-            // TODO
-        case wxSYS_BORDER_X:
-            // TODO
-        case wxSYS_BORDER_Y:
-            // TODO
-        case wxSYS_CURSOR_X:
-            // TODO
-        case wxSYS_CURSOR_Y:
-            // TODO
-        case wxSYS_DCLICK_X:
-            // TODO
-        case wxSYS_DCLICK_Y:
-            // TODO
-        case wxSYS_DRAG_X:
-            // TODO
-        case wxSYS_DRAG_Y:
-            // TODO
-        case wxSYS_EDGE_X:
-            // TODO
-        case wxSYS_EDGE_Y:
-            // TODO
-        case wxSYS_HSCROLL_ARROW_X:
-            // TODO
-        case wxSYS_HSCROLL_ARROW_Y:
-            // TODO
-        case wxSYS_HTHUMB_X:
-            // TODO
-        case wxSYS_ICON_X:
-            // TODO
-        case wxSYS_ICON_Y:
-            // TODO
-        case wxSYS_ICONSPACING_X:
-            // TODO
-        case wxSYS_ICONSPACING_Y:
-            // TODO
-        case wxSYS_WINDOWMIN_X:
-            // TODO
-        case wxSYS_WINDOWMIN_Y:
-            break;
+            
         case wxSYS_SCREEN_X:
             return_value = DisplayWidth( wxGlobalDisplay(), 0 );
             break;
         case wxSYS_SCREEN_Y:
             return_value = DisplayHeight( wxGlobalDisplay(), 0 );
             break;
-        case wxSYS_FRAMESIZE_X:
-            // TODO
-        case wxSYS_FRAMESIZE_Y:
-            // TODO
-        case wxSYS_SMALLICON_X:
-            // TODO
-        case wxSYS_SMALLICON_Y:
-            // TODO
-        case wxSYS_VSCROLL_ARROW_X:
-            // TODO
-        case wxSYS_VSCROLL_ARROW_Y:
-            // TODO
-        case wxSYS_VTHUMB_Y:
-            // TODO
-        case wxSYS_CAPTION_Y:
-            // TODO
-        case wxSYS_MENU_Y:
-            // TODO
-        case wxSYS_NETWORK_PRESENT:
-            // TODO
-        case wxSYS_PENWINDOWS_PRESENT:
-            // TODO
-        case wxSYS_SHOW_SOUNDS:
-            // TODO
-        case wxSYS_SWAP_BUTTONS:
-            // TODO
+
+        // TODO case wxSYS_MOUSE_BUTTONS:
+        // TODO case wxSYS_BORDER_X:
+        // TODO case wxSYS_BORDER_Y:
+        // TODO case wxSYS_CURSOR_X:
+        // TODO case wxSYS_CURSOR_Y:
+        // TODO case wxSYS_DCLICK_X:
+        // TODO case wxSYS_DCLICK_Y:
+        // TODO case wxSYS_DRAG_X:
+        // TODO case wxSYS_DRAG_Y:
+        // TODO case wxSYS_EDGE_X:
+        // TODO case wxSYS_EDGE_Y:
+        // TODO case wxSYS_HSCROLL_ARROW_X:
+        // TODO case wxSYS_HSCROLL_ARROW_Y:
+        // TODO case wxSYS_HTHUMB_X:
+        // TODO case wxSYS_ICON_X:
+        // TODO case wxSYS_ICON_Y:
+        // TODO case wxSYS_ICONSPACING_X:
+        // TODO case wxSYS_ICONSPACING_Y:
+        // TODO case wxSYS_WINDOWMIN_X:
+        // TODO case wxSYS_WINDOWMIN_Y:
+        // TODO case wxSYS_FRAMESIZE_X:
+        // TODO case wxSYS_FRAMESIZE_Y:
+        // TODO case wxSYS_SMALLICON_X:
+        // TODO case wxSYS_SMALLICON_Y:
+        // TODO case wxSYS_VSCROLL_ARROW_X:
+        // TODO case wxSYS_VSCROLL_ARROW_Y:
+        // TODO case wxSYS_VTHUMB_Y:
+        // TODO case wxSYS_CAPTION_Y:
+        // TODO case wxSYS_MENU_Y:
+        // TODO case wxSYS_NETWORK_PRESENT:
+        // TODO case wxSYS_PENWINDOWS_PRESENT:
+        // TODO case wxSYS_SHOW_SOUNDS:
+        // TODO case wxSYS_SWAP_BUTTONS:
+
         default:
-            ;
+            return_value = -1; // unsuported metric
     }
 
     return return_value;
@@ -276,9 +246,9 @@ bool wxSystemSettingsNative::HasFeature(wxSystemFeature index)
     {
         case wxSYS_CAN_ICONIZE_FRAME:
         case wxSYS_CAN_DRAW_FRAME_DECORATIONS:
-            return TRUE;
+            return true;
 
         default:
-            return FALSE;
+            return false;
     }
 }