>These used to be necessary but now that NULL is (void *)0 they
>aren't.
>
>I'd be inclined to keep them anyway - with the cast it is clearly not a
>bug, without is not so clear. There are also loads of them and it'd be a
>lot of churn to fix them all, for no real advantage. But I don't feel
>strongly about it, others may disagree.

It is neccesary for portable code.

The discussion goes a bit like this:

1. It is not needed here.
2. It is not needed in src/bin
3. It is not needed in usr.bin
4. Oh look, we deleted it out of sshd...

Seems risky.

Reply via email to