JonasToth added inline comments.
================
Comment at: clang/lib/Format/Format.cpp:1792
+ else if (Trimmed == "// clang-format on" ||
+ Trimmed == "/* clang-format on */")
FormattingOff = false;
----------------
MyDeveloperDay wrote:
> JonasToth wrote:
> > Should we allow
> > ```
> > /* clang-format off
> > It is just horrible for this piece of code. */
> > ```
> >
> > ? Multiline-comments could span multiple lines and to deactivates
> > clang-format and give reasons.
> initially that is kind of what I was trying to do, but if you look over in
> TokenLexer where its turned off for styling they check only for a single line
>
> So to be honest if we are going to change it we should change it in both
> places, but perhaps that is just overkill.
Totally, not necessary in my eyes, nice plus, so dont implement it :D
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58819/new/
https://reviews.llvm.org/D58819
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits