[PATCH] D67627: Clang-format: Add Whitesmiths indentation style

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 17 04:53:38 PDT 2019


krasimir added a comment.

Adding @djasper  as this is really a re-implementation of https://reviews.llvm.org/D6833.
Agree with @MyDeveloperDay that in general we should aim to document all the known cases where a style doesn't work (with FIXMEs etc.), but not submit failing tests. We can have tests that demonstrate the current (bad) behavior together with a FIXME comment about what is the expected good outcome instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67627





More information about the cfe-commits mailing list