Fix memory leak when a spacer is added, and crash when a window is added before wxSiz...

[wxWidgets.git] / src / gtk1 / statline.cpp

diff --git a/src/gtk1/statline.cpp b/src/gtk1/statline.cpp
index 60970765ed978d3ad4ef064a181807f972d522f9..3f95edd803b53931fa465226d912df09c52624fa 100644 (file)
--- a/src/gtk1/statline.cpp
+++ b/src/gtk1/statline.cpp
@@ -7,10 +7,13 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
 #pragma implementation "statline.h"
 #endif
 
+// For compilers that support precompilation, includes "wx.h".
+#include "wx/wxprec.h"
+
 #include "wx/statline.h"
 
 #if wxUSE_STATLINE
@@ -45,37 +48,42 @@ bool wxStaticLine::Create( wxWindow *parent, wxWindowID id,
         !CreateBase( parent, id, pos, size, style, wxDefaultValidator, name ))
     {
         wxFAIL_MSG( wxT("wxStaticLine creation failed") );
-       return FALSE;
+        return FALSE;
     }
 
     if ( IsVertical() )
     {
         m_widget = gtk_vseparator_new();
-       if (size.x == -1)
-       {
-           wxSize new_size( size );
-           new_size.x = 4;
-           SetSize( new_size );
-       }
+        if (size.x == -1)
+        {
+            wxSize new_size( size );
+            new_size.x = 4;
+            SetSize( new_size );
+        }
     }
     else
     {
         m_widget = gtk_hseparator_new();
-       if (size.y == -1)
-       {
-           wxSize new_size( size );
-           new_size.y = 4;
-           SetSize( new_size );
-       }
+        if (size.y == -1)
+        {
+            wxSize new_size( size );
+            new_size.y = 4;
+            SetSize( new_size );
+        }
     }
 
     m_parent->DoAddChild( this );
 
-    PostCreation();
-
-    Show( TRUE );
+    PostCreation(size);
 
     return TRUE;
 }
 
+// static
+wxVisualAttributes
+wxStaticLine::GetClassDefaultAttributes(wxWindowVariant WXUNUSED(variant))
+{
+    return GetDefaultAttributesFromGTKWidget(gtk_vseparator_new);
+}
+
 #endif