Re: CVS commit: src/sys/kern

[Valery Ushakov <uwe%stderr.spb.ru@localhost>]

On Mon, Oct 10, 2016 at 01:22:08 +0000, David A. Holland wrote:

> Modified Files:
> 	src/sys/kern: uipc_socket.c
> 
> Log Message:
> PR 49636 Henning Petersen: use "0L" to return 0 from a function returning
> long, and test its returned value against "0L" instead of "0".
> 
> This is not especially necessary, but it's also harmless.

No, it's not.  It sends a strong signal that integer promotions are
not enough here, and that explicit cast/sufix is required for some
reason.  Yet, no such reason is provided in the PR.  Please revert.
Or are you going to tag all long zeroes in the tree as 0L?

-uwe