Skip to content

Kirkshoop/filterimprovement#371

Merged
kirkshoop merged 2 commits intoReactiveX:masterfrom
kirkshoop:kirkshoop/filterimprovement
Apr 10, 2017
Merged

Kirkshoop/filterimprovement#371
kirkshoop merged 2 commits intoReactiveX:masterfrom
kirkshoop:kirkshoop/filterimprovement

Conversation

@kirkshoop
Copy link
Copy Markdown
Member

proposed changes to #364

@elelel, does this look okay?

whitespace changes are due to enforcement of the .editorconfig

@ghost ghost added the cla-not-required label Apr 7, 2017
@elelel
Copy link
Copy Markdown
Contributor

elelel commented Apr 7, 2017

Sure, I'm not informed enough to make suggestions here :)

Are we safe from the user calling on_next with a const rvalue directly?

@kirkshoop kirkshoop force-pushed the kirkshoop/filterimprovement branch from b6bd14e to b274a85 Compare April 9, 2017 00:43
@kirkshoop
Copy link
Copy Markdown
Member Author

Thanks!

Yes, one of the weird parts of perfect-forwarding in C++ is that since the Value&& v parameter is named, it is not an R-Value which is why std::forward must be used to recover the original type.

This means that this use of as_const in on_next will not ever be passed an R-Value.

@kirkshoop kirkshoop force-pushed the kirkshoop/filterimprovement branch from b274a85 to d6edcab Compare April 10, 2017 20:06
@kirkshoop
Copy link
Copy Markdown
Member Author

@elel, thank you for the help in getting this done.

@kirkshoop kirkshoop merged commit 201dd99 into ReactiveX:master Apr 10, 2017
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