Make pczt::Bundle::extract take self by reference.

CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to Rust's notion of
 
 ## [Unreleased]
 
+### Changed
+- `orchard::pczt::Bundle::extract` now takes its `self` argument by
+  reference instead of by value.
+
 ## [0.12.0] - 2025-12-05
 
 ### Added

src/circuit.rs

@@ -1115,7 +1115,7 @@ mod tests {
                 let mut rng = OsRng;
 
                 let (circuit, instance) = generate_circuit_instance(OsRng);
-                let instances = &[instance.clone()];
+                let instances = core::slice::from_ref(&instance);
 
                 let pk = ProvingKey::build();
                 let proof = Proof::create(&pk, &[circuit], instances, &mut rng).unwrap();

src/pczt/tx_extractor.rs

@@ -28,7 +28,7 @@ impl super::Bundle {
     ///
     /// [regular `Bundle`]: crate::Bundle
     pub fn extract<V: TryFrom<i64>>(
-        self,
+        &self,
     ) -> Result<Option<crate::Bundle<Unbound, V>>, TxExtractorError> {
         self.to_tx_data(
             |action| {

src/value.rs

@@ -6,8 +6,8 @@
 //! - [`ValueSum`], the sum of note values within an Orchard [`Action`] or [`Bundle`].
 //!   It is a signed 64-bit integer (with range [`VALUE_SUM_RANGE`]).
 //! - `valueBalanceOrchard`, which is a signed 63-bit integer. This is represented
-//!    by a user-defined type parameter on [`Bundle`], returned by
-//!    [`Bundle::value_balance`] and [`Builder::value_balance`].
+//!   by a user-defined type parameter on [`Bundle`], returned by
+//!   [`Bundle::value_balance`] and [`Builder::value_balance`].
 //!
 //! If your specific instantiation of the Orchard protocol requires a smaller bound on
 //! valid note values (for example, Zcash's `MAX_MONEY` fits into a 51-bit integer), you