Re: [PATCH v2 2/4] xdiff/xprepare: refactor abort cleanups
- From
Tay Ray Chuan <rctay89@gmail.com>
- Date
- Jul 7, 2011, 04:05 UTC
- 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