djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land.
Generally looks good, one minor simplification. ================ Comment at: lib/Format/TokenAnnotator.cpp:2484 + if (Right.is(tok::r_brace) && Right.MatchingParen && + Right.MatchingParen->is(TT_DictLiteral) && + Right.MatchingParen->Previous && ---------------- Could you use Right.MatchingParen->endsSequence(TT_DictLiteral, tok::at) here? Repository: rC Clang https://reviews.llvm.org/D44816 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits