HazardyKnusperkeks added inline comments.
================
Comment at: clang/lib/Format/Format.cpp:963
LLVMStyle.SpacesInCStyleCastParentheses = false;
+ LLVMStyle.SpacesInLineComments = {/*Minimum=*/1, /*Maximum=*/-1u};
LLVMStyle.SpaceAfterCStyleCast = false;
----------------
curdeius wrote:
> I don't know precisely the LLVM style but does it allow more than one space
> (as Maximum would suggest)?
> Are there any tests covering that?
> And what about other styles, no need to set min/max for them?
The part with the LLVM Style from my test case did run exactly so without any
modification, so yes it allows more than one space.
Since there was no option before (that I'm aware of) all other styles behaved
exactly like that. I did not check the style guides if they say anything about
that, I just preserved the old behavior when nothing is configured.
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
