[PATCH] D93844: [clang-format] Add possibility to be based on parent directory

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 18 02:45:46 PST 2021


HazardyKnusperkeks added a comment.

In D93844#2504376 <https://reviews.llvm.org/D93844#2504376>, @MyDeveloperDay wrote:

> What I can't easily tell from the tests is if you are overriding any styles defined in the parent with a local style.

Yes I do, but I will try to make it clearer in the tests.



================
Comment at: clang/include/clang/Format/Format.h:57
+  // the parent directories. It is not part of the actual style for formatting.
+  // Thus the // instead of ///.
+  bool InheritsParentConfig;
----------------
MyDeveloperDay wrote:
> I don't quite understand this last sentence
If I would use `///` it would end up in the `ClangFormatStyleOptions.rst`. I only want to make clear it is no mistake that it only got `//` while all other members are documented with `///`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93844



More information about the cfe-commits mailing list