curdeius added a comment.
I thought about using signed ints for Minimum and Maximum in the style, but
then, we won't be able to use `-1` for Maximum the same way as now and signed
int makes no sense for Minimum...
================
Comment at: clang/lib/Format/BreakableToken.cpp:799-800
+ FirstLineSpaceChange = Minimum - SpacesInPrefix;
+ } else if (static_cast<unsigned>(SpacesInPrefix) >
+ Style.SpacesInLineCommentPrefix.Maximum) {
FirstLineSpaceChange =
----------------
Not strictly necessary, but without the cast we have a signed/unsigned mismatch
warning on comparison.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119680/new/
https://reviews.llvm.org/D119680
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
