From fdb28267387593974f4157dcc9558c8757805230 Mon Sep 17 00:00:00 2001 From: Eric Findlay Date: Wed, 28 Oct 2015 10:39:22 +0900 Subject: Added try! example to documentation.md --- src/doc/trpl/documentation.md | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) (limited to 'src') diff --git a/src/doc/trpl/documentation.md b/src/doc/trpl/documentation.md index e101d4bc0d4..e130109e354 100644 --- a/src/doc/trpl/documentation.md +++ b/src/doc/trpl/documentation.md @@ -373,6 +373,36 @@ we can add the `#[macro_use]` attribute. Second, we’ll need to add our own `main()` as well. Finally, a judicious use of `#` to comment out those two things, so they don’t show up in the output. +Another case where the use of `#` is handy is when you want to ignore +error handling. Lets say you want the following, + +```rust +/// use std::io; +/// let mut input = String::new(); +/// try!(io::stdin().read_line(&mut input)); +``` + +The problem is that `try!` returns a `Result` and test functions +don't return anything so this will give a mismatched types error. + +```rust +/// A doc test using try! +/// +/// ``` +/// use std::io; +/// # fn f() -> io::Result<()> { +/// let mut input = String::new(); +/// try!(io::stdin().read_line(&mut input)); +/// # Ok(()) +/// # } +/// # f(); +/// ``` +``` + +You can get around this by wrapping the code in a function. This catches +and swallows the `Result` when running tests on the docs. This +pattern appears regularly in the standard library. + ### Running documentation tests To run the tests, either: -- cgit 1.4.1-3-g733a5 From b2c3745229e8a0472f2a4ebede7496572e6b6d0e Mon Sep 17 00:00:00 2001 From: Eric Findlay Date: Thu, 29 Oct 2015 09:43:54 +0900 Subject: Added "ignore" to rustdoc example, r=steveklabnik Fixes #29234 --- src/doc/trpl/documentation.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/doc/trpl/documentation.md b/src/doc/trpl/documentation.md index e130109e354..191a51dffdb 100644 --- a/src/doc/trpl/documentation.md +++ b/src/doc/trpl/documentation.md @@ -376,7 +376,7 @@ things, so they don’t show up in the output. Another case where the use of `#` is handy is when you want to ignore error handling. Lets say you want the following, -```rust +```rust,ignore /// use std::io; /// let mut input = String::new(); /// try!(io::stdin().read_line(&mut input)); -- cgit 1.4.1-3-g733a5 From 46b30ccd895d6fc990474cb6f130b41d0a9b8dcf Mon Sep 17 00:00:00 2001 From: Eric Findlay Date: Sun, 8 Nov 2015 10:03:58 +0900 Subject: Added foo() to rustdoc example, r=steveklabnik Fixes #29234 --- src/doc/trpl/documentation.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/doc/trpl/documentation.md b/src/doc/trpl/documentation.md index 191a51dffdb..b3f79f2f1df 100644 --- a/src/doc/trpl/documentation.md +++ b/src/doc/trpl/documentation.md @@ -385,17 +385,17 @@ error handling. Lets say you want the following, The problem is that `try!` returns a `Result` and test functions don't return anything so this will give a mismatched types error. -```rust +```rust,ignore /// A doc test using try! /// /// ``` /// use std::io; -/// # fn f() -> io::Result<()> { +/// # fn foo() -> io::Result<()> { /// let mut input = String::new(); /// try!(io::stdin().read_line(&mut input)); /// # Ok(()) /// # } -/// # f(); +/// # foo(); /// ``` ``` -- cgit 1.4.1-3-g733a5 From dda7a3c2a195b7d99f4173f3636a8a7d25ca8c0c Mon Sep 17 00:00:00 2001 From: Eric Findlay Date: Sun, 8 Nov 2015 11:00:03 +0900 Subject: Fixed "foo()" in try! example, r=steveklabnik --- src/doc/trpl/documentation.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/doc/trpl/documentation.md b/src/doc/trpl/documentation.md index b3f79f2f1df..dc91c90b0fd 100644 --- a/src/doc/trpl/documentation.md +++ b/src/doc/trpl/documentation.md @@ -395,8 +395,8 @@ don't return anything so this will give a mismatched types error. /// try!(io::stdin().read_line(&mut input)); /// # Ok(()) /// # } -/// # foo(); /// ``` +# fn foo() {} ``` You can get around this by wrapping the code in a function. This catches -- cgit 1.4.1-3-g733a5 From e3433e3b519dfaddedd5f18a514604a779799da3 Mon Sep 17 00:00:00 2001 From: Stepan Koltsov Date: Sun, 8 Nov 2015 17:17:02 +0300 Subject: Fix outdated comment in Vec::from_iter Since commit 46068c9da, call to `reserve()` on empty vec allocates exactly requested capacity, so unroll of first iteration may help only with branch prediction. --- src/libcollections/vec.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'src') diff --git a/src/libcollections/vec.rs b/src/libcollections/vec.rs index 716a07be2b4..c4e4059429f 100644 --- a/src/libcollections/vec.rs +++ b/src/libcollections/vec.rs @@ -1220,8 +1220,7 @@ impl FromIterator for Vec { // expanded on this iteration in every case when the iterable is not // empty, but the loop in extend_desugared() is not going to see the // vector being full in the few subsequent loop iterations. - // So we get better branch prediction and the possibility to - // construct the vector with initial estimated capacity. + // So we get better branch prediction. let mut iterator = iterable.into_iter(); let mut vector = match iterator.next() { None => return Vec::new(), -- cgit 1.4.1-3-g733a5 From 621e259b78fb992745c3cf194e28be5a8610a97c Mon Sep 17 00:00:00 2001 From: Kevin Butler Date: Mon, 9 Nov 2015 03:42:22 +0000 Subject: libstd: add example for PathBuf::push --- src/libstd/path.rs | 15 +++++++++++++++ 1 file changed, 15 insertions(+) (limited to 'src') diff --git a/src/libstd/path.rs b/src/libstd/path.rs index b9a58a11764..6b5e16ae113 100644 --- a/src/libstd/path.rs +++ b/src/libstd/path.rs @@ -1013,6 +1013,21 @@ impl PathBuf { /// * if `path` has a root but no prefix (e.g. `\windows`), it /// replaces everything except for the prefix (if any) of `self`. /// * if `path` has a prefix but no root, it replaces `self`. + /// + /// # Examples + /// + /// ``` + /// use std::path::PathBuf; + /// + /// let mut path = PathBuf::new(); + /// path.push("/tmp"); + /// path.push("file.bk"); + /// assert_eq!(path, PathBuf::from("/tmp/file.bk")); + /// + /// // Pushing an absolute path replaces the current path + /// path.push("/etc/passwd"); + /// assert_eq!(path, PathBuf::from("/etc/passwd")); + /// ``` #[stable(feature = "rust1", since = "1.0.0")] pub fn push>(&mut self, path: P) { self._push(path.as_ref()) -- cgit 1.4.1-3-g733a5 From c618c5f36a3260351a09f4b4dc51b2e5d1359fbc Mon Sep 17 00:00:00 2001 From: Ivan Ivaschenko Date: Mon, 9 Nov 2015 17:49:30 +0200 Subject: Doc: Fix broken link on for-loop.html --- src/doc/trpl/dining-philosophers.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/doc/trpl/dining-philosophers.md b/src/doc/trpl/dining-philosophers.md index 5f66a5b9e29..50d758c3a10 100644 --- a/src/doc/trpl/dining-philosophers.md +++ b/src/doc/trpl/dining-philosophers.md @@ -232,7 +232,7 @@ also called a ‘vector’, and it’s a growable array type. We then use a [`for`][for] loop to iterate through the vector, getting a reference to each philosopher in turn. -[for]: for-loops.html +[for]: loops.html#for In the body of the loop, we call `p.eat()`, which is defined above: -- cgit 1.4.1-3-g733a5 From abb9c9008fbb253f82dfc0e7c72dcd107e820fb7 Mon Sep 17 00:00:00 2001 From: Steve Klabnik Date: Tue, 10 Nov 2015 01:39:23 +0100 Subject: Some cleanup on after #29684 * wrap to 80 cols * small grammar fix, missing 'the' --- src/libcore/iter.rs | 27 +++++++++++++-------------- 1 file changed, 13 insertions(+), 14 deletions(-) (limited to 'src') diff --git a/src/libcore/iter.rs b/src/libcore/iter.rs index da7d673cd96..f8c6e3cfdd7 100644 --- a/src/libcore/iter.rs +++ b/src/libcore/iter.rs @@ -371,22 +371,21 @@ pub trait Iterator { /// /// # Implementation notes /// - /// It is not enforced that an iterator implementation yields the - /// declared number of elements. A buggy iterator may yield less - /// than the lower bound or more than the upper bound of elements. + /// It is not enforced that an iterator implementation yields the declared + /// number of elements. A buggy iterator may yield less than the lower bound + /// or more than the upper bound of elements. /// - /// `size_hint()` is primarily intended to be used for optimizations - /// such as reserving space for the elements of the iterator, but - /// must not be trusted to e.g. omit bounds checks in unsafe code. - /// An incorrect implementation of `size_hint()` should not lead to - /// memory safety violations. + /// `size_hint()` is primarily intended to be used for optimizations such as + /// reserving space for the elements of the iterator, but must not be + /// trusted to e.g. omit bounds checks in unsafe code. An incorrect + /// implementation of `size_hint()` should not lead to memory safety + /// violations. /// - /// That said, the implementation should provide a correct - /// estimation, because otherwise it would be a violation of the - /// trait's protocol. + /// That said, the implementation should provide a correct estimation, + /// because otherwise it would be a violation of the trait's protocol. /// - /// The default implementation returns `(0, None)` which is correct - /// for any iterator. + /// The default implementation returns `(0, None)` which is correct for any + /// iterator. /// /// # Examples /// @@ -2750,7 +2749,7 @@ pub trait ExactSizeIterator: Iterator { /// implementation, you can do so. See the [trait-level] docs for an /// example. /// - /// This function has the same safety guarantees as [`size_hint()`] + /// This function has the same safety guarantees as the [`size_hint()`] /// function. /// /// [trait-level]: trait.ExactSizeIterator.html -- cgit 1.4.1-3-g733a5