
28 Feb
2010
28 Feb
'10
8:18 p.m.
Mathias Gaunard wrote:
As asked during review, you made boost::adaptors::filtered and friends function objects.
My bad, I was in a hurry and didn't take a proper look. I meant that you made filtered(range, predicate) work, on top of range | filtered(predicate), as was asked by some people during review if I remember well. But it appears you didn't. This version doesn't seem to be any different than the review one.
However, they take their arguments by copy (which can be a serious problem with ranges), and don't provide a result_type or result<> template.
If the function object is only used for the operator| syntax, of course, this is a lesser concern.