LGTM

Closed by https://reviews.llvm.org/rG91f60b44958f, https://reviews.llvm.org/rL357908, https://reviews.llvm.org/rC357908 (sorry, I forgot to update the body of the commit message to close this automatically)

Fixed formatting nit.

I updated ClangFormatStyleOptions.rst by hand, is there a way to do that automatically instead (for future patches)?

Updated with fixes. Thanks for pointing out that the options are ordered alphabetically, I hadn't noticed that!

Removed unnecessary parens.

@klimek I agree that the rule is somewhat arbitrary. However, it's the style rule of an established codebase with many users (I don't have a concrete number, but the project has 1400 stars on github). I've found this patch useful when contributing to JUCE and I thought others might too.

@MyDeveloperDay I'm sorry, you're absolutely correct. I'd got some other JUCE-related changes mixed up in this PR. Should be fixed now.

@MyDeveloperDay I'm not sure you built this branch. Perhaps you applied this patch to an older version of the repo. For me, SpaceBeforeCpp11BracedListOptions is defined at Format.h:1570.

LGTM

The code looks good now. There's just a few places left where the formatting looks a bit suspect (sorry for missing those last time). I think once that's fixed this will be good to go.

This looks much better now, thanks for taking another look! I've flagged some formatting/spelling nits, and I think the tests/docs could be a little more explicit about the behaviour of WithoutElse. Other than that, looks great!

Does it make sense to allow AllowShortIfElseStatementsOnASingleLine to be enabled when AllowShortIfStatementsOnASingleLine is not?

Thanks for the review, and for approving this PR. It's very much appreciated!

I've rebased onto master, and removed unrelated formatting changes. I've also tried to remove some of the duplicate parens-related expressions.

Would someone review this please? I'm not sure who to add for review (sorry), maybe one of the following?
@klimek @Typz @krasimir