[PATCH] D19586: Misleading Indentation check

Etienne Bergeron via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 29 10:51:30 PDT 2016


etienneb added a comment.

In http://reviews.llvm.org/D19586#417063, @xazax.hun wrote:

> In http://reviews.llvm.org/D19586#417053, @etienneb wrote:
>
> > The rule also apply for statements in a same compound:
> >
> >   {
> >     statement1();
> >     statement2();
> >       statement3();
> >
> >
> > But this can be a further improvement.
>
>
> I believe this might be an intentional omission, since this can not imply semantic problems. But I agree that, this addition makes sense.


Fair enough. This checker is in 'readbility', so I don't see why not. 
But, feel free to postpone. Or let someone else take it.


http://reviews.llvm.org/D19586





More information about the cfe-commits mailing list