Re: glib' error patch

[Darin Adler <darin eazel com>]

on 12/7/00 2:42 PM, Mathieu Lacage at mathieu eazel com wrote:

> @@ -29,6 +29,8 @@
> static GError* 
> g_error_new_valist(GQuark         domain,
> gint           code,
> +           const gpointer data,

"const gpointer" is not useful here. It just means the function body can't
modify its local copy of the <data> parameter, and has no effect on what you
can pass to the function or what the function can do. You mean "gpointer"
("gconstpointer" is something else). This mistake is repeated a few times in
both the C file and the header.

    -- Darin