[Gtk-sharp-list] patch for gnome binding

Mike Kestner mkestner@ximian.com
Mon, 12 Jul 2004 11:34:58 -0500


On Mon, 2004-07-12 at 10:55, Martin Willemoes Hansen wrote:

> > I'm not convinced it's a good idea to expose set_Config publicly. I
> > would just invoke the glue method from the ctor instead and leave the
> > property read only.
> 
> Ok, here is yet another updated patch ;) 

Why did you switch to using g_object_set?  Are you certain that will
take a ref and assign the config field of the GnomePrintJob struct for
the job?  It seems really inefficient at best to do it that way.  Why
not just:

job->config = gnome_print_config_ref (config);

Also, your .custom should import and use
gnomesharp_print_job_set_print_config, since that's what you named the
method in the glue.

-- 
Mike Kestner <mkestner@ximian.com>