[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