Apply suggestions from code review

Co-authored-by: Daira Emma Hopwood <[email protected]>
zcash · Sep 1, 2023 · 727cddd · 727cddd
1 parent 363dea9
commit 727cddd
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/zcash_client_backend/src/data_api.rs b/zcash_client_backend/src/data_api.rs
@@ -572,6 +572,12 @@ impl AccountBirthday {
     }
 
     #[cfg(feature = "test-dependencies")]
+    /// Constructs a new [`AccountBirthday`] at Sapling activation, with no
+    /// "recover until" height.
+    ///
+    /// # Panics
+    ///
+    /// Panics if the Sapling activation height is not set.
     pub fn from_sapling_activation<P: zcash_primitives::consensus::Parameters>(
         params: &P,
     ) -> AccountBirthday {
@@ -612,9 +618,9 @@ pub trait WalletWrite: WalletRead {
     /// wallet state, you should use this method to add all of the desired accounts before scanning
     /// the chain from the seed's birthday height.
     ///
-    /// By convention, wallets should only allow a new account to be generated after funds have
-    /// been received by the currently-available account (in order to enable automated account
-    /// recovery).
+    /// By convention, wallets should only allow a new account to be generated after confirmed
+    /// funds have been received by the currently-available account (in order to enable automated
+    /// account recovery).
     ///
     /// [ZIP 316]: https://zips.z.cash/zip-0316
     fn create_account(