MyDeveloperDay added inline comments.
================ Comment at: clang/lib/Format/TokenAnnotator.cpp:3803 + if (Line.startsWith(Keywords.kw_interface) && + Style.BraceWrapping.AfterClass) + return true; ---------------- krasimir wrote: > nit: the `&& Style.BraceWrapping.AfterClass` is unnecessary since we've > ensured that in the parent `if`. I'd just fold the `kw_interface` case into > the previous `if`, and then it seems we can further fold the remaining inner > `if` into the parent `if`, resulting in this whole section (lines 3796--3805) > turning into a single large `if` statement. nice catch, will do CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108810/new/ https://reviews.llvm.org/D108810 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
