wallet: Remove redundant lambda function arg in handleTransactionChanged #12932

pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2018_04_redundant_argument changing 1 files +1 −1

laanwj commented at 1:25 PM on April 10, 2018: member

Makes the build warning-clean again here:

bitcoin/src/interfaces/wallet.cpp:425:18: warning: lambda capture 'this' is not used [-Wunused-lambda-capture]
            [fn, this](CWallet*, const uint256& txid, ChangeType status) { fn(txid, status); }));
                 ^

wallet: Remove redundant lambda function arg in handleTransactionChanged

Makes the build warning-clean again here:

    bitcoin/src/interfaces/wallet.cpp:425:18: warning: lambda capture 'this' is not used [-Wunused-lambda-capture]
                [fn, this](CWallet*, const uint256& txid, ChangeType status) { fn(txid, status); }));
                     ^

9b9d717263

laanwj added the label Refactoring on Apr 10, 2018
laanwj added the label Wallet on Apr 10, 2018
promag commented at 1:30 PM on April 10, 2018: member

utACK 9b9d717.
practicalswift commented at 1:52 PM on April 10, 2018: contributor

utACK 9b9d7172636fa8b907cc595b194a496433fdd609
ryanofsky commented at 2:07 PM on April 10, 2018: member

utACK 9b9d7172636fa8b907cc595b194a496433fdd609, sorry for warning noise!
MarcoFalke merged this on Apr 10, 2018
MarcoFalke closed this on Apr 10, 2018
MarcoFalke referenced this in commit 8d651ae320 on Apr 10, 2018
PastaPastaPasta referenced this in commit ee4effdc19 on Nov 1, 2020
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels