Merge pull request #1561

d561f4ad enable clang checks that were disabled (Chris Vickio)
0aefb2f6 remove std::move from return statements (pessimizing-move warning) (Chris Vickio)
629d5b76 change counter from bool to int (deprecated-increment-bool warning) (Chris Vickio)
fb76d439 add extra braces around subobjects (missing-braces warning) (Chris Vickio)
3b6d5f25 make struct/class declarations consistent (mismatched-tags warning) (Chris Vickio)
fcf66925 remove unused fields from network_throttle (unused-private-field warning) (Chris Vickio)
296f8c16 inline unused function (for unused-function warning) (Chris Vickio)

src/wallet/api/address_book.h

@@ -32,8 +32,7 @@
 #include "wallet/wallet2.h"
 
 namespace Monero {
-    
-class AddressBookRow;
+
 class WalletImpl;
 
 class AddressBookImpl : public AddressBook

src/wallet/api/transaction_history.h

@@ -33,7 +33,6 @@
 
 namespace Monero {
 
-class TransactionInfo;
 class WalletImpl;
 
 class TransactionHistoryImpl : public TransactionHistory

src/wallet/api/wallet.h

@@ -134,7 +134,7 @@ private:
     friend class PendingTransactionImpl;
     friend class UnsignedTransactionImpl;    
     friend class TransactionHistoryImpl;
-    friend class Wallet2CallbackImpl;
+    friend struct Wallet2CallbackImpl;
     friend class AddressBookImpl;
 
     tools::wallet2 * m_wallet;