Apply suggestions from code review
Co-authored-by: Daira Emma Hopwood <daira@jacaranda.org>
This commit is contained in:
parent
1447d8ea01
commit
bcea060c86
|
@ -118,6 +118,7 @@ where
|
|||
/// received note must have in the blockchain in order to be considered for being
|
||||
/// spent. A value of 10 confirmations is recommended and 0-conf transactions are
|
||||
/// not supported.
|
||||
/// * `change_memo`: A memo to be included in the change output
|
||||
///
|
||||
/// # Examples
|
||||
///
|
||||
|
@ -174,7 +175,8 @@ where
|
|||
/// Amount::from_u64(1).unwrap(),
|
||||
/// None,
|
||||
/// OvkPolicy::Sender,
|
||||
/// 10
|
||||
/// 10,
|
||||
/// None
|
||||
/// )
|
||||
///
|
||||
/// # }
|
||||
|
|
|
@ -76,7 +76,7 @@ impl TransactionBalance {
|
|||
})
|
||||
}
|
||||
|
||||
/// The change values proposed by the [`ChangeStrategy`] that computed this balance.
|
||||
/// The change values proposed by the [`ChangeStrategy`] that computed this balance.
|
||||
pub fn proposed_change(&self) -> &[ChangeValue] {
|
||||
&self.proposed_change
|
||||
}
|
||||
|
|
|
@ -26,7 +26,8 @@ pub struct SingleOutputChangeStrategy {
|
|||
}
|
||||
|
||||
impl SingleOutputChangeStrategy {
|
||||
/// Constructs a new [`SingleOutputChangeStrategy`] with the specified fee rule.
|
||||
/// Constructs a new [`SingleOutputChangeStrategy`] with the specified fee rule
|
||||
/// and change memo.
|
||||
pub fn new(fee_rule: FixedFeeRule, change_memo: Option<MemoBytes>) -> Self {
|
||||
Self {
|
||||
fee_rule,
|
||||
|
|
|
@ -34,7 +34,7 @@ pub struct SingleOutputChangeStrategy {
|
|||
|
||||
impl SingleOutputChangeStrategy {
|
||||
/// Constructs a new [`SingleOutputChangeStrategy`] with the specified ZIP 317
|
||||
/// fee parameters.
|
||||
/// fee parameters and change memo.
|
||||
pub fn new(fee_rule: Zip317FeeRule, change_memo: Option<MemoBytes>) -> Self {
|
||||
Self {
|
||||
fee_rule,
|
||||
|
|
Loading…
Reference in New Issue