Address comments.

Author: tnull

lightning/src/routing/router.rs

@@ -494,10 +494,7 @@ impl<'a> PaymentPath<'a> {
 	}
 
 	fn get_path_penalty_msat(&self) -> u64 {
-		if self.hops.len() < 1 {
-			return u64::max_value();
-		}
-		self.hops.first().unwrap().0.path_penalty_msat
+		self.hops.first().map(|h| h.0.path_penalty_msat).unwrap_or(u64::max_value())
 	}
 
 	fn get_total_fee_paid_msat(&self) -> u64 {
@@ -1478,8 +1475,8 @@ where L::Target: Logger {
 
 				// First, we drop some expensive low-value paths entirely if possible, since fewer
 				// paths is better: the payment is less likely to fail. In order to do so, we sort
-				// by value and fall back to total fees paid, i.e., in case of equal values values
-				// we prefer lower cost paths.
+				// by value and fall back to total fees paid, i.e., in case of equal values we
+				// prefer lower cost paths.
 				cur_route.sort_unstable_by(|a, b| {
 					a.get_value_msat().cmp(&b.get_value_msat())
 						// Reverse ordering for fees, so we drop higher-fee paths first