[refactor] Inline last case of wtx.IsFromMe and remove function

morcos · morcos · commit acbbc052ce87 · 2017-02-14T13:09:45.000-05:00
No change in behavior.
diff --git a/src/wallet/rpcwallet.cpp b/src/wallet/rpcwallet.cpp
@@ -1358,7 +1358,7 @@ void ListTransactions(const CWalletTx& wtx, const string& strAccount, int nMinDe
     wtx.GetAmounts(listReceived, listSent, nFee, strSentAccount, filter);
 
     bool fAllAccounts = (strAccount == string("*"));
-    bool involvesWatchonly = wtx.IsFromMe(ISMINE_WATCH_ONLY);
+    bool involvesWatchonly = (wtx.GetDebit(ISMINE_WATCH_ONLY) > 0);
 
     // Sent
     if ((!listSent.empty() || nFee != 0) && (fAllAccounts || strAccount == strSentAccount))
diff --git a/src/wallet/wallet.h b/src/wallet/wallet.h
@@ -421,11 +421,6 @@ class CWalletTx : public CMerkleTx
     void GetAccountAmounts(const std::string& strAccount, CAmount& nReceived,
                            CAmount& nSent, CAmount& nFee, const isminefilter& filter) const;
 
-    bool IsFromMe(const isminefilter& filter) const
-    {
-        return (GetDebit(filter) > 0);
-    }
-
     // True if only scriptSigs are different
     bool IsEquivalentTo(const CWalletTx& tx) const;
 
