After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 129899 - Binding friendly gnome_program_init variant
Binding friendly gnome_program_init variant
Status: RESOLVED FIXED
Product: libgnome
Classification: Deprecated
Component: general
HEAD
Other All
: Urgent major
: ---
Assigned To: libgnome maintainer
libgnome maintainer
Depends on:
Blocks: 140059
 
 
Reported: 2003-12-23 15:16 UTC by Torsten Schoenfeld
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.5/2.6

Attachments
New API proposal (6.71 KB, patch)
2004-07-04 09:58 UTC, Gustavo Carneiro 		none Details | Review

Description Torsten Schoenfeld 2003-12-23 15:16:37 UTC 
Aloha,

since 2003/11/27 the GObject library doesn't allow construct-only
properties to be set after construction anymore (revision 1.63 and 1.64 of
glib/gobject/gobject.c).

Unfortunately, the Perl bindings for libgnome relied on that ability to
wrap gnome_program_init.

In order to get it working again, we'd need a gnome_program_init variant
that takes a GValue array or something similiar instead of a va_list.

Could such a function be added?

TIA,
-Torsten
Comment 1 Gustavo Carneiro 2004-07-04 09:58:04 UTC 
Created attachment 29199 [details] [review]
New API proposal

This patch adds a new API function that receives guint nparams, GParameter
*params.  It is consistent with g_object_newv API:
gpointer g_object_newv (GType object_type, guint n_parameters, GParameter
*parameters);
Comment 2 Gustavo Carneiro 2004-07-14 13:34:04 UTC 
API freeze is just 5 days away!  Can I commit, please?  I'm sure this patch
statisfies both python and perl bindings.
Comment 3 Gustavo Carneiro 2004-07-18 13:50:14 UTC 
API freeze tomorrow! ppp-lease!
Comment 4 Anders Carlsson 2004-07-18 14:07:16 UTC 
Sorry for the late reply, I've been away. 

Although GnomeProgram has other problems wrt bindings, and although I'd like to
get rid of it, I don't really see any problem with this patch.

One minor comment, I don't think the inline is needed, but other than that it's
good to commit.
Comment 5 Anders Carlsson 2004-07-19 09:51:01 UTC 
I went ahead and committed this.