[ITK-dev] New clang warnings: reserved-id-macro
Bradley Lowekamp
blowekamp at mail.nih.gov
Fri Dec 12 09:22:06 EST 2014
And all the remote modules should be updated as well...
On Dec 11, 2014, at 3:30 PM, Sean McBride <sean at rogue-research.com> wrote:
> On Thu, 11 Dec 2014 15:20:31 -0500, Bradley Lowekamp said:
>
>> Having the header guarder not begin with an '_' seems wrong to me.
>>
>> What about _<project>_<filename>?
>
> That works too as long as <project> starts lowercase, as starting with underscore+uppercase is also reserved. So "_itkFooBar_h" would be ok, but not "_ITKFOOBAR_H" for example.
>
> Adding another underscore between seems gratuitous though.
>
> This change will touch hundreds and hundreds of files, so you guys decide on the scheme, then I'll do it. :)
>
> Cheers,
>
> --
> ____________________________________________________________
> Sean McBride, B. Eng sean at rogue-research.com
> Rogue Research www.rogue-research.com
> Mac Software Developer Montréal, Québec, Canada
>
>
More information about the Insight-developers
mailing list