Looks good to me.

Laca

On Wed, 2007-03-14 at 14:26 +0000, Damien Carbery wrote:
> Although this is a very basic patch, I am sending it for review because 
> I am unsure whether it is the right solution.
> 
> libgnomeprint 2.18.0 fails with the following compilation error:
> "gpa-utils.c", line 118: operands have incompatible types:
>           pointer to unsigned char ":" pointer to char
> 
> I changed:
> -g_print (" {%s}", ((GPAKey *) node)->value ? ((GPAKey *) node)->value : 
> "");
> +g_print (" {%s}", ((GPAKey *) node)->value ? ((GPAKey *) node)->value : 
> (guchar *)"");
> (note the cast of the empty string).
> 
> The commit message for the change in svn.gnome.org is:
> * libgnomeprint/gpa/gpa-utils.c (gpa_utils_dump_tree_with_level):
>       Guard against printing NULL for %s.
> http://svn.gnome.org/viewcvs/libgnomeprint/trunk/libgnomeprint/gpa/gpa-utils.c?r1=1804&r2=2240
> 
> Damien
> 


Reply via email to