[PATCH] D104222: [clang-format] [PR50702] Lamdba processing does not respect AfterClass and AfterNamespace

Christophe Calmejane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 16 01:23:03 PDT 2021


christophe-calmejane added a comment.

In D104222#2821375 <https://reviews.llvm.org/D104222#2821375>, @MyDeveloperDay wrote:

>> @MyDeveloperDay 
>> Of course, sorry for the delay, here it is: https://github.com/christophe-calmejane/Hive/blob/master/.clang-format
>
> You don't have `AllowShortLambdasOnASingleLine` set to be `false` or `None`

Just a clarification, this file is meant to be used with clang-format v7.0.0 along with my small patch fixing formatting issues that were not included in the original merge (see the other post). This means it's using the defaults from back then ;)
I never had a chance to upgrade clang-format because lambda formatting was broken (some of my test cases were failing). I will try to build your latest patch and see how it goes. I would expect my project re-formatting to not change much :D
Thanks for your work btw.


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

https://reviews.llvm.org/D104222



More information about the cfe-commits mailing list