Prazek added inline comments.

================
Comment at: clang-tidy/utils/ExprSequence.cpp:154
+    return SyntheticStmtSourceMap.lookup(S);
+  else
+    return S;
----------------
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


https://reviews.llvm.org/D27700



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to