On Fri, Jun 19, 2009 at 11:15 AM, Mike Pinkerton <pinker...@chromium.org>wrote:

> On Fri, Jun 19, 2009 at 2:11 PM, Peter Kasting<pkast...@google.com> wrote:
> > On Fri, Jun 19, 2009 at 11:07 AM, Nicolas Sylvain <nsylv...@chromium.org
> >
> > wrote:
> >>
> >> We should say that we prefer  "#if defined(OS_WIN)" over "#ifdef
> OS_WIN".
> >
> > In general, you should always prefer "#if defined(FOO)" over "#ifdef
> FOO",
> > e.g. because you can add "|| defined(BAR)" to the former later.
> > PK
>
> This is already documented in our Chromium coding style guide. Do we
> need to replicate it?


Oops. It was not there when I look at the coding style a long time ago..
Nevermind!

Thanks

Nicolas


>
>
> --
> Mike Pinkerton
> Mac Weenie
> pinker...@google.com
>

--~--~---------~--~----~------------~-------~--~----~
Chromium Developers mailing list: chromium-dev@googlegroups.com 
View archives, change email options, or unsubscribe: 
    http://groups.google.com/group/chromium-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to