[PATCH] D75791: [clang-format] Added new option IndentExternBlock

Sat Mar 7 08:35:26 PST 2020

MarcusJohnson91 added a comment.

In D75791#1911133 <https://reviews.llvm.org/D75791#1911133>, @MyDeveloperDay wrote:

> you need documentation and release note changes too

The comments were only for testing, I'll remove them.

The tests had to change because the behavior has changed slightly.

In practice it should be the same because LLVMStyle.IndentExternBlock default is set to false, but previously the BraceWrapping.AfterExternBlock = true code would indent as well, and now the behavior of BraceWrapping.AfterExternBlock only effects the brace wrapping.

As for the release notes, which file should I edit for that, and also which version will this even end up in? probably 11 right, because 10 is in RC status right now?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75791