Gtkmm-forge Digest, Vol 6, Issue 22



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-owner 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. Do not try to unsubscribe gtkmm-forge from gtkmm-list.


Today's Topics:

   1. [Bug 351217] SlotTransfer used in	Gnome::Vfs::Transfer should
      return (g)int (gnomemm (bugzilla.gnome.org))
   2. [Bug 362181] Cairo::RefPtr leaking memory
      (gtkmm (bugzilla.gnome.org))
   3. [Bug 362181] Cairo::RefPtr leaking memory
      (gtkmm (bugzilla.gnome.org))
   4. [Bug 362181] Cairo::RefPtr leaking memory
      (gtkmm (bugzilla.gnome.org))
   5. [Bug 362181] Cairo::RefPtr leaking memory
      (gtkmm (bugzilla.gnome.org))
   6. [Bug 371487]	Gnome::Vfs::DirectoryHandle::list_load has wrong
      signature (gnomemm (bugzilla.gnome.org))
   7. [Bug 362181] Cairo::RefPtr leaking memory
      (gtkmm (bugzilla.gnome.org))
   8. [Bug 362181] Cairo::RefPtr leaking memory
      (gtkmm (bugzilla.gnome.org))


----------------------------------------------------------------------

Message: 1
Date: Mon, 27 Nov 2006 21:18:46 +0000 (UTC)
From: "gnomemm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 351217] SlotTransfer used in
	Gnome::Vfs::Transfer should return (g)int
To: gtkmm-forge lists sourceforge net
Message-ID: <20061127211846 10EA86C4162 box gnome org>
Content-Type: text/plain; charset=utf-8

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=351217

  gnomemm | gnome-vfsmm | Ver: unspecified





------- Comment #5 from Murray Cumming  2006-11-27 21:17 UTC -------
> I'm not completely sure if it's ok to typedef Transfer2::ProgressInfo as
> Transfer::ProgressInfo, as there are no changes necessary over there. But to me
> it seems better then copying and pasting the same code.

That would still leave people to use the other stuff in the Transfer namespace
that's in transfer-progress.h. So they'd be using Transfer2 sometimes and
Transfer at other times.

I'd prefer to totally deprecate the Transfer namespace. That would mean we must
create a transfer-progress-2.[h|cc] file.

> mostly the same code being duplicated

You might want to reimplement Transfer stuff in terms of Transfer2 to, to
reduce copy/pasting.

And we can lose the existing Transfer example code. It should just use the
non-deprecated Transfer2 API.

"Transfer2" isn't an ideal name, but I can't think of a better name. 

> I haven't added the implementation of Transfer::remove(StringArrayHandle&...).
> ABI break or not?

Sorry, I don't know what you mean. If you can produce a complete patch, I will,
of course, check it for ABI breaks.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email



------------------------------

Message: 2
Date: Mon, 27 Nov 2006 21:35:34 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 362181] Cairo::RefPtr leaking memory
To: gtkmm-forge lists sourceforge net
Message-ID: <20061127213534 0F09A6C4163 box gnome org>
Content-Type: text/plain; charset=utf-8

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=362181

  gtkmm | general | Ver: 2.10.x


Murray Cumming changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #77092|application/octet-stream    |text/plain
          mime type|                            |




-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email



------------------------------

Message: 3
Date: Mon, 27 Nov 2006 21:37:54 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 362181] Cairo::RefPtr leaking memory
To: gtkmm-forge lists sourceforge net
Message-ID: <20061127213754 B18FF6C4162 box gnome org>
Content-Type: text/plain; charset=utf-8

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=362181

  gtkmm | general | Ver: 2.10.x





------- Comment #7 from Murray Cumming  2006-11-27 21:36 UTC -------
Attempting to paste my valgrind --leak-check=full output:

0== 23,040 (1,440 direct, 21,600 indirect) bytes in 30 blocks are definitely
lost in loss record 108 of 113
==29050==    at 0x4021396: malloc (vg_replace_malloc.c:149)
==29050==    by 0x4898569: cairo_create (cairo.c:188)
==29050==    by 0x47ADF2B: gdk_cairo_create (gdkcairo.c:46)
==29050==    by 0x434C16D: Gdk::Drawable::create_cairo_context()
(drawable.cc:370)
==29050==    by 0x804CA5B: CairoTest::on_expose_event(_GdkEventExpose*) (in
/home/murrayc/a.out)
==29050==    by 0x4248ED2:
Gtk::Widget_Class::expose_event_callback(_GtkWidget*, _GdkEventExpose*)
(widget.cc:4401)
==29050==    by 0x44E9557: _gtk_marshal_BOOLEAN__BOXED (gtkmarshalers.c:83)
==29050==    by 0x48F88CE: g_type_class_meta_marshal (gclosure.c:567)
==29050==    by 0x48F8EFE: g_closure_invoke (gclosure.c:490)
==29050==    by 0x490972D: signal_emit_unlocked_R (gsignal.c:2476)
==29050==    by 0x490A853: g_signal_emit_valist (gsignal.c:2207)
==29050==    by 0x490AE48: g_signal_emit (gsignal.c:2241)
==29050==    by 0x460CCD8: gtk_widget_event_internal (gtkwidget.c:3911)
==29050==    by 0x44E7CB2: gtk_main_do_event (gtkmain.c:1380)
==29050==    by 0x47C82E7: gdk_window_process_updates_internal
(gdkwindow.c:2324)
==29050==    by 0x47C83BA: gdk_window_process_all_updates (gdkwindow.c:2387)
==29050==    by 0x47C843B: gdk_window_update_idle (gdkwindow.c:2245)
==29050==    by 0x49771E6: g_idle_dispatch (gmain.c:4073)
==29050==    by 0x4974CAC: g_main_context_dispatch (gmain.c:2049)
==29050==    by 0x4977F85: g_main_context_iterate (gmain.c:2681)


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email



------------------------------

Message: 4
Date: Mon, 27 Nov 2006 21:39:08 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 362181] Cairo::RefPtr leaking memory
To: gtkmm-forge lists sourceforge net
Message-ID: <20061127213908 23C656C417B box gnome org>
Content-Type: text/plain; charset=utf-8

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=362181

  gtkmm | general | Ver: 2.10.x





------- Comment #8 from Murray Cumming  2006-11-27 21:37 UTC -------
So, possibly we are deleting the C++ instance but not unrefing the C instance.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email



------------------------------

Message: 5
Date: Mon, 27 Nov 2006 23:10:31 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 362181] Cairo::RefPtr leaking memory
To: gtkmm-forge lists sourceforge net
Message-ID: <20061127231031 0E3346C413F box gnome org>
Content-Type: text/plain; charset=utf-8

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=362181

  gtkmm | general | Ver: 2.10.x





------- Comment #9 from austinf gmail com  2006-11-27 23:09 UTC -------
This has been my workaround that seems to fix it...

in the expose function,

cairo_t* crc = do_cairo_create();
Cairo::Context* cr = new Cairo::Context(crc, true);
draw(cr);
delete cr;


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email



------------------------------

Message: 6
Date: Tue, 28 Nov 2006 10:05:39 +0000 (UTC)
From: "gnomemm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 371487]
	Gnome::Vfs::DirectoryHandle::list_load has wrong signature
To: gtkmm-forge lists sourceforge net
Message-ID: <20061128100539 8D5B06C40C3 box gnome org>
Content-Type: text/plain; charset=utf-8

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=371487

  gnomemm | gnome-vfsmm | Ver: 2.10





------- Comment #6 from Maik Beckmann  2006-11-28 10:04 UTC -------
Will the patch be applied?

Maik


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email



------------------------------

Message: 7
Date: Tue, 28 Nov 2006 18:15:49 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 362181] Cairo::RefPtr leaking memory
To: gtkmm-forge lists sourceforge net
Message-ID: <20061128181549 5A0756C411F box gnome org>
Content-Type: text/plain; charset=utf-8

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=362181

  gtkmm | general | Ver: 2.10.x





------- Comment #10 from Murray Cumming  2006-11-28 18:14 UTC -------
Created an attachment (id=77301)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=77301&action=view)
cairo_refs.patch

I found that and 2 other places where we do excess references, in different
ways. This patch should fix that, though I'd like you to test it a bit before I
commit it. I feel like I've done this before, though maybe it was something
else.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email



------------------------------

Message: 8
Date: Tue, 28 Nov 2006 18:36:48 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 362181] Cairo::RefPtr leaking memory
To: gtkmm-forge lists sourceforge net
Message-ID: <20061128183648 5AE6D6C4115 box gnome org>
Content-Type: text/plain; charset=utf-8

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=362181

  gtkmm | general | Ver: 2.10.x





------- Comment #11 from jonner  2006-11-28 18:35 UTC -------
I haven't looked at this at all yet.  Are you saying that there are no problems
in  Cairo::RefPtr itself, just in gtkmm functions that use cairo them?  


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email



------------------------------

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

------------------------------

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


End of Gtkmm-forge Digest, Vol 6, Issue 22
******************************************



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]