HazardyKnusperkeks added inline comments.

================
Comment at: clang/lib/Format/BreakableToken.cpp:790
+          (Style.Language != FormatStyle::LK_TextProto ||
+           OriginalPrefix[i].substr(0, 2) != "##")) {
+        Prefix[i] = IndentPrefix.str();
----------------
HazardyKnusperkeks wrote:
> MyDeveloperDay wrote:
> > is this case covered by a unit test at all? sorry can you explain why you 
> > are looking for "##"?
> It is covered by multiple tests, that's how I was made aware of it. :)
> If you look at the code before it only adds a space if the old prefix is "#" 
> not "##" which is also found by `getLineCommentIndentPrefix`. As it seems in 
> `TextProto` "##" should not be touched. I can of course add a test in my test 
> function.
> 
> Now I see a change, in the code before "#" was only accepted when the 
> language is `TextProto`, now it is always. But I think for that to happen the 
> parser (or lexer?) should have assigned something starting with"#" as 
> comment, right? But I can change that.
Okay # # is formatted, I try again:
If you look at the code before it only adds a space if the old prefix is "#" 
not "`##`" which is also found by `getLineCommentIndentPrefix`. As it seems in 
`TextProto` "`##`" should not be touched.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92257/new/

https://reviews.llvm.org/D92257

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to