[PATCH] D35955: clang-format: Add preprocessor directive indentation

Manuel Klimek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 9 01:47:48 PDT 2017


klimek added a comment.

In https://reviews.llvm.org/D35955#835994, @euhlmann wrote:

> In https://reviews.llvm.org/D35955#835439, @klimek wrote:
>
> > I think if we need this info, we can just make it count down to -1 again (or, but that's isomorphic, let it run from 0 and make sure we never index into the data structures at 0 :)
>
>
> Should I do one of these things? Let me know how you'd like me to implement this.


So my suggestion would be to let it count down to -1 again and put a check around indexing into the data structures.


https://reviews.llvm.org/D35955





More information about the cfe-commits mailing list