alexfh added inline comments.
================ Comment at: clang-tidy/utils/ExprSequence.cpp:154 + return SyntheticStmtSourceMap.lookup(S); + else + return S; ---------------- Prazek wrote: > alexfh wrote: > > nit: No `else` after return, please. > Not sure if he should change it in this patch - it is just move of this class > to different file, so I am not sure if it is good do introduce small changes > to it now. > I guess pushin NFC patch with this after would be good solution For me it's usually easier to fix than to postpone to a different patch. Feel free to do either. https://reviews.llvm.org/D27700 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
