curdeius accepted this revision. curdeius added a comment. This revision is now accepted and ready to land.
LGTM. ================ Comment at: clang/lib/Format/TokenAnnotator.cpp:2376 void TokenAnnotator::annotate(AnnotatedLine &Line) { - for (SmallVectorImpl<AnnotatedLine *>::iterator I = Line.Children.begin(), - E = Line.Children.end(); - I != E; ++I) { - annotate(**I); - } + for (auto &Child : Line.Children) + annotate(*Child); ---------------- Nit: I'd keep the type. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115067/new/ https://reviews.llvm.org/D115067 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits