Skip to content

env: remove unwrap in examples in favor of try op - #50736

Merged
bors merged 1 commit into
rust-lang:masterfrom
udoprog:env-try-op
May 17, 2018
Merged

env: remove unwrap in examples in favor of try op#50736
bors merged 1 commit into
rust-lang:masterfrom
udoprog:env-try-op

Conversation

@udoprog

Copy link
Copy Markdown
Contributor

No description provided.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @shepmaster

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

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 14, 2018
@shepmaster

shepmaster commented May 15, 2018

Copy link
Copy Markdown
Member

I think I'm generally in favor of this, but I'm hesitant to accept it as-is because doctests will be getting the ability to use ? as well. Exposing the main here is thus annoying visual churn.

I could see if we used the # to hide the main:

+/// # fn main() -> std::io::Result<()> {+/// let path = env::current_dir()?;+/// println!("The current directory is {}", path.display());+/// # Ok(())+/// # }

But I wonder if hiding the Ok is too much.

@udoprog

Copy link
Copy Markdown
ContributorAuthor

@shepmaster this is only documentation, we can patch it again when try-op is available for doctests.

This change puts it in line with other similar doctests (e.g. https://doc.rust-lang.org/std/io/trait.Read.html) which will also have to be patched later.

@shepmaster

Copy link
Copy Markdown
Member

Ah, alright then.

@bors r+

@bors

bors commented May 16, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit f73c4a4 has been approved by shepmaster

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 16, 2018
@shepmaster

Copy link
Copy Markdown
Member

@bors rollup

@shepmaster

Copy link
Copy Markdown
Member

Thanks!

@kennytmkennytm mentioned this pull request May 16, 2018
kennytm added a commit to kennytm/rust that referenced this pull request May 16, 2018
env: remove unwrap in examples in favor of try op
bors added a commit that referenced this pull request May 17, 2018
Rollup of 17 pull requests
Successful merges:
- #50170 (Implement From for more types on Cow)
- #50638 (Don't unconditionally set CLOEXEC twice on every fd we open on Linux)
- #50656 (Fix `fn main() -> impl Trait` for non-`Termination` trait)
- #50669 (rustdoc: deprecate `#![doc(passes, plugins, no_default_passes)]`)
- #50726 (read2: Use inner function instead of closure)
- #50728 (Fix rustdoc panic with `impl Trait` in type parameters)
- #50736 (env: remove unwrap in examples in favor of try op)
- #50740 (Remove LazyBTreeMap.)
- #50752 (Add missing error codes in libsyntax-ext asm)
- #50779 (Make mutable_noalias and arg_align_attributes be tracked)
- #50787 (Fix run-make wasm tests)
- #50788 (Fix an ICE when casting a nonexistent const)
- #50789 (Ensure libraries built in stage0 have unique metadata)
- #50793 (tidy: Add a check for empty UI test files)
- #50797 (fix a typo in signed-integer::from_str_radix())
- #50808 (Stabilize num::NonZeroU*)
- #50809 (GitHub: Stop treating Cargo.lock as a generated file.)
Failed merges:
@bors
bors merged commit f73c4a4 into rust-lang:masterMay 17, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@udoprog@rust-highfive@shepmaster@bors