[ITK] [ITK-dev] New clang warnings: reserved-id-macro

Matt McCormick matt.mccormick at kitware.com
Tue Dec 16 03:06:41 EST 2014


On Mon, Dec 15, 2014 at 4:47 PM, Sean McBride <sean at rogue-research.com>
wrote:
>
> On Mon, 15 Dec 2014 09:08:43 -0500, Bradley Lowekamp said:
>
> >Thanks for taking this on.
> >
> >I believe at the TCON on friday we agreed with Matt's suggestion:
> >
> >itkFooBar_h
> >
> >I hope that there will be a fairly robust command line to do this work.
> >Please include it in the commit messages so that it can easily be done
> >again later for others modules.
>
> OK, my first patch is here, hopefully it won't explode gerrit:
>
> <
> http://review.source.kitware.com/#/q/status:open+project:ITK+branch:master+topic:include-guards,n,z
> >
>
>
Looks great!  I added a few minor comments on removing most of the .cxx
entries and improving the commit messages.

Thanks,
Matt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/community/attachments/20141216/c80abe9e/attachment.html>
-------------- next part --------------
_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at
http://www.kitware.com/opensource/opensource.html

Kitware offers ITK Training Courses, for more information visit:
http://kitware.com/products/protraining.php

Please keep messages on-topic and check the ITK FAQ at:
http://www.itk.org/Wiki/ITK_FAQ

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/insight-developers


More information about the Community mailing list