From e9059cb8aaad3515e174ffa72783e36e34b95dd1 Mon Sep 17 00:00:00 2001 From: Cameron Steffen Date: Thu, 8 Feb 2024 16:41:48 -0600 Subject: [PATCH] Improve Option::inspect docs --- library/core/src/option.rs | 15 ++++++++++----- library/core/src/result.rs | 8 ++++++-- 2 files changed, 16 insertions(+), 7 deletions(-) diff --git a/library/core/src/option.rs b/library/core/src/option.rs index ab005228984e0..e37e62e32c44a 100644 --- a/library/core/src/option.rs +++ b/library/core/src/option.rs @@ -1073,18 +1073,23 @@ impl Option { } } - /// Calls the provided closure with a reference to the contained value (if [`Some`]). + /// Calls a function with a reference to the contained value if [`Some`]. + /// + /// Returns the original option. /// /// # Examples /// /// ``` - /// let v = vec![1, 2, 3, 4, 5]; + /// let list = vec![1, 2, 3]; /// - /// // prints "got: 4" - /// let x: Option<&usize> = v.get(3).inspect(|x| println!("got: {x}")); + /// // prints "got: 2" + /// let x = list + /// .get(1) + /// .inspect(|x| println!("got: {x}")) + /// .expect("list should be long enough"); /// /// // prints nothing - /// let x: Option<&usize> = v.get(5).inspect(|x| println!("got: {x}")); + /// list.get(5).inspect(|x| println!("got: {x}")); /// ``` #[inline] #[stable(feature = "result_option_inspect", since = "1.76.0")] diff --git a/library/core/src/result.rs b/library/core/src/result.rs index 1f448984e531c..80f7fe75e8230 100644 --- a/library/core/src/result.rs +++ b/library/core/src/result.rs @@ -830,7 +830,9 @@ impl Result { } } - /// Calls the provided closure with a reference to the contained value (if [`Ok`]). + /// Calls a function with a reference to the contained value if [`Ok`]. + /// + /// Returns the original result. /// /// # Examples /// @@ -851,7 +853,9 @@ impl Result { self } - /// Calls the provided closure with a reference to the contained error (if [`Err`]). + /// Calls a function with a reference to the contained value if [`Err`]. + /// + /// Returns the original result. /// /// # Examples ///