cleanup: derive wxGauge from wxGaugeBase; removed redundant code from Create() (use...

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

diff --git a/include/wx/msw/gauge95.h b/include/wx/msw/gauge95.h
index 9bd85fd7d934f422cde4179d4399e735b3bc6085..052a7ae7129e8ada8cb597a6304d2e415c89b09c 100644 (file)
--- a/include/wx/msw/gauge95.h
+++ b/include/wx/msw/gauge95.h
@@ -23,51 +23,43 @@
 WXDLLEXPORT_DATA(extern const wxChar*) wxGaugeNameStr;
 
 // Group box
-class WXDLLEXPORT wxGauge95 : public wxControl
+class WXDLLEXPORT wxGauge95 : public wxGaugeBase
 {
 public:
-    wxGauge95(void) { m_rangeMax = 0; m_gaugePos = 0; }
-
-    wxGauge95(wxWindow *parent, wxWindowID id,
-            int range,
-            const wxPoint& pos = wxDefaultPosition,
-            const wxSize& size = wxDefaultSize,
-            long style = wxGA_HORIZONTAL,
-            const wxValidator& validator = wxDefaultValidator,
-            const wxString& name = wxGaugeNameStr)
+    wxGauge95() { }
+
+    wxGauge95(wxWindow *parent,
+              wxWindowID id,
+              int range,
+              const wxPoint& pos = wxDefaultPosition,
+              const wxSize& size = wxDefaultSize,
+              long style = wxGA_HORIZONTAL,
+              const wxValidator& validator = wxDefaultValidator,
+              const wxString& name = wxGaugeNameStr)
     {
-        Create(parent, id, range, pos, size, style, validator, name);
+        (void)Create(parent, id, range, pos, size, style, validator, name);
     }
 
-    bool Create(wxWindow *parent, wxWindowID id,
-            int range,
-            const wxPoint& pos = wxDefaultPosition,
-            const wxSize& size = wxDefaultSize,
-            long style = wxGA_HORIZONTAL,
-            const wxValidator& validator = wxDefaultValidator,
-            const wxString& name = wxGaugeNameStr);
-
-    void SetShadowWidth(int w);
-    void SetBezelFace(int w);
-    void SetRange(int r);
-    void SetValue(int pos);
+    bool Create(wxWindow *parent,
+                wxWindowID id,
+                int range,
+                const wxPoint& pos = wxDefaultPosition,
+                const wxSize& size = wxDefaultSize,
+                long style = wxGA_HORIZONTAL,
+                const wxValidator& validator = wxDefaultValidator,
+                const wxString& name = wxGaugeNameStr);
 
-    int GetShadowWidth(void) const ;
-    int GetBezelFace(void) const ;
-    int GetRange(void) const ;
-    int GetValue(void) const ;
-
-    bool SetForegroundColour(const wxColour& col);
-    bool SetBackgroundColour(const wxColour& col);
+    // set gauge range/value
+    virtual void SetRange(int range);
+    virtual void SetValue(int pos);
 
     // overriden base class virtuals
-    virtual bool AcceptsFocus() const { return FALSE; }
-
-    virtual void Command(wxCommandEvent& WXUNUSED(event)) {} ;
+    virtual bool SetForegroundColour(const wxColour& col);
+    virtual bool SetBackgroundColour(const wxColour& col);
 
 protected:
-    int      m_rangeMax;
-    int      m_gaugePos;
+    virtual WXDWORD MSWGetStyle(long style, WXDWORD *exstyle) const;
+    virtual wxSize DoGetBestSize() const;
 
 
     DECLARE_DYNAMIC_CLASS_NO_COPY(wxGauge95)

diff --git a/src/msw/gauge95.cpp b/src/msw/gauge95.cpp
index 9d8669063f5c617d0a627635961d6a46bb4ad9e9..9cc38eb1711c259454219353b5feb28308ab19e8 100644 (file)
--- a/src/msw/gauge95.cpp
+++ b/src/msw/gauge95.cpp
@@ -25,19 +25,19 @@
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
-#pragma hdrstop
+    #pragma hdrstop
 #endif
 
 #ifndef WX_PRECOMP
     #include "wx/defs.h"
 #endif
 
-#if wxUSE_GAUGE && defined(__WIN95__)
+#if wxUSE_GAUGE
 
 #include "wx/msw/gauge95.h"
 #include "wx/msw/private.h"
 
-#if defined(__WIN95__) && !(defined(__GNUWIN32_OLD__) && !defined(__CYGWIN10__))
+#if !(defined(__GNUWIN32_OLD__) && !defined(__CYGWIN10__))
     #include <commctrl.h>
 #endif
 
@@ -107,10 +107,10 @@ wxEND_FLAGS( wxGaugeStyle )
 IMPLEMENT_DYNAMIC_CLASS_XTI(wxGauge, wxControl,"wx/gauge.h")
 
 wxBEGIN_PROPERTIES_TABLE(wxGauge95)
-       wxPROPERTY( Value , int , SetValue, GetValue, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
-       wxPROPERTY( Range , int , SetRange, GetRange, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
-       wxPROPERTY( ShadowWidth , int , SetShadowWidth, GetShadowWidth, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
-       wxPROPERTY( BezelFace , int , SetBezelFace, GetBezelFace, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
+    wxPROPERTY( Value , int , SetValue, GetValue, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
+    wxPROPERTY( Range , int , SetRange, GetRange, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
+    wxPROPERTY( ShadowWidth , int , SetShadowWidth, GetShadowWidth, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
+    wxPROPERTY( BezelFace , int , SetBezelFace, GetBezelFace, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
     wxPROPERTY_FLAGS( WindowStyle , wxGaugeStyle , long , SetWindowStyleFlag , GetWindowStyleFlag , , 0 /*flags*/ , wxT("Helpstring") , wxT("group")) // style
 wxEND_PROPERTIES_TABLE()
 
@@ -123,139 +123,102 @@ IMPLEMENT_DYNAMIC_CLASS(wxGauge95, wxControl)
 #endif
 
 // ============================================================================
-// implementation
+// wxGauge95 implementation
 // ============================================================================
 
-bool wxGauge95::Create(wxWindow *parent, wxWindowID id,
-           int range,
-           const wxPoint& pos,
-           const wxSize& size,
-           long style,
-           const wxValidator& wxVALIDATOR_PARAM(validator),
-           const wxString& name)
-{
-  SetName(name);
-#if wxUSE_VALIDATORS
-  SetValidator(validator);
-#endif // wxUSE_VALIDATORS
-
-  if (parent) parent->AddChild(this);
-  m_gaugePos = 0;
-
-  m_windowStyle = style;
-
-  if ( id == -1 )
-      m_windowId = (int)NewControlId();
-  else
-      m_windowId = id;
-
-  int x = pos.x;
-  int y = pos.y;
-  int width = size.x;
-  int height = size.y;
-
-  WXDWORD exStyle = 0;
-  long msFlags = MSWGetStyle(style, & exStyle) ;
-
-  if (m_windowStyle & wxGA_VERTICAL)
-    msFlags |= PBS_VERTICAL;
-
-  if (m_windowStyle & wxGA_SMOOTH)
-    msFlags |= PBS_SMOOTH;
+// ----------------------------------------------------------------------------
+// wxGauge95 creation
+// ----------------------------------------------------------------------------
 
-  HWND wx_button =
-      CreateWindowEx(exStyle, PROGRESS_CLASS, NULL, msFlags,
-                    0, 0, 0, 0, (HWND) parent->GetHWND(), (HMENU)m_windowId,
-                    wxGetInstance(), NULL);
+bool wxGauge95::Create(wxWindow *parent,
+                       wxWindowID id,
+                       int range,
+                       const wxPoint& pos,
+                       const wxSize& size,
+                       long style,
+                       const wxValidator& validator,
+                       const wxString& name)
+{
+    if ( !CreateControl(parent, id, pos, size, style, validator, name) )
+        return false;
 
-  m_hWnd = (WXHWND)wx_button;
+    if ( !MSWCreateControl(PROGRESS_CLASS, wxEmptyString, pos, size) )
+        return false;
 
-  SetBackgroundColour(parent->GetBackgroundColour());
-  SetForegroundColour(parent->GetForegroundColour());
+    SetBackgroundColour(parent->GetBackgroundColour());
+    SetForegroundColour(parent->GetForegroundColour());
 
-  // Subclass again for purposes of dialog editing mode
-  SubclassWin((WXHWND) wx_button);
+    SetRange(range);
 
-  SetRange(range);
+    return true;
+}
 
-  SetFont(parent->GetFont());
+WXDWORD wxGauge95::MSWGetStyle(long style, WXDWORD *exstyle) const
+{
+    WXDWORD msStyle = wxControl::MSWGetStyle(style, exstyle);
 
-  if (width == -1)
-    width = 50;
-  if (height == -1)
-    height = 28;
-  SetSize(x, y, width, height);
+    if ( style & wxGA_VERTICAL )
+        msStyle |= PBS_VERTICAL;
 
-  ShowWindow(GetHwnd(), SW_SHOW);
+    if ( style & wxGA_SMOOTH )
+        msStyle |= PBS_SMOOTH;
 
-  return TRUE;
+    return msStyle;
 }
 
-void wxGauge95::SetShadowWidth(int WXUNUSED(w))
-{
-}
+// ----------------------------------------------------------------------------
+// wxGauge95 geometry
+// ----------------------------------------------------------------------------
 
-void wxGauge95::SetBezelFace(int WXUNUSED(w))
+wxSize wxGauge95::DoGetBestSize() const
 {
+    // VZ: no idea where does 28 come from, it was there before my changes and
+    //     as nobody ever complained I guess we can leave it...
+    return wxSize(100, 28);
 }
 
+// ----------------------------------------------------------------------------
+// wxGauge95 setters
+// ----------------------------------------------------------------------------
+
 void wxGauge95::SetRange(int r)
 {
-  m_rangeMax = r;
+    m_rangeMax = r;
 
 #ifdef PBM_SETRANGE32
-  SendMessage(GetHwnd(), PBM_SETRANGE32, 0, r);
+    ::SendMessage(GetHwnd(), PBM_SETRANGE32, 0, r);
 #else // !PBM_SETRANGE32
-  // fall back to PBM_SETRANGE (limited to 16 bits)
-  SendMessage(GetHwnd(), PBM_SETRANGE, 0, MAKELPARAM(0, r));
+    // fall back to PBM_SETRANGE (limited to 16 bits)
+    ::SendMessage(GetHwnd(), PBM_SETRANGE, 0, MAKELPARAM(0, r));
 #endif // PBM_SETRANGE32/!PBM_SETRANGE32
 }
 
 void wxGauge95::SetValue(int pos)
 {
-  m_gaugePos = pos;
-
-  SendMessage(GetHwnd(), PBM_SETPOS, pos, 0);
-}
-
-int wxGauge95::GetShadowWidth() const
-{
-  return 0;
-}
-
-int wxGauge95::GetBezelFace() const
-{
-  return 0;
-}
-
-int wxGauge95::GetRange() const
-{
-  return m_rangeMax;
-}
+    m_gaugePos = pos;
 
-int wxGauge95::GetValue() const
-{
-  return m_gaugePos;
+    ::SendMessage(GetHwnd(), PBM_SETPOS, pos, 0);
 }
 
 bool wxGauge95::SetForegroundColour(const wxColour& col)
 {
     if ( !wxControl::SetForegroundColour(col) )
-        return FALSE;
+        return false;
 
-    SendMessage(GetHwnd(), PBM_SETBARCOLOR, 0, (LPARAM)wxColourToRGB(col));
+    ::SendMessage(GetHwnd(), PBM_SETBARCOLOR, 0, (LPARAM)wxColourToRGB(col));
 
-    return TRUE;
+    return true;
 }
 
 bool wxGauge95::SetBackgroundColour(const wxColour& col)
 {
     if ( !wxControl::SetBackgroundColour(col) )
-        return FALSE;
+        return false;
 
-    SendMessage(GetHwnd(), PBM_SETBKCOLOR, 0, (LPARAM)wxColourToRGB(col));
+    ::SendMessage(GetHwnd(), PBM_SETBKCOLOR, 0, (LPARAM)wxColourToRGB(col));
 
-    return TRUE;
+    return true;
 }
 
 #endif // wxUSE_GAUGE
+