Fix checked arithmetic comparisons

This commit fixes the improper checking of addition in the convenience methods for the `Account` type.
jmcph4 · Apr 7, 2020 · 02afee2 · 02afee2
1 parent a57a0e4
commit 02afee2
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/account.rs b/src/account.rs
@@ -82,7 +82,7 @@ impl Account {
     pub fn add_balance(&mut self, amount: AccountBalance) ->
         Result<(), AccountError> {
         /* bounds check */
-        if self.balance.checked_add(amount) != None {
+        if self.balance.checked_add(amount).is_none() {
             return Err(AccountError::BalanceOutOfBounds);
         }
 
@@ -108,7 +108,8 @@ impl Account {
             return Err(AccountError::AssetNotFound);
         }
 
-        if self.holdings.get_mut(&ticker).unwrap().checked_add(amount) != None {
+        if self.holdings.get_mut(&ticker).unwrap().checked_add(amount).
+            is_none() {
             return Err(AccountError::HoldingOutOfBounds);
         }