euhlmann added inline comments.
================
Comment at: lib/Format/UnwrappedLineParser.cpp:737
+ for (auto& Line : Lines) {
+ if (Line.InPPDirective && Line.Level > 0)
+ --Line.Level;
----------------
krasimir wrote:
> Wouldn't this also indent lines continuing macro definitions, as in:
> ```
> #define A(x) int f(int x) { \
> return x; \
> }
> ```
Multi-line macros look this under this patch
```
#if A
# define MACRO \
void a(int x) { \
b(); \
c(); \
d(); \
e(); \
f(); \
}
#endif
```
With previous clang-format, no preprocessor indentation:
```
#if A
#define MACRO \
void a(int x) { \
b(); \
c(); \
d(); \
e(); \
f(); \
}
#endif
```
https://reviews.llvm.org/D35955
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
