From: Tay Ray Chuan <rctay89@gmail.com>
Date: Thu, 07 Jul 2011 04:05:26 GMT
Subject: Re: [PATCH v2 2/4] xdiff/xprepare: refactor abort cleanups
Message-ID: <CALUzUxoSBVfzEhvKQahhRXCDsFs5nQvfHeohHUJMeJ7QYMv3wA@mail.gmail.com>
In-Reply-To: <4E152E31.4020105@cisco.com>

On Thu, Jul 7, 2011 at 11:55 AM, Phil Hord <hordp@cisco.com> wrote:
> Because 'free(NULL)' is handled sanely (as a no-op).
> http://pubs.opengroup.org/onlinepubs/009695399/functions/free.html

Yup, I got what Junio meant after he posted an example. (Thanks Junio.)

> I haven't looked, but I assume xdl_cha_free does the same thing.  I only
> assume this because Junio seems to imply it, though.

xdl_cha_(init|free)() do something more complicated, so we don't have
to set xdf->cha to NULL, unlike the others.

-- 
Cheers,
Ray Chuan