GNOME.org
 

Gtkmm-forge Digest, Vol 21, Issue 2

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 514241] patch to add Gst::Buffer
      (gnomemm (bugzilla.gnome.org))
   2. [Bug 514241] patch to add Gst::Buffer
      (gnomemm (bugzilla.gnome.org))
   3. [Bug 511972] glibmm doesn't have an AsyncQueue	class
      (glibmm (bugzilla.gnome.org))
   4. [Bug 512717] ValueBase classes "enhancements"
      (glibmm (bugzilla.gnome.org))
   5. [Bug 514241] patch to add Gst::Buffer
      (gnomemm (bugzilla.gnome.org))
   6. [Bug 514241] patch to add Gst::Buffer
      (gnomemm (bugzilla.gnome.org))
   7. [Bug 514241] patch to add Gst::Buffer
      (gnomemm (bugzilla.gnome.org))
   8. [Bug 514241] patch to add Gst::Buffer
      (gnomemm (bugzilla.gnome.org))


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

Message: 1
Date: Mon,  4 Feb 2008 09:38:10 +0000 (UTC)
From: "gnomemm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: gtkmm-forge lists sourceforge net
Message-ID: <20080204093810 4986423F519 label gnome org>
Content-Type: text/plain; charset=utf-8

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=514241

  gnomemm | gstreamermm | Ver: unspecified




------- Comment #1 from Siavash Safi  2008-02-04 09:38 UTC -------
Created an attachment (id=104368)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=104368&action=view)
Gst::Buffer patch


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=514241.



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

Message: 2
Date: Mon,  4 Feb 2008 11:22:45 +0000 (UTC)
From: "gnomemm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: gtkmm-forge lists sourceforge net
Message-ID: <20080204112245 AC54A23F522 label gnome org>
Content-Type: text/plain; charset=utf-8

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=514241

  gnomemm | gstreamermm | Ver: unspecified

Murray Cumming changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |murrayc murrayc com




------- Comment #2 from Murray Cumming  2008-02-04 11:22 UTC -------
Thanks. I don't think it's quite ready:

- Several enums are hand-written. Why can't _WRAP_ENUM() be used for these. In
general, please add explanatory comments in the file when doing unusual things
such as this.
- void Buffer::copy_metadata(Glib::RefPtr<Buffer> other_buffer, BufferCopyFlags
flags):
  This should take a const RefPtr<>&. Why isn't _WRAP_METHOD() used for this?
- Several methods should be const and/or have const method overloads.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=514241.



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

Message: 3
Date: Mon,  4 Feb 2008 11:28:27 +0000 (UTC)
From: "glibmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 511972] glibmm doesn't have an
	AsyncQueue	class
To: gtkmm-forge lists sourceforge net
Message-ID: <20080204112827 AFD2C23F520 label gnome org>
Content-Type: text/plain; charset=utf-8

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=511972

  glibmm | threads | Ver: 2.15.x




------- Comment #12 from Murray Cumming  2008-02-04 11:28 UTC -------
Thanks. That's a big help (though the indenting and doxygen comment style is
still not the same as the rest of glibmm).

I would still like to see discussion of this on gtkmm-list, because this is not
something that I currently need to use.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=511972.



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

Message: 4
Date: Mon,  4 Feb 2008 11:38:47 +0000 (UTC)
From: "glibmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 512717] ValueBase classes
	"enhancements"
To: gtkmm-forge lists sourceforge net
Message-ID: <20080204113847 1789D23F522 label gnome org>
Content-Type: text/plain; charset=utf-8

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=512717

  glibmm | general | Ver: 2.15.x

Murray Cumming changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |murrayc murrayc com




------- Comment #3 from Murray Cumming  2008-02-04 11:38 UTC -------
> The semi-automatic mechanism installed in the ValueBase classes does not affect
> code that already use the ValueBase class.  It is only triggered when setting
> ValueBase classes that have not been initialized.  If one is declared but 
> never initialized or set, warnings are generated as usual and initialization before
> usage works just as it has before, so previous code never even notices the
difference.

This is this code:

// Make sure class is initialized before setting
+  if (G_VALUE_TYPE(&gobject_) == 0)
+    init(value_type());

I don't think that would cause a warning, but that's OK - I wouldn't want it
to. I would like this change to go in, separate from the other changes. But I
suspect that the check should be in intit() instead of before using init().

I would like a  ValueBase::operator bool too, though it's not very useful now
that it's unnecessary. However:
- I don't understand why you've used bool_type instead of bool there.
- I don't understand what's "safe" about it.
- I'd rather not have things like this without any explanatory comments:
+ typedef void (ValueBase::*bool_type)() const;
+  void void_function() const {}

Maybe you could submit a new patch for that.


Then let's look at what's left.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=512717.



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

Message: 5
Date: Mon,  4 Feb 2008 12:08:16 +0000 (UTC)
From: "gnomemm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: gtkmm-forge lists sourceforge net
Message-ID: <20080204120816 C4E3A23F521 label gnome org>
Content-Type: text/plain; charset=utf-8

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=514241

  gnomemm | gstreamermm | Ver: unspecified




------- Comment #3 from Siavash Safi  2008-02-04 12:08 UTC -------
- _WRAP_ENUM() fails on those enums

- Buffer::copy_metadata(), The C API has silly argument order:
http://gstreamer.freedesktop.org/data/doc/gstreamer/head/gstreamer/html/gstreamer-GstBuffer.html#gst-buffer-copy-metadata

- I'll check all methods :)

I'll add comments and attach a new patch


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=514241.



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

Message: 6
Date: Mon,  4 Feb 2008 12:31:30 +0000 (UTC)
From: "gnomemm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: gtkmm-forge lists sourceforge net
Message-ID: <20080204123130 2C2BE23F521 label gnome org>
Content-Type: text/plain; charset=utf-8

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=514241

  gnomemm | gstreamermm | Ver: unspecified




------- Comment #4 from Murray Cumming  2008-02-04 12:31 UTC -------
> - _WRAP_ENUM() fails on those enums

Please don't make me guess how it fails.

> Buffer::copy_metadata(), The C API has silly argument order

Oh, yes. That should be filed as a bug I think - other language bindings will
also assume that the first parameter is a self/this.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=514241.



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

Message: 7
Date: Tue,  5 Feb 2008 20:41:10 +0000 (UTC)
From: "gnomemm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: gtkmm-forge lists sourceforge net
Message-ID: <20080205204110 9DCB723F53B label gnome org>
Content-Type: text/plain; charset=utf-8

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=514241

  gnomemm | gstreamermm | Ver: unspecified

Siavash Safi changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #104368|0                           |1
        is obsolete|                            |




------- Comment #5 from Siavash Safi  2008-02-05 20:41 UTC -------
Created an attachment (id=104504)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=104504&action=view)
Gst::Buffer patch


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=514241.



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

Message: 8
Date: Tue,  5 Feb 2008 21:15:35 +0000 (UTC)
From: "gnomemm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: gtkmm-forge lists sourceforge net
Message-ID: <20080205211535 A674123F522 label gnome org>
Content-Type: text/plain; charset=utf-8

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=514241

  gnomemm | gstreamermm | Ver: unspecified

Murray Cumming changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED




------- Comment #6 from Murray Cumming  2008-02-05 21:15 UTC -------
Thanks. I will commit this and make the corrections afterwards.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=514241.



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

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

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

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


End of Gtkmm-forge Digest, Vol 21, Issue 2
******************************************
[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]