Re: gnome_db_control_widget



> Ok, then how about adding a function to tell if a component supports a property
> (the same way I did with the static function is_prop_supported()), and two
> generic functions to which we pass the GnomeDbControlWidget, the property
> name the argument.
> 
> void gnome_db_control_widget_get_prop(GnomeDbControlWidget *wid, const gchar
> *prop_name, ...);
> void gnome_db_control_widget_set_prop(GnomeDbControlWidget *wid, const gchar
> *prop_name, ...);
> 
> The ... MUST be ONE argument for both functions.
> the set function will require the arg to be of type gboolean, gchar *,...
> and for the get function of type gboolean*, gchar**, ...
> 
Yes, perfect! The only thing to avoid is to do it specific for
something.

> > I will commit very soon what I've done with the GnomeDbControl class and
> > then you can see how I've done this.
> 
> Ok!
I'm finishing, so the commit will be in a couple of hours.

> 
> Maybe we could even #define some very usefull property names like
> #define GNOME_DB_CONTROL_PROP_DSN "dsn"
> etc.
> 
Yes again, thus we don't have to write over and over the same string,
and we can safely change it in the futture if we need to.
> 
> PS: do you think I should add any functionnalities to the gnome-db control
> browser or is it enough like that?
> 
Well, it's a test program (or could it be a generic controls browser?),
so add whatever you need to test.




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]