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

Added mapAll which will map/recover in a single PF #3681

Closed
wants to merge 1 commit into from
Closed

Added mapAll which will map/recover in a single PF #3681

wants to merge 1 commit into from

Conversation

sksamuel
Copy link

No description provided.

@retronym retronym modified the milestones: 2.11.1-RC1, 2.11.0-RC5 Apr 15, 2014
@sksamuel
Copy link
Author

PLS REBUILD/pr-scala@25f703d

@scala-jenkins
Copy link

(kitty-note-to-self: ignore 41650814)
🐱 Roger! Rebuilding pr-scala for 25f703d. 🚨

@retronym
Copy link
Member

retronym commented May 8, 2014

Could you please coordinate this proposed change with the in-progress discussions to extend the Futures API

https://groups.google.com/forum/#!topic/scala-sips/4LIO5NpXYaE
https://docs.google.com/document/d/1Vza2-B3FzgtleX8RNs6rqRbhOn9jotrUfFB77h0DIGU/edit

See also:

https://groups.google.com/forum/#!topic/scala-sips/qi9X_E0FiEg

ping @viktorklang @richdougherty who might be more aware of that status of those discussions than me.

I will close this PR for now, in the interests of batching the API enhancements to Futures up a little bit to end up with a more coherent design.

@retronym retronym closed this May 8, 2014
@Ichoran
Copy link
Contributor

Ichoran commented May 8, 2014

I don't think the name is advisable, given that a partial function is perfectly free to map far less than "all". It could cover less than either recover or map do.

@sksamuel
Copy link
Author

sksamuel commented May 8, 2014

That makes sense. We a new name then :)
On 8 May 2014 22:19, "Ichoran" notifications@github.com wrote:

I don't think the name is advisable, given that a partial function is
perfectly free to map far less than "all". It could cover less than either
recover or map do.


Reply to this email directly or view it on GitHubhttps://github.com//pull/3681#issuecomment-42607614
.

@viktorklang
Copy link
Contributor

See my work-in-progress for a SIP-14 update here: https://github.com/viktorklang/scala/pull/6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants