Skip to content

A small section of futures in the tutorial on tasks and correction to the doc in future.rs #6537

New issue

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

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

Already on GitHub? # to your account

Closed
wants to merge 10 commits into from

Conversation

osaut
Copy link
Contributor

@osaut osaut commented May 16, 2013

  • The example given in future.rs was corrected.
  • I have added a small section describing futures in tutorial on tasks. It is far from being complete as I am stil learning !

@osaut
Copy link
Contributor Author

osaut commented May 16, 2013

Many thanks to dbaupp !

fn partial_sum(start: uint) -> f64 {
let mut local_sum = 0f64;
for uint::range(start*100000, (start+1)*100000) |num| {
local_sum += (num as f64 + 1).pow(-2.0);
Copy link
Member

Choose a reason for hiding this comment

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

I made a mistake on the gist btw, the + 1 here needs to be + 1.0.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My bad, I should have double-checked. I have just made a fix for it.

@osaut
Copy link
Contributor Author

osaut commented May 16, 2013

AFAIK the failures are not related to this PR (which only changes docs).

@osaut osaut closed this May 17, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 4, 2021
[beta] Backport of rust-lang#6375 - field_reassign_with_default fix

With the pinned nightly we can test backports to our beta branch now 🎉

cc rust-lang#6515

changelog: beta 1.50: Backport of private fields fix in [`field_reassign_with_default`] lints
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants