GNOME.org
 

[gtkmm] Gtkmm-forge digest, Vol 1 #177 - 7 msgs

Send Gtkmm-forge mailing list submissions to
	gtkmm-forge lists sourceforge net

To subscribe or unsubscribe via the World Wide Web, visit
	https://lists.sourceforge.net/lists/listinfo/gtkmm-forge
or, via email, send a message with subject or body 'help' to
	gtkmm-forge-request lists sourceforge net

You can reach the person managing the list at
	gtkmm-forge-admin lists sourceforge net

When replying, please edit your Subject line so it is more specific
than "Re: Contents of Gtkmm-forge digest..."


gtkmm-forge is the mailing list that receives gtkmm bug reports from bugzilla.  A daily digest is sent to gtkmm-main, to encourage people to help fixing the bugs.


Today's Topics:

   1. [Bug 89557] New - incorrect reference counting for gobject (bugzilla-daemon widget gnome org)
   2. [Bug 85979] Changed - template instantiation problem in TreeView::append_column under Sun Forte C++ (bugzilla-daemon widget gnome org)
   3. [Bug 87944] Changed - Review Menu API (bugzilla-daemon widget gnome org)
   4. [Bug 87944] Changed - Review Menu API (bugzilla-daemon widget gnome org)
   5. [Bug 87944] Changed - Review Menu API (bugzilla-daemon widget gnome org)
   6. [Bug 89557] Changed - incorrect reference counting for gobject (bugzilla-daemon widget gnome org)
   7. [Bug 89557] Changed - incorrect reference counting for gobject (bugzilla-daemon widget gnome org)

--__--__--

Message: 1
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Wed, 31 Jul 2002 15:36:12 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 89557] New - incorrect reference counting for gobject

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=89557

Changed by szombath bifab de 

--- shadow/89557	Wed Jul 31 15:36:12 2002
+++ shadow/89557.tmp.25647	Wed Jul 31 15:36:12 2002
@@ -0,0 +1,84 @@
+Bug#: 89557
+Product: gtkmm
+Version: 2.0
+OS: All
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: normal
+Priority: Normal
+Component: general
+AssignedTo: gtkmm-forge lists sourceforge net                            
+ReportedBy: szombath bifab de               
+TargetMilestone: ---
+URL: 
+Summary: incorrect reference counting for gobject
+
+We have discovered some irregularities in the gobject reference counting
+when using objects of short live time, e.g. dialogs. The sample program
+shows the behavior by incresing the number of toplevel windows. This
+vanishs when we apply the silly workaround by defining WITH_UNREF.
+
+The real reason for this misbehavior is that in the destructor of Object
+(gtk/gtkmm/object.cc) the reference counter will not be decremented if
+gobj() equals NULL. But other methods as disconnect_cpp_wrapper() which is
+called by Object::destroy_ for instance set gobj() to NULL, so that at the
+end the destructor forgets to decrease the reference counter resulting in
+the shown resource leak.
+
+Below the example program and test output:
+
+$ cat test.cc
+#include <gtkmm/main.h>
+#include <gtkmm/window.h>
+#include <gtkmm/dialog.h>
+#include <gtkmm/button.h>
+#include <iostream>
+
+using namespace std;
+using namespace Gtk;
+using SigC::slot;
+
+class MyDialog : public Dialog {
+public:
+  MyDialog() { add_button("Ok", 0); }
+  MyDialog::~MyDialog() {
+#ifdef WITH_UNREF
+    g_object_unref(G_OBJECT(gobj()));
+#endif
+  }
+};
+
+class MyWindow : public Window {
+public:
+  MyWindow() {
+    set_size_request(200, 200);
+    Box* mainBox = manage(new HBox);
+    Button* button = manage(new Button("Show Dialog"));
+    button->signal_clicked().connect(slot(*this, &MyWindow::ShowDialog));
+    mainBox->pack_start(*button);
+    add(*mainBox);
+  }
+  void ShowDialog() {
+    { 
+      MyDialog d; 
+      d.run();
+    }
+    list<Window*> toplevelwindows = list_toplevels();
+    cout << "toplevelwindows.size = " << toplevelwindows.size() << endl;
+  }
+};
+
+int main(int argc, char* argv[])
+{
+  Gtk::Main kit(argc, argv);
+  MyWindow win;
+  win.show_all();
+  kit.run(win);
+  return 0;
+}
+
+$ ./test
+toplevelwindows.size = 2
+toplevelwindows.size = 3
+toplevelwindows.size = 4



--__--__--

Message: 2
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Thu,  1 Aug 2002 05:19:02 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 85979] Changed - template instantiation problem in TreeView::append_column under Sun Forte C++

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=85979

Changed by szombath bifab de 

--- shadow/85979	Wed Jul 31 06:24:11 2002
+++ shadow/85979.tmp.30849	Thu Aug  1 05:19:02 2002
@@ -182,6 +182,23 @@
 reactions to my bug reports from the Sun Service give me some hope.
 
 ------- Additional Comments From murrayc usa net  2002-07-31 06:24 -------
 OK, if you can add columns without that particular method, as
 described, then I'll close this as not-a-bug, and add a comment to the
 documentation.
+
+------- Additional Comments From szombath bifab de  2002-08-01 05:19 -------
+We work around the compiler bug now this way:
+
+#ifdef __SUNPRO_CC
+    // Sun Forte C++ compiler can not handle template constructors
+    Gtk::TreeViewColumn c;
+    c.set_title(title);
+    c.pack_start(model);
+#else
+    Gtk::TreeViewColumn c(title, model);
+#endif
+    treeView.append_column(c);
+
+
+This will work for time Sun needs to fix their compiler bug, thus you
+can close it as not-a-bug in gtkmm.



--__--__--

Message: 3
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net, murrayc usa net
Cc: 
Date: Thu,  1 Aug 2002 10:02:04 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 87944] Changed - Review Menu API

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=87944

Changed by andreas holzmann epost de 

--- shadow/87944	Sat Jul 27 07:51:51 2002
+++ shadow/87944.tmp.21140	Thu Aug  1 10:02:04 2002
@@ -87,6 +87,11 @@
 changes the language.
 
 ------- Additional Comments From murrayc usa net  2002-07-15 12:09 -------
 I don't think that coders should have to specify an AccelGroup. Maybe
 they should have to specify a Gtk::Window, or we can find the
 Gtk::Window by recursively calling get_parent().
+
+------- Additional Comments From Andreas Holzmann epost de  2002-08-01 10:02 -------
+Created an attachment (id=10180)
+My modifications to the menu implementation
+



--__--__--

Message: 4
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net, murrayc usa net
Cc: 
Date: Thu,  1 Aug 2002 10:02:51 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 87944] Changed - Review Menu API

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=87944

Changed by andreas holzmann epost de 

--- shadow/87944	Thu Aug  1 10:02:04 2002
+++ shadow/87944.tmp.21164	Thu Aug  1 10:02:51 2002
@@ -92,6 +92,11 @@
 Gtk::Window by recursively calling get_parent().
 
 ------- Additional Comments From Andreas Holzmann epost de  2002-08-01 10:02 -------
 Created an attachment (id=10180)
 My modifications to the menu implementation
 
+
+------- Additional Comments From Andreas Holzmann epost de  2002-08-01 10:02 -------
+Created an attachment (id=10181)
+New file gtk/gtkmm/accelmap.cc
+



--__--__--

Message: 5
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net, murrayc usa net
Cc: 
Date: Thu,  1 Aug 2002 10:03:15 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 87944] Changed - Review Menu API

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=87944

Changed by andreas holzmann epost de 

--- shadow/87944	Thu Aug  1 10:02:51 2002
+++ shadow/87944.tmp.21257	Thu Aug  1 10:03:15 2002
@@ -97,6 +97,11 @@
 
 
 ------- Additional Comments From Andreas Holzmann epost de  2002-08-01 10:02 -------
 Created an attachment (id=10181)
 New file gtk/gtkmm/accelmap.cc
 
+
+------- Additional Comments From Andreas Holzmann epost de  2002-08-01 10:03 -------
+Created an attachment (id=10182)
+New file gtk/gtkmm/accelmap.h
+



--__--__--

Message: 6
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net, szombath bifab de
Cc: 
Date: Thu,  1 Aug 2002 14:51:31 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 89557] Changed - incorrect reference counting for gobject

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=89557

Changed by murrayc usa net 

--- shadow/89557	Wed Jul 31 15:36:12 2002
+++ shadow/89557.tmp.31094	Thu Aug  1 14:51:31 2002
@@ -79,6 +79,9 @@
 }
 
 $ ./test
 toplevelwindows.size = 2
 toplevelwindows.size = 3
 toplevelwindows.size = 4
+
+------- Additional Comments From murrayc usa net  2002-08-01 14:51 -------
+May I add this code to the gtkmm tests directory?



--__--__--

Message: 7
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Thu,  1 Aug 2002 14:54:26 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 89557] Changed - incorrect reference counting for gobject

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=89557

Changed by szombath bifab de 

--- shadow/89557	Thu Aug  1 14:51:31 2002
+++ shadow/89557.tmp.31406	Thu Aug  1 14:54:26 2002
@@ -82,6 +82,9 @@
 toplevelwindows.size = 2
 toplevelwindows.size = 3
 toplevelwindows.size = 4
 
 ------- Additional Comments From murrayc usa net  2002-08-01 14:51 -------
 May I add this code to the gtkmm tests directory?
+
+------- Additional Comments From szombath bifab de  2002-08-01 14:54 -------
+Of course.




--__--__--

_______________________________________________
Gtkmm-forge mailing list
Gtkmm-forge lists sourceforge net
https://lists.sourceforge.net/lists/listinfo/gtkmm-forge


End of Gtkmm-forge Digest
[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]