Send Gtkmm-forge mailing list submissions to
        [EMAIL PROTECTED]

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
        [EMAIL PROTECTED]

You can reach the person managing the list at
        [EMAIL PROTECTED]

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)"
        <[EMAIL PROTECTED]>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: [EMAIL PROTECTED]
Message-ID: <[EMAIL PROTECTED]>
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)"
        <[EMAIL PROTECTED]>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: [EMAIL PROTECTED]
Message-ID: <[EMAIL PROTECTED]>
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|                            |[EMAIL PROTECTED]




------- 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)"
        <[EMAIL PROTECTED]>
Subject: [gtkmm bugzilla] [Bug 511972] glibmm doesn't have an
        AsyncQueue      class
To: [EMAIL PROTECTED]
Message-ID: <[EMAIL PROTECTED]>
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)"
        <[EMAIL PROTECTED]>
Subject: [gtkmm bugzilla] [Bug 512717] ValueBase classes
        "enhancements"
To: [EMAIL PROTECTED]
Message-ID: <[EMAIL PROTECTED]>
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|                            |[EMAIL PROTECTED]




------- 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)"
        <[EMAIL PROTECTED]>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: [EMAIL PROTECTED]
Message-ID: <[EMAIL PROTECTED]>
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)"
        <[EMAIL PROTECTED]>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: [EMAIL PROTECTED]
Message-ID: <[EMAIL PROTECTED]>
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)"
        <[EMAIL PROTECTED]>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: [EMAIL PROTECTED]
Message-ID: <[EMAIL PROTECTED]>
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)"
        <[EMAIL PROTECTED]>
Subject: [gtkmm bugzilla] [Bug 514241] patch to add Gst::Buffer
To: [EMAIL PROTECTED]
Message-ID: <[EMAIL PROTECTED]>
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
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/gtkmm-forge


End of Gtkmm-forge Digest, Vol 21, Issue 2
******************************************
_______________________________________________
gtkmm-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/gtkmm-list

Reply via email to