GNOME.org
 

Gtkmm-forge Digest, Vol 21, Issue 3

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 512717] ValueBase classes "enhancements"
      (glibmm (bugzilla.gnome.org))
   2. [Bug 512717] ValueBase classes "enhancements"
      (glibmm (bugzilla.gnome.org))
   3. [Bug 515133] New: glibmm i18n-lib.h is not	delivered
      (glibmm (bugzilla.gnome.org))
   4. [Bug 515133] glibmm i18n-lib.h is not delivered
      (glibmm (bugzilla.gnome.org))
   5. [Bug 515135] New: glibmm needs the option not to	abort
      programs with different encodings (glibmm (bugzilla.gnome.org))
   6. [Bug 515135] glibmm needs the option not to abort	programs
      with different encodings (glibmm (bugzilla.gnome.org))
   7. [Bug 515373] New: Patch to add Gst::Index
      (gnomemm (bugzilla.gnome.org))


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

Message: 1
Date: Wed,  6 Feb 2008 00:49:46 +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: <20080206004946 C8E6D23F517 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




------- Comment #4 from Jos? Alburquerque  2008-02-06 00:49 UTC -------
(In reply to comment #3)

When I submitted this report, I was under the impression that the
Gst::Structure getter methods should return a ValueBase and not a boolean. 
After asking and considering I understand now that the getters should still
return booleans and return the result (if any) in ValueBase reference
parameters.  The boolean return allows for valid result testing so, as you say,
the need for a boolean operator is not that needed.

Furthermore, the Gst::Structure::get_field() method is complemented by the
has_field() method which also provides for field existance testing thus
providing a way to check for fields also without needing a ValueBase bool
operator.  So, again as you say, the operator may not be necessary;  However,
I'll try to answer your observations below. 

> 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'm questioning this patch's usefulness, but if it were to be used, the purpose
of the code above is to allow setting (or assigning to) a Value without
necessarily having to "init()" it first.  The code above (included in most
"set()" methods) initializes the Value before setting it if the value has not
been initialized already (G_VALUE_TYPE(...) == 0) (This is what I refer to as
"semi-automatic" initialization).

The new patch I'm attaching removes warnings if a Value is declared, but never
initialized or set by checking for this condition in the Value's destructor.

> 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.

Initially, I tried using the regular operator bool(), but this creates
arithmetic ambiguities with the stream insertion operator (<<) (making it
difficult for casting operators to be recognized) so I thought of using the
pointer operator (operator*) which can be used in boolean contexts but doesn't
create arithmetic ambiguities with the insertion operator. Passing a pointer to
the Value is what's not safe, however, because the value can be deleted this
way and this is how the bool_type weirdness came about.  I found a
"dissertation" on this and used the suggestions at
http://www.artima.com/cppsource/safebool.html.

> - 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.

I will comment in the new patch.

> 
> 
> Then let's look at what's left.
> 

I'm submitting the patch to follow through (it may or may not be useful and I'd
rather let you or someone who has a bit more experience than I do determine
that).  Please take your time deciding how useful (or not) this patch might be
(or not) and feel free to modify it.  Be it as it may, I'm perfectly happy
because as I already said that you've said, a bool operator may not be that
necessary.  Thanks.


-- 
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: 2
Date: Wed,  6 Feb 2008 00:52:04 +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: <20080206005204 8FB5D23F52F 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




------- Comment #5 from Jos? Alburquerque  2008-02-06 00:52 UTC -------
Created an attachment (id=104530)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=104530&action=view)
Patch with comments where necessary


-- 
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: 3
Date: Fri,  8 Feb 2008 03:02:35 +0000 (UTC)
From: "glibmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 515133] New: glibmm i18n-lib.h is not
	delivered
To: gtkmm-forge lists sourceforge net
Message-ID: <bug-515133-5595 http bugzilla 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=515133

  glibmm | general | Ver: unspecified
           Summary: glibmm i18n-lib.h is not delivered
           Product: glibmm
           Version: unspecified
          Platform: Other
        OS/Version: opensolaris
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: general
        AssignedTo: gtkmm-forge lists sourceforge net
        ReportedBy: Takao Fujiwara Sun COM
         QAContact: gtkmm-forge lists sourceforge net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


glibmm i18n-lib.h is not delivered because it's missied in Makefile.am

I'm attaching the 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=515133.



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

Message: 4
Date: Fri,  8 Feb 2008 03:03:43 +0000 (UTC)
From: "glibmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 515133] glibmm i18n-lib.h is not
	delivered
To: gtkmm-forge lists sourceforge net
Message-ID: <20080208030343 4397D23F59D 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=515133

  glibmm | general | Ver: unspecified




------- Comment #1 from Takao Fujiwara  2008-02-08 03:03 UTC -------
Created an attachment (id=104680)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=104680&action=view)
Patch for glib/glibmm/Makefile.am, glib/glibmm/i18n-lib.h

Attached the 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=515133.



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

Message: 5
Date: Fri,  8 Feb 2008 03:15:32 +0000 (UTC)
From: "glibmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 515135] New: glibmm needs the option
	not to	abort programs with different encodings
To: gtkmm-forge lists sourceforge net
Message-ID: <bug-515135-5595 http bugzilla 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=515135

  glibmm | strings | Ver: unspecified
           Summary: glibmm needs the option not to abort programs with
                    different encodings
           Product: glibmm
           Version: unspecified
          Platform: Other
        OS/Version: opensolaris
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: strings
        AssignedTo: gtkmm-forge lists sourceforge net
        ReportedBy: Takao Fujiwara Sun COM
         QAContact: gtkmm-forge lists sourceforge net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


Currently if Glib::locale_from_utf8() receives different encodings, it aborts
in Glib::ConvertError::throw_func.


Exception type class ConvertError has no handle.
stopped in __exdbg_notify_of_throw
Signal ABRT __lwp_kill
0xfcacaa88: __lwp_kill+0x0008:  bcc,a,pt  %icc,__lwp_kill+0x18  ! 0xfcacaa98

(dbx) where -h
  [1] __exdbg_notify_of_throw(0xffbff818, 0xffbff800, 0x1c1c, 0x1c00, 0x160f0,
0xfcd6e004): 0xfcd55a68 
  [2] _ex_debug_handshake1(0xfcd6e140, 0x0, 0x1, 0x16f68, 0xfcd6d758,
0xfcd6df28): 0xfcd56890 
  [3] _ex_throw_body(0xfcd6e140, 0x0, 0xffbffc58, 0x16ccc, 0x91918,
0xfcd6e140): 0xfcd56b0c 
  [4] __Crun::ex_throw(0xfcd6e140, 0x1, 0xff1c3820, 0x0, 0xfcd6d758,
0xfcd6df6c): 0xfcd56a68 
=>[5] Glib::ConvertError::throw_func(gobject = 0x46760) "convert.cc"
  [6] Glib::Error::throw_exception(gobject = 0x46760) "error.cc"
  [7] Glib::locale_from_utf8(utf8_string = CLASS) "convert.cc"
  [8] main(0x1, 0xffbffcbc, 0x22af0, 0x22800, 0xfadd0, 0xffbffc34): 0x11f98 

When we use g_locale_from_utf8(), the error handling is owned by each
application. I'ld need the option to avoid that applications always abort with
the different encodings.
I think we have the case that the actual users' directories have different
encoded file contents or file names against the current locale.

I'm attaching the 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=515135.



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

Message: 6
Date: Fri,  8 Feb 2008 03:16:50 +0000 (UTC)
From: "glibmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 515135] glibmm needs the option not to
	abort	programs with different encodings
To: gtkmm-forge lists sourceforge net
Message-ID: <20080208031650 7241423F596 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=515135

  glibmm | strings | Ver: unspecified




------- Comment #1 from Takao Fujiwara  2008-02-08 03:16 UTC -------
Created an attachment (id=104681)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=104681&action=view)
Patch for glib/src/convert.ccg

The idea is to use warnings instead of error aboarts.

Could you review the 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=515135.



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

Message: 7
Date: Sat,  9 Feb 2008 10:23:33 +0000 (UTC)
From: "gnomemm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 515373] New: Patch to add Gst::Index
To: gtkmm-forge lists sourceforge net
Message-ID: <bug-515373-5595 http bugzilla 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=515373

  gnomemm | gstreamermm | Ver: unspecified
           Summary: Patch to add Gst::Index
           Product: gnomemm
           Version: unspecified
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gstreamermm
        AssignedTo: gtkmm-forge lists sourceforge net
        ReportedBy: siavash safi gmail com
         QAContact: gtkmm-forge lists sourceforge net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


The patch adds Gst::Index


-- 
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=515373.



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

-------------------------------------------------------------------------
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 3
******************************************
[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]