GNOME.org
 

[gtkmm] Gtkmm-forge digest, Vol 1 #271 - 5 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 97494] New - Book chapter 8 errors (bugzilla-daemon widget gnome org)
   2. [Bug 97494] Changed - Book chapter 8 errors (bugzilla-daemon widget gnome org)
   3. [Bug 97498] New - Gtk::WindowGroup::create() is missing (bugzilla-daemon widget gnome org)
   4. [Bug 97498] Changed - Gtk::WindowGroup::create() is missing (bugzilla-daemon widget gnome org)
   5. [Bug 97494] Changed - Book chapter 8 errors (bugzilla-daemon widget gnome org)

--__--__--

Message: 1
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Sat,  2 Nov 2002 12:27:28 -0500 (EST)
Subject: [gtkmm bugzilla] [Bug 97494] New - Book chapter 8 errors

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=97494

Changed by olle endforward com 

--- shadow/97494	Sat Nov  2 12:27:28 2002
+++ shadow/97494.tmp.987	Sat Nov  2 12:27:28 2002
@@ -0,0 +1,35 @@
+Bug#: 97494
+Product: gtkmm
+Version: 2.0
+OS: other
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: minor
+Priority: Normal
+Component: reference documentation
+AssignedTo: gtkmm-forge lists sourceforge net                            
+ReportedBy: olle endforward com               
+QAContact: gtkmm-forge lists sourceforge net
+TargetMilestone: ---
+URL: 
+Summary: Book chapter 8 errors
+
+Chapter 8 of "the" book contains two errors.
+
+http://www.gtkmm.org/gtkmm2/docs/tutorial/html/ch08.html#id2834309
+
+  ModelColumns()
+    { add(m_col_text); add(m_col_number) }
+
+should be 
+
+  ModelColumns()
+    { add(m_col_text); add(m_col_number); }
+
+and in Model Columns 
+
+the object is called refListStore
+
+then in adding rows it is called m_refListStore which is seems what it
+should be.



--__--__--

Message: 2
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net, olle endforward com
Cc: 
Date: Sat,  2 Nov 2002 12:44:57 -0500 (EST)
Subject: [gtkmm bugzilla] [Bug 97494] Changed - Book chapter 8 errors

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=97494

Changed by murrayc usa net 

--- shadow/97494	Sat Nov  2 12:27:28 2002
+++ shadow/97494.tmp.4110	Sat Nov  2 12:44:57 2002
@@ -30,6 +30,9 @@
 and in Model Columns 
 
 the object is called refListStore
 
 then in adding rows it is called m_refListStore which is seems what it
 should be.
+
+------- Additional Comments From murrayc usa net  2002-11-02 12:44 -------
+Seems like you could provide a patch.



--__--__--

Message: 3
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net, daniel elstner gmx net
Cc: 
Date: Sat,  2 Nov 2002 13:05:12 -0500 (EST)
Subject: [gtkmm bugzilla] [Bug 97498] New - Gtk::WindowGroup::create() is missing

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=97498

Changed by daniel elstner gmx net 

--- shadow/97498	Sat Nov  2 13:05:12 2002
+++ shadow/97498.tmp.7967	Sat Nov  2 13:05:12 2002
@@ -0,0 +1,52 @@
+Bug#: 97498
+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: daniel elstner gmx net               
+TargetMilestone: ---
+URL: 
+Summary: Gtk::WindowGroup::create() is missing
+
+Gtk::WindowGroup is a Glib::Object derived class and should therefore have
+a static create() method that returns a Glib::RefPtr<>.  Right now,
+create() is missing and the default ctor is public instead.  This is
+dangerous; non-Gtk::Object classes should always be instantiated in free
+storage since they use pure reference counting for memory management.
+
+Attached is a patch that adds create() and makes the default ctor
+protected.  Neither change breaks ABI:  create() is only an addition, and
+the linker doesn't care about public/protected/private issues.  (I'm
+absolutely sure about that -- I made diffs of the nm output, and gtkmm
+itself used to redeclare methods in wrap_init.cc a while ago.)
+
+What remains are policy issues.  I'll try to outline the pros and cons:
+
+- con: The general policy used by GTK+ as well is that no API additions are
+allowed in a stable branch.  This is done to avoid binary dependencies on a
+specific micro version -- people might not realize they're using a function
+that didn't exist in prior micro releases.
+
+- pro: It's argueable that adding create() should be considered as API fix
+rather than API addition.  Obvious API fixes are allowed even during a
+freeze (e.g. I fixed a GLib function to use gssize rather than gsize after
+the official API freeze in the 1.3 series).  In our case ABI compatibility
+is also required, but that's no problem.
+
+- con: Although unlikely, people might already use Gtk::WindowGroup.  While
+compiled executables will continue to work, the source won't compile
+anymore due to the Gtk::WindowGroup ctor being protected.
+
+- pro: Instantiating Gtk::WindowGroup on stack is broken anyway.  I haven't
+tested it, but if it should indeed not segfault it'll at least cause a
+memory leak.
+
+Opinions?
+
+--Daniel



--__--__--

Message: 4
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net, daniel elstner gmx net
Cc: 
Date: Sat,  2 Nov 2002 13:06:25 -0500 (EST)
Subject: [gtkmm bugzilla] [Bug 97498] Changed - Gtk::WindowGroup::create() is missing

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=97498

Changed by daniel elstner gmx net 

--- shadow/97498	Sat Nov  2 13:05:12 2002
+++ shadow/97498.tmp.8131	Sat Nov  2 13:06:25 2002
@@ -47,6 +47,11 @@
 tested it, but if it should indeed not segfault it'll at least cause a
 memory leak.
 
 Opinions?
 
 --Daniel
+
+------- Additional Comments From daniel elstner gmx net  2002-11-02 13:06 -------
+Created an attachment (id=11989)
+the patch
+



--__--__--

Message: 5
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Sat,  2 Nov 2002 13:07:43 -0500 (EST)
Subject: [gtkmm bugzilla] [Bug 97494] Changed - Book chapter 8 errors

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=97494

Changed by olle endforward com 

--- shadow/97494	Sat Nov  2 12:44:57 2002
+++ shadow/97494.tmp.8351	Sat Nov  2 13:07:43 2002
@@ -33,6 +33,11 @@
 
 then in adding rows it is called m_refListStore which is seems what it
 should be.
 
 ------- Additional Comments From murrayc usa net  2002-11-02 12:44 -------
 Seems like you could provide a patch.
+
+------- Additional Comments From olle endforward com  2002-11-02 13:07 -------
+Created an attachment (id=11990)
+tutorial chapter 8 fix
+




--__--__--

_______________________________________________
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]