[PATCH] D140543: [clang-format] Add an option to format integer literal separators

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 1 06:38:56 PST 2023


aaron.ballman added inline comments.


================
Comment at: clang/docs/ClangFormatStyleOptions.rst:3168-3171
+  Separator format of integer literals of different bases.
+  <0: Remove separators.
+   0: Leave the literal as is.
+  >0: Insert separators between digits, starting from the rightmost digit.
----------------
This is causing the Sphinx build to fail:

Warning, treated as error:
/home/buildbot/llvm-build-dir/clang-sphinx-docs/llvm/build/tools/clang/docs/ClangFormatStyleOptions.rst:3170:Unexpected indentation.

https://lab.llvm.org/buildbot/#/builders/92/builds/38032

I think you need to remove one leading whitespace from line 3170 so that `0` aligns with `<` from the line above.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140543



More information about the cfe-commits mailing list