Re: [PATCH] 2.4.21-rc1 pointless IDE noise reduction

[Bill Davidsen]

On Thu, 24 Apr 2003, Richard B. Johnson wrote:

> > Two alternatives:
> >
> > (a)     !!(x & 0x400)
> >
> > (b)     (x & 0x400) >> 10
> >
> 
> I meant return ((foo & MASK) && 1);
> 
> Try it, you'll like it! No shifts, no jumps.

Sorry, I still find !!(foo & MASK) easier to read, because !! is only used
to convert to boolean. Sort of a "boolean cast" in effect. It jumps out at
you what is intended.

Anyway, a matter of taste, both generate jumpless code.

-- 
bill davidsen <[e-mail]>
  CTO, TMR Associates, Inc
Doing interesting things with little computers since 1979.