[PATCH] D27700: [clang-tidy] refactor ExprSequence out of misc-use-after-move check
Alexander Kornienko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 13 07:27:22 PST 2016
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
More information about the cfe-commits
mailing list