Hi Michal,

> it may make one think that the pointer is an int.
> Use explicit comparison against NULL (or !ptr).

I agree with you. Use of '0' to designate a NULL pointer is confusing
to most of us.

>  lib/alloca.c         | 10 +++++-----
>  tests/test-tsearch.c |  2 +-

Applied.

>  lib/getloadavg.c     | 12 ++++++------

Here I preferred to write NULL instead of the boolean negation operator.

>  lib/backup-find.c    |  2 +-
>  lib/obstack.c        | 10 +++++-----
>  lib/obstack.h        |  2 +-
>  tests/test-regex.c   |  2 +-

Paul may want to handle these.

>  lib/bitset/list.c    |  2 +-
>  lib/bitset/table.c   |  2 +-
>  lib/timevar.c        |  2 +-

Akim may want to handle these.

Bruno


Reply via email to