MyDeveloperDay added inline comments.
================ Comment at: clang/lib/Format/TokenAnnotator.cpp:1682 Current.NestingLevel == 0 && - !Current.Previous->is(tok::kw_operator)) { + !Current.Previous->isOneOf(tok::kw_operator, tok::identifier)) { // not auto operator->() -> xxx; ---------------- curdeius wrote: > MyDeveloperDay wrote: > > curdeius wrote: > > > Won't it break lambdas with an identifier (e.g. macro) before the arrow? > > > E.g.: > > > ``` > > > auto lmbd = [] NOEXCEPT -> int { > > > return 0; > > > }; > > > ``` > > it doesn't seem so, maybe one of the other rules is catching that > > > > auto lmbd = [] NOEXCEPT -> int { return 0; }; > > > Ok, but I'd like to see it as a test case if we don't have something similar > already. let me add that for completeness before I commit Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115903/new/ https://reviews.llvm.org/D115903 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits