[PATCH] D88084: [clang-format] Changed default styles BraceWrappping bool table to directly use variables

Marcus Johnson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 22 13:44:50 PDT 2020


MarcusJohnson91 added a comment.

In D88084#2287450 <https://reviews.llvm.org/D88084#2287450>, @MyDeveloperDay wrote:

> I noticed the pre-merge tests failed!

Yeah I just noticed that too, not sure what's up but I'll check into it, and yeah that's a good idea about initializing some of these duplicate variables in the constructor.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88084



More information about the cfe-commits mailing list