Re: Comments on 95362 (tree dnd) patch

[Matthias Clasen <maclas gmx de>]

Am Fre, 2003-09-05 um 21.48 schrieb Owen Taylor:

> Two tiny comments from browsing the path.
> 
> ===
> +static void
> +dest_row_free (gpointer data)
> +{
> +  DestRow *dr = (DestRow *)data;
> +
> +  if (!dr)
> +    return;
> ==
> 
> This isn't right - free functions never get NULL passed to them.
> 

Really ? The fact that g_free() silently accepts NULL is documented and
widely used behaviour.

Matthias