svn commit: r221059 - in head/sys: kern sys
Hans Petter Selasky
hselasky at c2i.net
Tue Apr 26 12:08:49 UTC 2011
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
More information about the svn-src-head
mailing list