Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adapt transform for #1668 #1671

Merged
merged 10 commits into from
Jul 28, 2015
Merged

Adapt transform for #1668 #1671

merged 10 commits into from
Jul 28, 2015

Conversation

hkaiser
Copy link
Member

@hkaiser hkaiser commented Jul 19, 2015

Adapt transform algorithm to conform to N4382 (Working Draft, C++ extensions for Ranges)

  • changing return types of transform
  • added projections
  • added overload for transform taking [first1, last1) and [first2, last2)
  • added transform_binary2 test

- added Proj parameter to for_each and for_each_n
- let for_each return iterator
- fly-by fix for distributed algorithms (don't forward arguments more than once)
- added projections
- added overload for transform taking [first1, last1) and [first2, last2)
- added transform_binary2 test
@hkaiser hkaiser force-pushed the adapt_transform_1668 branch 2 times, most recently from 92165d0 to 8b7b361 Compare July 20, 2015 23:47
: base_type(it), data_(data)
{}

local_iterator remote()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks odd: The const iterator returning a non-const iterator to its local counterpart.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a typedef to the const version anyways, but yah - I'll change it.

hkaiser added a commit that referenced this pull request Jul 28, 2015
@hkaiser hkaiser merged commit f6a6dde into master Jul 28, 2015
@hkaiser hkaiser deleted the adapt_transform_1668 branch July 28, 2015 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants