Some questions/ideas about the GdaSqlBuilder API

[Johannes Schmid-4]

Hi!

The API is not really straight-forward to use especially because the
method names don't give a clear indication of what happens. Murray made
some suggestions to me:

gda_sql_builder_ident => gda_sql_builder_add_id
(Murray complained that "ident" would remind him of "identity" instead
of "identifier")
gda_sql_builder_expr => gda_sql_builder_add_expr
gda_sql_builder_cond = gda_sql_builder_add_cond
gda_sql_builder_param = gda_sql_builder_add_param

IMHO that would make it clearer that something is added to the statement
(or at least to the pool of things the SqlBuilder knows about).

Regards,
Johannes


_______________________________________________
gnome-db-list mailing list
gnome-db-list@...
http://mail.gnome.org/mailman/listinfo/gnome-db-list

[Vivien Malerba]

2009/10/1 Johannes Schmid <jhs@...>:

> Hi!
>
> The API is not really straight-forward to use especially because the
> method names don't give a clear indication of what happens. Murray made
> some suggestions to me:
>
> gda_sql_builder_ident => gda_sql_builder_add_id
> (Murray complained that "ident" would remind him of "identity" instead
> of "identifier")
> gda_sql_builder_expr => gda_sql_builder_add_expr
> gda_sql_builder_cond = gda_sql_builder_add_cond
> gda_sql_builder_param = gda_sql_builder_add_param
>
> IMHO that would make it clearer that something is added to the statement
> (or at least to the pool of things the SqlBuilder knows about).

Sounds good to me. I can do this tonight if you want.

Vivien
_______________________________________________
gnome-db-list mailing list
gnome-db-list@...
http://mail.gnome.org/mailman/listinfo/gnome-db-list

[Johannes Schmid-4]

> Sounds good to me. I can do this tonight if you want.

Cool, thanks!

Johannes


_______________________________________________
gnome-db-list mailing list
gnome-db-list@...
http://mail.gnome.org/mailman/listinfo/gnome-db-list