[PATCH] D132256: [clang-format] Add DefinitionBlockSpacing option

Alecto Irene Perez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 22 13:35:21 PDT 2022


alecto added a comment.

If possible I would like to error out if MaxEmptyLinesToKeep < DefinitionBlockSpacing, because that case is probably a mistake on the part of the user, and a failure would be better than clang-format silently doing something unexpected, but I'm not sure where to put the logic for failure.

Where does clang-format handle style/input validation, and what method does it use for handling errors? (Does it rely on exceptions, or returning a "failure" state of some kind?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132256



More information about the cfe-commits mailing list