This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [patch] Cleanup: don't call free(NULL) unnecessarily
On Tue, Mar 5, 2013 at 7:18 PM, Siddhesh Poyarekar
<siddhesh.poyarekar@gmail.com> wrote:
> On 6 March 2013 00:48, Paul Pluzhnikov <ppluzhnikov@google.com> wrote:
>> + if (__builtin_expect (workstart != NULL, 0))
>> + free (workstart);
>
> Sorry, I'm a little late on this, but this should have been
> __glibc_unlikely() instead of __builtin_expect.
Thanks. Should I clean up other instances of __builtin_expect in the
same file as well?
--
Paul Pluzhnikov