sstwcw marked 7 inline comments as done. sstwcw added inline comments.
================ Comment at: clang/docs/tools/dump_format_style.py:293 state = State.InNestedStruct - field_type, field_name = re.match(r'([<>:\w(,\s)]+)\s+(\w+);', line).groups() + field_type, field_name = re.match(r'([<>:\w(,\s)]+)\s+(\w+)', line).groups() if field_type in enums: ---------------- This change is needed to parse the entry `bool PadOperators = true;`. ================ Comment at: clang/lib/Format/WhitespaceManager.cpp:273 +// RightJustify - Whether it is the token's right end or left end that +// gets moved to that column. template <typename F> ---------------- MyDeveloperDay wrote: > something odd here why not, the following its fits 80 columns > > ``` > // Column - The token for which Matches returns true is moved to this column. > // RightJustify - Whether it is the token's right end or left end that gets > // moved to that column. > ``` My text editor automatically limited comments to 72 columns. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119599/new/ https://reviews.llvm.org/D119599 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits