[clang-tools-extra] r187759 - Add test for header guard work-around.

John Thompson john.thompson.jtsoftware at gmail.com
Tue Aug 6 07:09:54 PDT 2013


Yep.  Thanks for catching it!

-John



On Mon, Aug 5, 2013 at 11:58 PM, Kim Gräsman <kim.grasman at gmail.com> wrote:

> Hi John,
>
> On Tue, Aug 6, 2013 at 2:17 AM, John Thompson
> <John.Thompson.JTSoftware at gmail.com> wrote:
> >
> > Added: clang-tools-extra/trunk/test/modularize/Inputs/HeaderGuardSub1.h
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/modularize/Inputs/HeaderGuardSub1.h?rev=187759&view=auto
> >
> ==============================================================================
> > --- clang-tools-extra/trunk/test/modularize/Inputs/HeaderGuardSub1.h
> (added)
> > +++ clang-tools-extra/trunk/test/modularize/Inputs/HeaderGuardSub1.h Mon
> Aug  5 19:17:40 2013
> > @@ -0,0 +1,4 @@
> > +#ifndef _HEADERGUARDSUB1_H_
> > +#define _HEADERGUARDSUB1_H_
> > +#include "HeaderGuardSubSub.h"
> > +#endif // _HEADERGUARDSUBSUB_H_
>
> Not that it matters except in the cosmetic, but this should be
>
>    #endif // _HEADERGUARDSUB1_H_
>
> right?
>
> - Kim
>



-- 
John Thompson
John.Thompson.JTSoftware at gmail.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130806/9adc2d38/attachment.html>


More information about the cfe-commits mailing list