GNOME.org
 

Gtkmm-forge Digest, Vol 11, Issue 12

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 388086] Gtk::RecentManager::Data can't be	instantiated
      (gtkmm (bugzilla.gnome.org))
   2. [Bug 423990] glibmm et al don't recognize new GNU	m4
      (glibmm (bugzilla.gnome.org))
   3. [Bug 388086] Gtk::RecentManager::Data can't be	instantiated
      (gtkmm (bugzilla.gnome.org))
   4. [Bug 388086] Gtk::RecentManager::Data can't be	instantiated
      (gtkmm (bugzilla.gnome.org))
   5. [Bug 388086] Gtk::RecentManager::Data can't be	instantiated
      (gtkmm (bugzilla.gnome.org))
   6. [Bug 388086] Gtk::RecentManager::Data can't be	instantiated
      (gtkmm (bugzilla.gnome.org))
   7. [Bug 388086] Gtk::RecentManager::Data can't be	instantiated
      (gtkmm (bugzilla.gnome.org))


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

Message: 1
Date: Wed, 18 Apr 2007 19:23:05 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 388086] Gtk::RecentManager::Data can't
	be	instantiated
To: gtkmm-forge lists sourceforge net
Message-ID: <20070418192305 E80546C41B0 box 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=388086

  gtkmm | build | Ver: 2.10.x




------- Comment #5 from Grant Patterson  2007-04-18 19:23 UTC -------
Thanks for doing this, Armin!

Indeed, we can change the Data API all we want since it was really
non-instantiable before this patch. I'm not sure the best way to do this, but
I'll point out that RecentManager returns RecentInfo's. Why shouldn't we pass
in what we'll get out? This might make RecentInfo kind of strange, since some
of its metadata depends on it having been added to a RecentManager already. But
yeah, doesn't make much sense to pass in a dummy struct and get out a fancy
object. Also note that GtkRecentData allows some data elements to be undefined;
display_name and description can both be NULL if we don't want to specify them.
Whatever the API winds up being, it should also allow such undefined elements. 


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



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

Message: 2
Date: Fri, 20 Apr 2007 03:56:32 +0000 (UTC)
From: "glibmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 423990] glibmm et al don't recognize
	new GNU	m4
To: gtkmm-forge lists sourceforge net
Message-ID: <20070420035632 ECD906C4279 box 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=423990

  glibmm | build | Ver: unspecified




------- Comment #3 from Cygwin Ports maintainer  2007-04-20 03:56 UTC -------
Created an attachment (id=86674)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=86674&action=view)
scripts/macros.m4 GNU m4 patch

> Thanks. I wonder why I'm not having this problem on Ubuntu Feisty (with m4
> 1.4.8build1):

"not having this problem" means that the result is yes or that the build still
works?  AFAICS the test only does something on Solaris if not GNU m4.

In any case, patch attached.


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



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

Message: 3
Date: Sun, 22 Apr 2007 12:28:01 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 388086] Gtk::RecentManager::Data can't
	be	instantiated
To: gtkmm-forge lists sourceforge net
Message-ID: <20070422122801 D928D6C4155 box 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=388086

  gtkmm | build | Ver: 2.10.x

Armin Burgmeier changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |armin arbur net




------- Comment #6 from Armin Burgmeier  2007-04-22 12:28 UTC -------
I think it is safe to change the groups field in the Data struct, since it
could not have been used anyway.

The mime_type, app_name and app_exec fields are required nevertheless, so they
cannot be empty. The display name could be empty in theory, but that would
require that we pass NULL to the C function which is not possible when using
Glib::ustring. Pretty the same applies to the description field, but I think
NULL is treated the same as the empty string here.

I therefore tried adding two add_item() overloads, one with a StringArrayHandle
for the groups field and one without, but this is rather difficult because the
compiler cannot decide which one to use if it gets a const  char* argument for
the field which can either be a ustring or the StringArrayHandle. It would
probably work when the StringArrayHandle was the last parameter, but I am no
longer fully convinced by this because one has to take pretty much care that
all possible use cases work as expected.

RecentInfo seems to be something different from RecentData because it points to
an existing recent item while RecentData just contains data to create a new
item. You are probably not supposed to create a RecentInfo yourself. There is
also already a difference in GTK+ between those two.


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



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

Message: 4
Date: Sun, 22 Apr 2007 13:37:46 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 388086] Gtk::RecentManager::Data can't
	be	instantiated
To: gtkmm-forge lists sourceforge net
Message-ID: <20070422133746 0B04C6C4158 box 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=388086

  gtkmm | build | Ver: 2.10.x




------- Comment #7 from Murray Cumming  2007-04-22 13:37 UTC -------
>  The display name could be empty in theory, but that would
> require that we pass NULL to the C function which is not possible when using
> Glib::ustring.

We can just do a (str.empty ? str.c_str() : NULL).

> the compiler cannot decide which one to use if it gets a const  char* argument > for the field which can either be a ustring or the StringArrayHandle.

Yes, so we would need to distinguish the methods by using a slightly different
method name.


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



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

Message: 5
Date: Sun, 22 Apr 2007 13:53:32 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 388086] Gtk::RecentManager::Data can't
	be	instantiated
To: gtkmm-forge lists sourceforge net
Message-ID: <20070422135332 203556C41C3 box 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=388086

  gtkmm | build | Ver: 2.10.x




------- Comment #8 from Armin Burgmeier  2007-04-22 13:53 UTC -------
So, what is your opinion on this? Should I just commit that patch? We can still
add more API in the future when someone points out an actual use case for
gtk_recent_manager_add_full().


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



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

Message: 6
Date: Sun, 22 Apr 2007 14:00:21 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 388086] Gtk::RecentManager::Data can't
	be	instantiated
To: gtkmm-forge lists sourceforge net
Message-ID: <20070422140021 27EC16C41B3 box 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=388086

  gtkmm | build | Ver: 2.10.x




------- Comment #9 from Murray Cumming  2007-04-22 14:00 UTC -------
The patch seems OK, though we might want to add those extra method overloads.
However, I have not yet decided whether we should commit this before branching
gtkmm.

It's hard to judge its importance without knowing when
gtk_recent_manager_add_full() (which it wraps) is needed. Grant, can you tell
us if this is actually very useful to you?


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



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

Message: 7
Date: Mon, 23 Apr 2007 18:49:51 +0000 (UTC)
From: "gtkmm (bugzilla.gnome.org)"
	<bugzilla-daemon bugzilla gnome org>
Subject: [gtkmm bugzilla] [Bug 388086] Gtk::RecentManager::Data can't
	be	instantiated
To: gtkmm-forge lists sourceforge net
Message-ID: <20070423184951 8F50F6C4224 box 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=388086

  gtkmm | build | Ver: 2.10.x




------- Comment #10 from Grant Patterson  2007-04-23 18:49 UTC -------
(In reply to comment #9)
> It's hard to judge its importance without knowing when
> gtk_recent_manager_add_full() (which it wraps) is needed. Grant, can you tell
> us if this is actually very useful to you?

A ran into this bug while implementing a most-recently-used list for VMware
Workstation and Player. I've got a virtual machine file path, name, MIME type,
app name, and command line. is_private is false and I have no use for group
names.

Currently (and I did this a while ago, so it's all a little hazy) I use the
GTK+ API to add each entry. I have to use those const_casts as you're doing in
the patch, which is assuredly a GTK+ bug as gtk_recent_manager_add_full treats
(and ought to treat) all fields in the data struct as const anyway.

I don't mind creating a Data object and filling in the fields that interest me.
Functions would be nice, and would require the caller to include those fields
that are required in the GTK+ API. (If the patch was submitted as-is, a caller
could create a Data object and just pass it to add_full uninitialized, with a
bunch of empty strings. Not desirable, and not possible in GTK+.) However the
permutations of included/unincluded arguments are many, and using different
names to differentiate these sounds like a real mess. I'd just like to see the
Data class instantiable; maybe we can think about revising the RecentManager
scheme in general at a later date.


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



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

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

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

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


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