[PATCH] [clang-tidy] Checker for inefficient use of algorithms on associative containers
Alexander Kornienko
alexfh at google.com
Mon Feb 23 08:51:38 PST 2015
In http://reviews.llvm.org/D7246#124990, @xazax.hun wrote:
> Thank you for the feedback. I think I should have seen the first problem in advance. I find the second one more subtle. Both should be fixed in r229552. I am not absolutely happy with the solution: checking for type equality modulo references. It would be better to check if appropriate type conversions exists, but I did not find a nice way to do it. What is your opinion?
I don't know if a better solution exists. The fix looks fine. Thanks!
http://reviews.llvm.org/D7246
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list