Yep.  Thanks for catching it!

-John



On Mon, Aug 5, 2013 at 11:58 PM, Kim Gräsman <[email protected]> wrote:

> Hi John,
>
> On Tue, Aug 6, 2013 at 2:17 AM, John Thompson
> <[email protected]> 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
[email protected]
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to