From 096d5447522811eb785973a3b36266f899c135a3 Mon Sep 17 00:00:00 2001
From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Sat, 5 Jun 2010 22:57:59 +0000
Subject: [PATCH] No changes, just refactor wxListBox initialization in wxMSW.

Extract member fields initialization in Init() method instead of duplicating
it in default ctor and Create().

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@64497 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---
 include/wx/msw/listbox.h | 7 ++++++-
 src/msw/listbox.cpp      | 6 +-----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/include/wx/msw/listbox.h b/include/wx/msw/listbox.h
index 54a53b1c58..38b1afcd15 100644
--- a/include/wx/msw/listbox.h
+++ b/include/wx/msw/listbox.h
@@ -38,7 +38,7 @@ class WXDLLIMPEXP_CORE wxListBox : public wxListBoxBase
 {
 public:
     // ctors and such
-    wxListBox();
+    wxListBox() { Init(); }
     wxListBox(wxWindow *parent, wxWindowID id,
             const wxPoint& pos = wxDefaultPosition,
             const wxSize& size = wxDefaultSize,
@@ -47,6 +47,8 @@ public:
             const wxValidator& validator = wxDefaultValidator,
             const wxString& name = wxListBoxNameStr)
     {
+        Init();
+
         Create(parent, id, pos, size, n, choices, style, validator, name);
     }
     wxListBox(wxWindow *parent, wxWindowID id,
@@ -177,6 +179,9 @@ protected:
 #endif
 
 private:
+    // common part of all ctors
+    void Init();
+
     // call this when items are added to or deleted from the listbox or an
     // items text changes
     void MSWOnItemsChanged();
diff --git a/src/msw/listbox.cpp b/src/msw/listbox.cpp
index 8d2d266c72..8a2dd34eec 100644
--- a/src/msw/listbox.cpp
+++ b/src/msw/listbox.cpp
@@ -149,8 +149,7 @@ wxOwnerDrawn *wxListBox::CreateLboxItem(size_t WXUNUSED(n))
 // creation
 // ----------------------------------------------------------------------------
 
-// Listbox item
-wxListBox::wxListBox()
+void wxListBox::Init()
 {
     m_noItems = 0;
     m_updateHorizontalExtent = false;
@@ -165,9 +164,6 @@ bool wxListBox::Create(wxWindow *parent,
                        const wxValidator& validator,
                        const wxString& name)
 {
-    m_noItems = 0;
-    m_updateHorizontalExtent = false;
-
     // initialize base class fields
     if ( !CreateControl(parent, id, pos, size, style, validator, name) )
         return false;
-- 
2.45.2