Hi,

On 28 February 2012 18:27, Ran Benita <[email protected]> wrote:
> Running the following semantic patch:
>
> @@
> expression E;
> @@
> - if (E != NULL)
> -     free(E);
> + free(E);
>
> Gives this:
> [...]
>
> And probably doesn't catch all of them. So maybe I can send this as a
> separate patch later?

Yes, please, that'd be fine.  You might also want to add if (!E) to your checks.

Cheers,
Daniel
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to