mclow.lists added a comment. Two comments answered. One still remaining.
================ Comment at: include/experimental/functional:256 @@ +255,3 @@ + +public: // TODO private: + _RandomAccessIterator1 __first_; ---------------- EricWF wrote: > Is this for testing? Well, for debugging. My first comment addressed this (basically, it will get changed). ================ Comment at: include/experimental/functional:313 @@ +312,3 @@ + + void build_suffix_table(_RandomAccessIterator1 __f, _RandomAccessIterator1 __l, + _BinaryPredicate __pred) ---------------- EricWF wrote: > This needs to be a reserved identifier right? You are correct. http://reviews.llvm.org/D11380 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
