On Tuesday 26 April 2011 13:53:33 Kostik Belousov wrote:
> On Tue, Apr 26, 2011 at 01:44:00PM +0200, Hans Petter Selasky wrote:
> > On Tuesday 26 April 2011 13:39:56 Konstantin Belousov wrote:
> > > + pending = !!callout_stop(&timeout_task->c);
> >
> > pending = (callout_stop(&timeout_task->c) != 0);
> >
> > ?
Hi,
This is just a nit I noticed.
>
> This line is about conversion from a boolean value to {0, 1} value set.
> If !! construct does not look stylish, then wouldn't we need to go
> with
> pending = (callout_stop(&timeout_task->c) != 0) ? 1 : 0;
> instead ?
The output from ! is already a boolean and "pending" is a u_int, so to be
correct it should be similar to what you suggest. I'm not sure what case
produce the less amount of code and which the compiler understands the best.
Probably it does not matter that much.
I was thinking that "!!" is depreceated, but "man style" is silent about it.
> Feel free to adjust whatever variant you prefer and commit it.
Ok.
--HPS
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
