1. reset default item in ~wxWindow (better to be safe)

[wxWidgets.git] / utils / serialize / serwnd.cpp

diff --git a/utils/serialize/serwnd.cpp b/utils/serialize/serwnd.cpp
index 1efa161336765ba31d937aa659ae2cf48889361a..a7790942cc5bfa28c5348e11186c683339709f36 100644 (file)
--- a/utils/serialize/serwnd.cpp
+++ b/utils/serialize/serwnd.cpp
@@ -26,6 +26,7 @@
 #include <wx/serbase.h>
 #include <wx/statusbr.h>
 #include <wx/mdi.h>
 #include <wx/serbase.h>
 #include <wx/statusbr.h>
 #include <wx/mdi.h>

+#include "wx/log.h"

 #include "serwnd.h"
 
 
 #include "serwnd.h"
 
 


@@ -49,15 +50,16 @@ IMPLEMENT_SERIAL_CLASS(wxMDIClientWindow, wxWindow)
 void WXSERIAL(wxWindow)::StoreObject(wxObjectOutputStream& s)
 {
   wxWindow *win_object = (wxWindow *)Object();
 void WXSERIAL(wxWindow)::StoreObject(wxObjectOutputStream& s)
 {
   wxWindow *win_object = (wxWindow *)Object();

-  wxNode *node = win_object->GetChildren()->First();

 
   if (s.FirstStage()) {
 
   if (s.FirstStage()) {

+    wxNode *node = win_object->GetChildren()->First();
+

     s.AddChild(win_object->GetConstraints());
     s.AddChild(win_object->GetValidator());
 
     // BAD HACK, but I don't have access to the internal variable of wxWindow.
     s.AddChild(win_object->GetConstraints());
     s.AddChild(win_object->GetValidator());
 
     // BAD HACK, but I don't have access to the internal variable of wxWindow.

-    m_bg_colour = win_object->GetDefaultBackgroundColour();
-    m_fg_colour = win_object->GetDefaultForegroundColour();
+    m_bg_colour = win_object->GetBackgroundColour();
+    m_fg_colour = win_object->GetForegroundColour();

     s.AddChild(&m_bg_colour);
     s.AddChild(&m_fg_colour);
     s.AddChild(win_object->GetFont());
     s.AddChild(&m_bg_colour);
     s.AddChild(&m_fg_colour);
     s.AddChild(win_object->GetFont());


@@ -71,14 +73,16 @@ void WXSERIAL(wxWindow)::StoreObject(wxObjectOutputStream& s)
   wxDataOutputStream data(s);
   int x,y,w,h;
 
   wxDataOutputStream data(s);
   int x,y,w,h;
 

-  data.WriteString(win_object->GetName());
-  data.WriteString(win_object->GetLabel());
-  data.WriteString(win_object->GetTitle());
+  data.WriteString( win_object->GetName() );
+  data.WriteString( win_object->GetLabel() );
+  data.WriteString( win_object->GetTitle() );

 
 

-  data.Write8(win_object->GetAutoLayout());
-  data.Write8(win_object->IsShown());
+  data.Write8( win_object->GetAutoLayout() );
+  data.Write8( win_object->IsShown() );

   data.Write32( win_object->GetWindowStyleFlag() );
   data.Write32( win_object->GetWindowStyleFlag() );

-  data.Write32(win_object->GetId());
+  data.Write32( win_object->GetId() );
+  wxLogDebug( "Number = %d", win_object->GetChildren()->Number() );
+  data.Write8( win_object->GetChildren()->Number() );

 
   win_object->GetSize(&w, &h);
   win_object->GetPosition(&x, &y);
 
   win_object->GetSize(&w, &h);
   win_object->GetPosition(&x, &y);


@@ -92,6 +96,26 @@ void WXSERIAL(wxWindow)::LoadObject(wxObjectInputStream& s)
 {
   wxDataInputStream data_s(s);
   wxWindow *win_object = (wxWindow *)Object();
 {
   wxDataInputStream data_s(s);
   wxWindow *win_object = (wxWindow *)Object();

+  wxColour *colour;
+  wxFont *font;
+
+  if (s.SecondCall()) {
+    /* I assume we will never create raw wxWindow object */
+    (void)s.GetChild(); // We pass wxLayoutConstraints.
+    (void)s.GetChild(); // We pass wxValidator.
+
+    colour = (wxColour *)s.GetChild();
+    if (colour)
+      win_object->SetBackgroundColour(*colour);
+    colour = (wxColour *)s.GetChild();
+    if (colour)
+      win_object->SetForegroundColour(*colour);
+    font = (wxFont *)s.GetChild();
+    if (font)
+      win_object->SetFont(*font);
+    s.RemoveChildren(m_number);
+    return;
+  }

 
   m_parent = (wxWindow *)s.GetParent();
 
 
   m_parent = (wxWindow *)s.GetParent();
 


@@ -103,18 +127,22 @@ void WXSERIAL(wxWindow)::LoadObject(wxObjectInputStream& s)
   m_shown       = data_s.Read8();
   m_style       = data_s.Read32();
   m_id          = data_s.Read32();
   m_shown       = data_s.Read8();
   m_style       = data_s.Read32();
   m_id          = data_s.Read32();

+  m_number      = data_s.Read8();

 
   m_x = data_s.Read16();
   m_y = data_s.Read16();
   m_w = data_s.Read16();
   m_h = data_s.Read16();
 
 
   m_x = data_s.Read16();
   m_y = data_s.Read16();
   m_w = data_s.Read16();
   m_h = data_s.Read16();
 

-  /* I assume we will never create raw wxWindow object */
+  (void)s.GetChild(); // We pass wxLayoutConstraints.

 
   m_validator = (wxValidator *)s.GetChild();
 
   m_validator = (wxValidator *)s.GetChild();

-  win_object->SetDefaultBackgroundColour(*((wxColour *)s.GetChild()));
-  win_object->SetDefaultForegroundColour(*((wxColour *)s.GetChild()));
-  win_object->SetFont(*((wxFont *)s.GetChild()));
+  if (!m_validator)
+    m_validator = (wxValidator *)&wxDefaultValidator;
+
+  s.RemoveChildren(m_number+3);
+
+  s.Recall();

 
   return;
 }
 
   return;
 }


@@ -242,12 +270,15 @@ void WXSERIAL(wxFrame)::LoadObject(wxObjectInputStream& s)
 
   WXSERIAL(wxWindow)::LoadObject(s);
 
 
   WXSERIAL(wxWindow)::LoadObject(s);
 

+  if (s.SecondCall())
+    return;
+

   wxDataInputStream data_s(s);
 
   wxDataInputStream data_s(s);
 

-  frame->SetMenuBar(mbar);

   if (frame->GetClassInfo() == CLASSINFO(wxFrame))
     frame->Create(m_parent, m_id, m_title, wxPoint(m_x, m_y),
                   wxSize(m_w, m_h), m_style, m_name);
   if (frame->GetClassInfo() == CLASSINFO(wxFrame))
     frame->Create(m_parent, m_id, m_title, wxPoint(m_x, m_y),
                   wxSize(m_w, m_h), m_style, m_name);

+  frame->SetMenuBar(mbar);

 
   frame->CreateStatusBar(data_s.Read8());
 }
 
   frame->CreateStatusBar(data_s.Read8());
 }


@@ -328,17 +359,21 @@ void WXSERIAL(wxMenuItem)::StoreObject(wxObjectOutputStream& s)
   wxMenuItem *item = (wxMenuItem *)Object();
 
   if (s.FirstStage()) {
   wxMenuItem *item = (wxMenuItem *)Object();
 
   if (s.FirstStage()) {

+#ifdef __WXGTK__

     s.AddChild(item->GetSubMenu());
     s.AddChild(item->GetSubMenu());

+#endif

     return;
   }
 
   wxDataOutputStream data_s(s);
 
     return;
   }
 
   wxDataOutputStream data_s(s);
 

+#ifdef __WXGTK__

   data_s.Write8( item->GetId() );
   data_s.WriteString( item->GetText() );
   data_s.Write8( item->IsCheckable() );
   data_s.Write8( item->IsEnabled() );
   data_s.Write8( item->IsChecked() );
   data_s.Write8( item->GetId() );
   data_s.WriteString( item->GetText() );
   data_s.Write8( item->IsCheckable() );
   data_s.Write8( item->IsEnabled() );
   data_s.Write8( item->IsChecked() );

+#endif

 }
 
 void WXSERIAL(wxMenuItem)::LoadObject(wxObjectInputStream& s)
 }
 
 void WXSERIAL(wxMenuItem)::LoadObject(wxObjectInputStream& s)


@@ -346,12 +381,14 @@ void WXSERIAL(wxMenuItem)::LoadObject(wxObjectInputStream& s)
   wxMenuItem *item = (wxMenuItem *)Object();
   wxDataInputStream data_s(s);
 
   wxMenuItem *item = (wxMenuItem *)Object();
   wxDataInputStream data_s(s);
 

+#ifdef __WXGTK__

   item->SetId( data_s.Read8() );
   item->SetText( data_s.ReadString() );
   item->SetCheckable( data_s.Read8() );
   item->Enable( data_s.Read8() );
   item->Check( data_s.Read8() );
   item->SetSubMenu( (wxMenu *)s.GetChild() );
   item->SetId( data_s.Read8() );
   item->SetText( data_s.ReadString() );
   item->SetCheckable( data_s.Read8() );
   item->Enable( data_s.Read8() );
   item->Check( data_s.Read8() );
   item->SetSubMenu( (wxMenu *)s.GetChild() );

+#endif

 }
 
 /////////////////////////////////////////////////////////////////////////////
 }
 
 /////////////////////////////////////////////////////////////////////////////


@@ -365,6 +402,9 @@ void WXSERIAL(wxPanel)::LoadObject(wxObjectInputStream& s)
 {
   WXSERIAL(wxWindow)::LoadObject(s);
 
 {
   WXSERIAL(wxWindow)::LoadObject(s);
 

+  if (s.SecondCall())
+    return;
+

   ((wxPanel *)Object())->Create(m_parent, m_id, wxPoint(m_x, m_y),
                                 wxSize(m_w, m_h), m_style, m_name);
 }
   ((wxPanel *)Object())->Create(m_parent, m_id, wxPoint(m_x, m_y),
                                 wxSize(m_w, m_h), m_style, m_name);
 }


@@ -380,6 +420,9 @@ void WXSERIAL(wxDialog)::LoadObject(wxObjectInputStream& s)
 {
   WXSERIAL(wxWindow)::LoadObject(s);
 
 {
   WXSERIAL(wxWindow)::LoadObject(s);
 

+  if (s.SecondCall())
+    return;
+

   ((wxDialog *)Object())->Create(m_parent, m_id, m_title, wxPoint(m_x, m_y),
                                  wxSize(m_w, m_h), m_style, m_name);
 }
   ((wxDialog *)Object())->Create(m_parent, m_id, m_title, wxPoint(m_x, m_y),
                                  wxSize(m_w, m_h), m_style, m_name);
 }


@@ -404,6 +447,11 @@ void WXSERIAL(wxMDIParentFrame)::LoadObject(wxObjectInputStream& s)
   wxMDIParentFrame *frame = (wxMDIParentFrame *)Object();
   wxMDIClientWindow *client;
 
   wxMDIParentFrame *frame = (wxMDIParentFrame *)Object();
   wxMDIClientWindow *client;
 

+  if (s.SecondCall()) {
+    WXSERIAL(wxFrame)::LoadObject(s);
+    return;
+  }
+

   client = (wxMDIClientWindow *) s.GetChild();
 
   frame->Create(m_parent, m_id, m_title, wxPoint(m_x, m_y),
   client = (wxMDIClientWindow *) s.GetChild();
 
   frame->Create(m_parent, m_id, m_title, wxPoint(m_x, m_y),


@@ -423,6 +471,9 @@ void WXSERIAL(wxMDIChildFrame)::LoadObject(wxObjectInputStream& s)
 {
   WXSERIAL(wxFrame)::LoadObject(s);
 
 {
   WXSERIAL(wxFrame)::LoadObject(s);
 

+  if (s.SecondCall())
+    return;
+

   ((wxMDIChildFrame *)Object())->Create((wxMDIParentFrame *)m_parent,
                                         m_id, m_title,
                                         wxPoint(m_x, m_y), wxSize(m_w, m_h),
   ((wxMDIChildFrame *)Object())->Create((wxMDIParentFrame *)m_parent,
                                         m_id, m_title,
                                         wxPoint(m_x, m_y), wxSize(m_w, m_h),


@@ -440,5 +491,8 @@ void WXSERIAL(wxMDIClientWindow)::LoadObject(wxObjectInputStream& s)
 {
   WXSERIAL(wxWindow)::LoadObject(s);
 
 {
   WXSERIAL(wxWindow)::LoadObject(s);
 

+  if (s.SecondCall())
+    return;
+

   ((wxMDIClientWindow *)Object())->CreateClient((wxMDIParentFrame *)m_parent, m_style);
 }
   ((wxMDIClientWindow *)Object())->CreateClient((wxMDIParentFrame *)m_parent, m_style);
 }