This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG253985d58caf: [clang-format] Add MinDigits suboptions to
IntegerLi
owenpan requested review of this revision.
owenpan marked an inline comment as done.
owenpan added inline comments.
Comment at: clang/include/clang/Format/Format.h:4299
InsertNewlineAtEOF == R.InsertNewlineAtEOF &&
IntegerLiteralSeparator.Binary == R.Integ
owenpan updated this revision to Diff 509511.
owenpan added a comment.
Adds `IntegerLiteralSeparatorStyle::operator==`.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147111/new/
https://reviews.llvm.org/D147111
Files:
clang/docs/ClangFormatStyleOptions.rst
clang/include/clang/Forma
HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/include/clang/Format/Format.h:4299
InsertNewlineAtEOF == R.InsertNewlineAtEOF &&
IntegerLiteralSeparator
owenpan created this revision.
owenpan added reviewers: MyDeveloperDay, HazardyKnusperkeks, rymiel.
owenpan added a project: clang-format.
Herald added a project: All.
owenpan requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Closes https://g