[PATCH] D22725: [clang-tidy] Add check 'modernize-use-algorithm'

Jonas Devlieghere via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 23 11:33:32 PST 2016


JDevlieghere abandoned this revision.
JDevlieghere added a comment.

I'm abandoning this revision because I think this check is getting overly complex. There's still the problem of supporting arguments that can have side effects, and then there's also the unaddressed issue of code possibly using the void* return type of these two functions. Any solution to this would require rather inelegant and complex code that is, in my opinion, simply not warranted by the added value of this check.


Repository:
  rL LLVM

https://reviews.llvm.org/D22725





More information about the cfe-commits mailing list