[PATCH] D147111: [clang-format] Add MinDigits suboptions to IntegerLiteralSeparator

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 29 03:37:17 PDT 2023


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.Binary == R.IntegerLiteralSeparator.Binary &&
+           IntegerLiteralSeparator.BinaryMinDigits ==
----------------
Could we move that to (a not yet existing) `IntegerLiteralSeparatorStyle::operator==`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147111



More information about the cfe-commits mailing list