[PATCH] D34649: Remove addtional parameters in function std::next() and std::prev()

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 5 17:55:37 PDT 2017


EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.

> Creating a function pointer with proper parameters pointing to std::next() or std::prev() should work.

Actually, according to the standard taking the address of a STL function is UB, and users should avoid doing it as much as possible since it's fundamentally non-portable.

However as a QoI change I think this is OK.


https://reviews.llvm.org/D34649





More information about the cfe-commits mailing list