fixed bug: fixed two of the 'quark > 0' console errors.

diff --git a/src/extension/extension.cpp b/src/extension/extension.cpp
index 97d71c5ee2ffba31a6e21d71d38cf5d1df5263a5..631ab43fe2e61703d5195f4867601c7040e89811 100644 (file)
--- a/src/extension/extension.cpp
+++ b/src/extension/extension.cpp
@@ -131,7 +131,15 @@ Extension::~Extension (void)
     delete timer;
     timer = NULL;
     /** \todo Need to do parameters here */
-
+    
+    // delete parameters: 
+    for (GSList * list = parameters; list != NULL; list = g_slist_next(list)) {
+        Parameter * param = reinterpret_cast<Parameter *>(list->data);
+        delete param;
+    }
+    g_slist_free(parameters);
+    
+    
     for (unsigned int i = 0 ; i < _deps.size(); i++) {
         delete _deps[i];
     }
@@ -600,7 +608,6 @@ public:
         this->pack_start(*widg, true, true, 2);
         if (tooltip != NULL) {
             _tooltips.set_tip(*widg, Glib::ustring(tooltip));
-            // printf("Setting tooltip: %s\n", tooltip);
         }
         return;
     };

diff --git a/src/extension/paramnotebook.cpp b/src/extension/paramnotebook.cpp
index eb568ac45b7524aabd8b4dc2488691d3f1fcaa47..cd8bddcaa0e435ceb59de3bafaa41d08922bd448 100644 (file)
--- a/src/extension/paramnotebook.cpp
+++ b/src/extension/paramnotebook.cpp
@@ -47,7 +47,7 @@ private:
     GSList * parameters; /**< A table to store the parameters for this page.\r
                               This only gets created if there are parameters on this\r
                               page */\r
-    Gtk::Tooltips _tooltips;\r
+    Gtk::Tooltips * _tooltips;\r
     \r
 public:\r
     static ParamNotebookPage * makepage (Inkscape::XML::Node * in_repr, Inkscape::Extension::Extension * in_ext);\r
@@ -65,19 +65,18 @@ ParamNotebookPage::ParamNotebookPage (const gchar * name, const gchar * guitext,
     Parameter(name, guitext, desc, scope, ext)\r
 {\r
     parameters = NULL;\r
-    \r
+        \r
     // Read XML to build page\r
     if (xml != NULL) {\r
         Inkscape::XML::Node *child_repr = sp_repr_children(xml);\r
         while (child_repr != NULL) {\r
             char const * chname = child_repr->name();\r
-            if (chname[0] == '_') /* Allow _ for translation of tags */\r
+            if (chname[0] == '_') // Allow _ for translation of tags\r
                 chname++;\r
             if (!strcmp(chname, "param") || !strcmp(chname, "_param")) {\r
                 Parameter * param;\r
                 param = Parameter::make(child_repr, ext);\r
-                if (param != NULL)\r
-                    parameters = g_slist_append(parameters, param);\r
+                if (param != NULL) parameters = g_slist_append(parameters, param);\r
             }\r
             child_repr = sp_repr_next(child_repr);\r
         }\r
@@ -88,6 +87,7 @@ ParamNotebookPage::ParamNotebookPage (const gchar * name, const gchar * guitext,
 \r
 ParamNotebookPage::~ParamNotebookPage (void)\r
 {\r
+    if (_tooltips) delete _tooltips;\r
     //destroy parameters\r
     for (GSList * list = parameters; list != NULL; list = g_slist_next(list)) {\r
         Parameter * param = reinterpret_cast<Parameter *>(list->data);\r
@@ -188,7 +188,9 @@ ParamNotebookPage::makepage (Inkscape::XML::Node * in_repr, Inkscape::Extension:
 */\r
 Gtk::Widget *\r
 ParamNotebookPage::get_widget (SPDocument * doc, Inkscape::XML::Node * node)\r
-{\r
+{                      \r
+    if (!_tooltips) _tooltips = new Gtk::Tooltips();\r
+    \r
     Gtk::VBox * vbox = Gtk::manage(new Gtk::VBox);\r
     vbox->set_border_width(5);  \r
     \r
@@ -200,8 +202,7 @@ ParamNotebookPage::get_widget (SPDocument * doc, Inkscape::XML::Node * node)
         \r
         vbox->pack_start(*widg, true, true, 2);\r
         if (tip != NULL) {\r
-            _tooltips.set_tip(*widg, Glib::ustring(tip));\r
-            // printf("Setting tooltip: %s\n", tooltip);\r
+            _tooltips->set_tip(*widg, Glib::ustring(tip));\r
         }\r
     }\r
         \r
@@ -224,12 +225,11 @@ ParamNotebook::ParamNotebook (const gchar * name, const gchar * guitext, const g
     pages = NULL;\r
     \r
     // Read XML tree to add pages:\r
-    // printf("Extension Constructor: ");\r
     if (xml != NULL) {\r
         Inkscape::XML::Node *child_repr = sp_repr_children(xml);\r
         while (child_repr != NULL) {\r
             char const * chname = child_repr->name();\r
-            if (chname[0] == '_') /* Allow _ for translation of tags */\r
+            if (chname[0] == '_') // Allow _ for translation of tags\r
                 chname++;\r
             if (!strcmp(chname, "page")) {\r
                 ParamNotebookPage * page;\r