Hi Jindrich,

On Tue, 2005-09-13 at 13:07 +0200, Jindrich Novy wrote:
> -           my_second = resolve_symlinks (second);
> -           if (my_second == NULL) {
> -               g_free (my_first);
> +           if (my_second == NULL)
>                 return buf;
> -           }

Why do you drop the g_free of my_first here?

By the way, this patch is a bit sloppy wrt whitespace. And could you
please add a change log entry?

Leonard.

-- 
mount -t life -o ro /dev/dna /genetic/research


_______________________________________________
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel

Reply via email to