Skip to content

Conversation

@bguiz
Copy link
Contributor

@bguiz bguiz commented May 3, 2015

  • File::open is for opening a file in read-only mode
  • File::create is for opening a file in write-only mode, which is what we want instead for this example to make sense

- `FIle::open` is for opening a file in read-only mode
- `FIle::create` is for opening a file in write-only mode, which is what we want instead for this example to make sense
@rust-highfive
Copy link
Contributor

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@bguiz
Copy link
Contributor Author

bguiz commented May 3, 2015

r? @steveklabnik Documentation only change: https://doc.rust-lang.org/book/error-handling.html

@steveklabnik
Copy link
Contributor

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented May 3, 2015

📌 Commit 75a3e29 has been approved by steveklabnik

@steveklabnik
Copy link
Contributor

Thanks!

@bors
Copy link
Collaborator

bors commented May 4, 2015

⌛ Testing commit 75a3e29 with merge b05ad9a...

@bors
Copy link
Collaborator

bors commented May 4, 2015

💔 Test failed - auto-linux-64-opt

@alexcrichton
Copy link
Member

@bors: retry

On Mon, May 4, 2015 at 4:55 AM, bors [email protected] wrote:

[image: 💔] Test failed - auto-linux-64-opt
http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/4791


Reply to this email directly or view it on GitHub
#25068 (comment).

Manishearth added a commit to Manishearth/rust that referenced this pull request May 5, 2015
- `File::open` is for opening a file in read-only mode
- `File::create` is for opening a file in write-only mode, which is what we want instead for this example to make sense
Manishearth added a commit to Manishearth/rust that referenced this pull request May 5, 2015
 - `File::open` is for opening a file in read-only mode
- `File::create` is for opening a file in write-only mode, which is what we want instead for this example to make sense
@bors bors merged commit 75a3e29 into rust-lang:master May 5, 2015
@bguiz bguiz deleted the patch-3 branch May 10, 2015 11:31
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