GNOME.org
 

[gtkmm] Gtkmm-forge digest, Vol 1 #253 - 8 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 90689] Changed - Patch to check for existence of 'dot' (bugzilla-daemon widget gnome org)
   2. [Bug 95606] New - Warning and crash when removing and readding widgets. (bugzilla-daemon widget gnome org)
   3. [Bug 95606] Changed - Warning and crash when removing and readding widgets. (bugzilla-daemon widget gnome org)
   4. [Bug 90689] Changed - Patch to check for existence of 'dot' (bugzilla-daemon widget gnome org)
   5. [Bug 95606] Changed - Warning and crash when removing and readding widgets. (bugzilla-daemon widget gnome org)
   6. [Bug 94658] Changed - TextBuffer insert method using RefPtr<TextBuffer> yields core dump on G_IS_OBJECT assertion (bugzilla-daemon widget gnome org)
   7. [Bug 95606] Changed - Warning and crash when removing and readding widgets. (bugzilla-daemon widget gnome org)
   8. [Bug 93787] Changed - Outputting ustring with operator << converts implicitly (bugzilla-daemon widget gnome org)

--__--__--

Message: 1
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Fri, 11 Oct 2002 15:33:03 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 90689] Changed - Patch to check for existence of 'dot'

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

Changed by btb debian org 

--- shadow/90689	Fri Oct 11 15:04:54 2002
+++ shadow/90689.tmp.31511	Fri Oct 11 15:33:03 2002
@@ -98,6 +98,11 @@
 http://www.chaosreigns.com/code/springgraph/
 
 Here is what the ref. docs look like when springgraph is used:
 http://pokey.css.washington.edu/debian/gtkmm/gtkmm1.3/gtkmm1.3-1.3.24/docs/reference/html/
 
 -brad
+
+------- Additional Comments From btb debian org  2002-10-11 15:33 -------
+I seem to have been mistaken about springgraph.  I can't get it to
+work at all, now.
+



--__--__--

Message: 2
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Sat, 12 Oct 2002 08:26:37 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 95606] New - Warning and crash when removing and readding widgets.

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

Changed by matty chariot net au 

--- shadow/95606	Sat Oct 12 08:26:37 2002
+++ shadow/95606.tmp.24469	Sat Oct 12 08:26:37 2002
@@ -0,0 +1,69 @@
+Bug#: 95606
+Product: gtkmm
+Version: 2.0
+OS: Linux
+OS Details: Debian Testing
+Status: NEW   
+Resolution: 
+Severity: critical
+Priority: Normal
+Component: general
+AssignedTo: gtkmm-forge lists sourceforge net                            
+ReportedBy: matty chariot net au               
+TargetMilestone: ---
+URL: 
+Summary: Warning and crash when removing and readding widgets.
+
+I get the following warning on my code:
+
+Gtk-CRITICAL **: file gtkcontainer.c: line 878 (gtk_container_remove):
+assertion `GTK_IS_CONTAINER (container)' failed
+
+after this occurs a number of times a crash occurs.
+
+The following is the smallest test case I can make to demonstrate the problem:
+
+
+#include <gtkmm.h>
+
+class TestWindow : public Gtk::Window {
+public:
+
+  Gtk::VBox box;
+  Gtk::HSeparator *sep;
+  Gtk::Frame *frame;
+  Gtk::Button button;
+    
+  TestWindow () {
+      button.set_label( "Reset" );
+      button.signal_clicked().connect( SigC::slot( *this,
+&TestWindow::on_click ) );
+      box.pack_start( button );
+      add( box );
+      new_widget();
+  }
+  
+  void new_widget() {
+      sep = new Gtk::HSeparator();
+      frame = new Gtk::Frame();
+      frame->add( *sep );
+      box.pack_start( *frame );
+      show_all();
+  }
+  
+  void on_click() {
+      box.remove( *frame );
+      delete frame;
+      delete sep;
+      new_widget();
+  }
+
+};
+
+int main( int argc, char *argv[] ) {
+
+  Gtk::Main main_app( argc, argv );
+  main_app.run( *( new TestWindow() ) );
+  return 0;
+
+}



--__--__--

Message: 3
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Sat, 12 Oct 2002 08:55:19 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 95606] Changed - Warning and crash when removing and readding widgets.

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

Changed by matty chariot net au 

--- shadow/95606	Sat Oct 12 08:26:37 2002
+++ shadow/95606.tmp.29317	Sat Oct 12 08:55:19 2002
@@ -64,6 +64,9 @@
 
   Gtk::Main main_app( argc, argv );
   main_app.run( *( new TestWindow() ) );
   return 0;
 
 }
+
+------- Additional Comments From matty chariot net au  2002-10-12 08:55 -------
+Compiler is g++-3.2, this is with a CVS pull from September 20.



--__--__--

Message: 4
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net, btb debian org
Cc: 
Date: Sat, 12 Oct 2002 09:37:24 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 90689] Changed - Patch to check for existence of 'dot'

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

Changed by murrayc usa net 

--- shadow/90689	Fri Oct 11 15:33:03 2002
+++ shadow/90689.tmp.4379	Sat Oct 12 09:37:24 2002
@@ -103,6 +103,10 @@
 -brad
 
 ------- Additional Comments From btb debian org  2002-10-11 15:33 -------
 I seem to have been mistaken about springgraph.  I can't get it to
 work at all, now.
 
+
+------- Additional Comments From murrayc usa net  2002-10-12 09:37 -------
+Look at the ListStore page, for instance. Only dot properly combines
+virtual base classes.



--__--__--

Message: 5
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Sat, 12 Oct 2002 09:40:28 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 95606] Changed - Warning and crash when removing and readding widgets.

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

Changed by murrayc usa net 

--- shadow/95606	Sat Oct 12 08:55:19 2002
+++ shadow/95606.tmp.5062	Sat Oct 12 09:40:28 2002
@@ -2,13 +2,13 @@
 Product: gtkmm
 Version: 2.0
 OS: Linux
 OS Details: Debian Testing
 Status: NEW   
 Resolution: 
-Severity: critical
+Severity: normal
 Priority: Normal
 Component: general
 AssignedTo: gtkmm-forge lists sourceforge net                            
 ReportedBy: matty chariot net au               
 TargetMilestone: ---
 URL: 
@@ -67,6 +67,9 @@
   return 0;
 
 }
 
 ------- Additional Comments From matty chariot net au  2002-10-12 08:55 -------
 Compiler is g++-3.2, this is with a CVS pull from September 20.
+
+------- Additional Comments From murrayc usa net  2002-10-12 09:40 -------
+Surely you can simplify this more. Maybe just removing one widget?



--__--__--

Message: 6
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net, fshim cawthra com
Cc: 
Date: Sat, 12 Oct 2002 09:50:15 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 94658] Changed - TextBuffer insert method using RefPtr<TextBuffer> yields core dump on G_IS_OBJECT assertion

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

Changed by murrayc usa net 

--- shadow/94658	Thu Oct  3 02:06:55 2002
+++ shadow/94658.tmp.6779	Sat Oct 12 09:50:15 2002
@@ -1,13 +1,13 @@
 Bug#: 94658
 Product: gtkmm
 Version: 2.0
 OS: Linux
 OS Details: Linux i386 - Distro: RedHat 7.3 with gtkmm-2.0 additions
-Status: NEW   
-Resolution: 
+Status: RESOLVED   
+Resolution: INVALID
 Severity: normal
 Priority: Normal
 Component: general
 AssignedTo: gtkmm-forge lists sourceforge net                            
 ReportedBy: fshim cawthra com               
 TargetMilestone: ---
@@ -336,6 +336,11 @@
 TextBuffer class referencing/deferencing.
 
 Let me know what you think.
 
 Regards,
 Frank Shim
+
+------- Additional Comments From murrayc usa net  2002-10-12 09:50 -------
+I tried the C++ test code without problems. I seem to remember that we
+fixed something like this recently so maybe you should try a newer
+version. Please reopen the bug if it's still a problem. 



--__--__--

Message: 7
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Sat, 12 Oct 2002 09:51:53 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 95606] Changed - Warning and crash when removing and readding widgets.

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

Changed by matty chariot net au 

--- shadow/95606	Sat Oct 12 09:40:28 2002
+++ shadow/95606.tmp.6995	Sat Oct 12 09:51:53 2002
@@ -70,6 +70,18 @@
 
 ------- Additional Comments From matty chariot net au  2002-10-12 08:55 -------
 Compiler is g++-3.2, this is with a CVS pull from September 20.
 
 ------- Additional Comments From murrayc usa net  2002-10-12 09:40 -------
 Surely you can simplify this more. Maybe just removing one widget?
+
+------- Additional Comments From matty chariot net au  2002-10-12 09:51 -------
+I tried to simplify this further but was unable to.  It took me about
+8 hours to reduce my program to this test case, and this is indicative
+of how fragile and specific the error situation is.
+
+You could probably remove button and box, these are primarily used to
+perform the removal action.  However, you'd need some alternate means
+of triggering the removal action, like on a timer or close event.
+
+danielk suggested attempting to remove sep from frame before deletion,
+and this seems to work OK as a workaround.



--__--__--

Message: 8
From: bugzilla-daemon widget gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Sat, 12 Oct 2002 11:20:08 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 93787] Changed - Outputting ustring with operator << converts implicitly

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

Changed by olau hardworking dk 

--- shadow/93787	Fri Oct 11 14:17:26 2002
+++ shadow/93787.tmp.22979	Sat Oct 12 11:20:08 2002
@@ -250,6 +250,43 @@
 
 --Daniel
 
 
 ------- Additional Comments From murrayc usa net  2002-10-11 14:17 -------
 Where should we mention it exactly?
+
+------- Additional Comments From olau hardworking dk  2002-10-12 11:20 -------
+I'm not feeling quite convinced yet. :-)
+
+As I see it, there is no solution that will solve all possible needs;
+the question is really more that of what situations require explicit
+character conversions. As it is now, it's possible to put UTF-8
+strings in std::strings without any trouble, but using streams
+requires explicit conversions. Isn't it much more useful the other way
+round? Why wouldn't you just use a Glib::ustring if you wanted to
+store UTF-8 strings?
+
+It also seems much more likely that if you really need a std::string,
+it needs to be in the locale's encoding. It's usually announced
+heavily if a library wants or outputs UTF-8 so it's not too difficult
+to remember explicit measures in that case. It's not so the other way
+round. With the current approach, boost::lexical_cast results in
+corrupted strings without anyone noticing.
+
+Thing is, you can't convert much to a string without going through a
+stringstream. So in effect the standard already does enforce a
+particular encoding on strings, at least if you want to avoid the mess
+of having std::strings with different encodings in the same program.
+Besides, we wouldn't really be enforcing anything for all
+std::strings, we would just change the assumed encoding of
+std::strings used with Gtkmm from UTF-8 to the locale's encoding.
+Extra functions can handle the case where this assumption is wrong.
+
+The cost of the conversions is not important here, I think. Unless
+you're sloppy with your string types, the conversions are actually
+needed, whether implicit or not.
+
+
+If I've not managed to convince you, I think it would be better to
+remove the implicit std::string conversion completely and introduce
+something like string_from_utf8 and string_from_locale instead. This
+would at least force one to think about the issue.




--__--__--

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