File tree Expand file tree Collapse file tree 1 file changed +3
-10
lines changed Expand file tree Collapse file tree 1 file changed +3
-10
lines changed Original file line number Diff line number Diff line change @@ -1490,16 +1490,9 @@ where L::Target: Logger {
14901490 // by value and fall back to total fees paid, i.e., in case of equal values values
14911491 // we prefer lower cost paths.
14921492 cur_route. sort_unstable_by ( |a, b| {
1493- let a_value = a. get_value_msat ( ) ;
1494- let b_value = b. get_value_msat ( ) ;
1495- if a_value. cmp ( & b_value) . is_eq ( ) {
1496- let a_fees = a. get_total_fee_paid_msat ( ) ;
1497- let b_fees = b. get_total_fee_paid_msat ( ) ;
1498- // Reverse ordering here, so we drop higher-fee paths first
1499- b_fees. cmp ( & a_fees)
1500- } else {
1501- a_value. cmp ( & b_value)
1502- }
1493+ a. get_value_msat ( ) . cmp ( & b. get_value_msat ( ) )
1494+ // Reverse ordering for fees, so we drop higher-fee paths first
1495+ . then_with ( || b. get_total_fee_paid_msat ( ) . cmp ( & a. get_total_fee_paid_msat ( ) ) )
15031496 } ) ;
15041497
15051498 // We should make sure that at least 1 path left.
You can’t perform that action at this time.
0 commit comments