Call CreateControl instead of CreateBase

[wxWidgets.git] / src / common / dynload.cpp

diff --git a/src/common/dynload.cpp b/src/common/dynload.cpp
index 9494d42fb97f701cfe8388f8edb0a213cecab97a..5bb2496dff8b44063a705259fd032bfa60f153ec 100644 (file)
--- a/src/common/dynload.cpp
+++ b/src/common/dynload.cpp
@@ -87,7 +87,7 @@ wxPluginLibrary::wxPluginLibrary(const wxString &libname, int flags)
 
     if( m_handle != 0 )
     {
 
     if( m_handle != 0 )
     {

-        UpdateClassInfo();
+        UpdateClasses();

         RegisterModules();
     }
     else
         RegisterModules();
     }
     else


@@ -102,7 +102,7 @@ wxPluginLibrary::~wxPluginLibrary()
     if( m_handle != 0 )
     {
         UnregisterModules();
     if( m_handle != 0 )
     {
         UnregisterModules();

-        RestoreClassInfo();
+        RestoreClasses();

     }
 }
 
     }
 }
 


@@ -120,7 +120,7 @@ bool wxPluginLibrary::UnrefLib()
     wxASSERT_MSG( m_objcount == 0,
                   _T("Library unloaded before all objects were destroyed") );
 
     wxASSERT_MSG( m_objcount == 0,
                   _T("Library unloaded before all objects were destroyed") );
 

-    if ( --m_linkcount == 0 )
+    if ( m_linkcount == 0 || --m_linkcount == 0 )

     {
         delete this;
         return TRUE;
     {
         delete this;
         return TRUE;


@@ -133,27 +133,12 @@ bool wxPluginLibrary::UnrefLib()
 // Private methods
 // ------------------------
 
 // Private methods
 // ------------------------
 

-void wxPluginLibrary::UpdateClassInfo()
+void wxPluginLibrary::UpdateClasses()

 {
 {

-    wxClassInfo     *info;
-    wxHashTable     *t = wxClassInfo::sm_classTable;
-
-        // FIXME: Below is simply a cut and paste specialisation of
-        //        wxClassInfo::InitializeClasses.  Once this stabilises,
-        //        the two should probably be merged.
-        //
-        //        Actually it's becoming questionable whether we should merge
-        //        this info with the main ClassInfo tables since we can nearly
-        //        handle this completely internally now and it does expose
-        //        certain (minimal % user_stupidy) risks.
-
-    for(info = m_after; info != m_before; info = info->m_next)
+    for (wxClassInfo *info = m_after; info != m_before; info = info->m_next)

     {
         if( info->m_className )
         {
     {
         if( info->m_className )
         {

-            if( t->Get(info->m_className) == 0 )
-                t->Put(info->m_className, (wxObject *)info);
-

             // Hash all the class names into a local table too so
             // we can quickly find the entry they correspond to.
             (*ms_classes)[info->m_className] = this;
             // Hash all the class names into a local table too so
             // we can quickly find the entry they correspond to.
             (*ms_classes)[info->m_className] = this;


@@ -161,27 +146,16 @@ void wxPluginLibrary::UpdateClassInfo()
     }
 }
 
     }
 }
 

-void wxPluginLibrary::RestoreClassInfo()
+void wxPluginLibrary::RestoreClasses()

 {
 {

-    wxClassInfo *info;
+    // Check if there is a need to restore classes.
+    if (!ms_classes)
+        return;

 
 

-    for(info = m_after; info != m_before; info = info->m_next)
+    for(wxClassInfo *info = m_after; info != m_before; info = info->m_next)

     {
     {

-        wxClassInfo::sm_classTable->Delete(info->m_className);

         ms_classes->erase(ms_classes->find(info->m_className));
     }
         ms_classes->erase(ms_classes->find(info->m_className));
     }

-
-    if( wxClassInfo::sm_first == m_after )
-        wxClassInfo::sm_first = m_before;
-    else
-    {
-        info = wxClassInfo::sm_first;
-        while( info->m_next && info->m_next != m_after ) info = info->m_next;
-
-        wxASSERT_MSG( info, _T("ClassInfo from wxPluginLibrary not found on purge"));
-
-        info->m_next = m_before;
-    }

 }
 
 void wxPluginLibrary::RegisterModules()
 }
 
 void wxPluginLibrary::RegisterModules()


@@ -252,7 +226,8 @@ void wxPluginLibrary::UnregisterModules()
     for ( it = m_wxmodules.begin(); it != m_wxmodules.end(); ++it )
         wxModule::UnregisterModule( *it );
 
     for ( it = m_wxmodules.begin(); it != m_wxmodules.end(); ++it )
         wxModule::UnregisterModule( *it );
 

-    WX_CLEAR_LIST(wxModuleList, m_wxmodules);
+    // NB: content of the list was deleted by UnregisterModule calls above:
+    m_wxmodules.clear();

 }
 
 
 }
 
 


@@ -487,7 +462,7 @@ wxLibrary::wxLibrary(wxDllType handle)
 
     // Some system may use a local heap for library.
     get_first = (t_get_first)GetSymbol(_T("wxGetClassFirst"));
 
     // Some system may use a local heap for library.
     get_first = (t_get_first)GetSymbol(_T("wxGetClassFirst"));

-    // It is a wxWindows DLL.
+    // It is a wxWidgets DLL.

     if (get_first)
         PrepareClasses(get_first());
 }
     if (get_first)
         PrepareClasses(get_first());
 }