On Thu, Dec 10, 2009 at 10:45 AM, Jonathan Dixon <[email protected]> wrote:

> In essence:
>
> return DoWork(&foo)
> #if defined(OS_POSIX)
>     && DoWork(&posix_specific)
> #endif
>     ;  // <-- Lint complains about this guy
>

I'd prefer this:

#if defined(OS_POSIX)
  return DoWork(&foo) && DoWork(&posix_specific);
#else
  return DoWork(&foo);
#endif

The same number of lines, but much easier to read.

PK

-- 
Chromium Developers mailing list: [email protected] 
View archives, change email options, or unsubscribe: 
    http://groups.google.com/group/chromium-dev

Reply via email to