refactor: Simplify SendCoinsDialog::updateCoinControlState #284

1. 
   promag commented at 8:16 pm on April 18, 2021: contributor

   This PR doesn’t change behaviour, removes the coin control argument from updateCoinControlState since it’s a class member.
2. refactor, qt: Simplify SendCoinsDialog::updateCoinControlState 5f438d66c1
3. hebasto added the label Refactoring on Apr 18, 2021
4. hebasto added the label Wallet on Apr 19, 2021
5. hebasto renamed this:
   refactor, qt: Simplify SendCoinsDialog::updateCoinControlState
   refactor: Simplify SendCoinsDialog::updateCoinControlState
   on Apr 19, 2021
6. 
   hebasto commented at 4:49 pm on April 20, 2021: member

   Concept ACK on simplifying code.
7. hebasto approved
8. 
   hebasto commented at 6:11 pm on April 20, 2021: member

   ACK 5f438d66c1fbc0e524d12fef233f2ed2952e6f17, I have reviewed the code and it looks OK, I agree it can be merged.

   I see this change as a nice https://github.com/bitcoin/bitcoin/pull/18894 follow up.

9. 
   jonatack commented at 9:32 am on April 23, 2021: contributor

   Nice simplification.

   Code review ACK 5f438d66c1fbc0e524d12fef233f2ed2952e6f17

10. kristapsk approved
11. 
    kristapsk commented at 1:24 pm on April 25, 2021: contributor

    utACK 5f438d66c1fbc0e524d12fef233f2ed2952e6f17. Code looks correct.
12. hebasto merged this on Apr 25, 2021
13. hebasto closed this on Apr 25, 2021

    ---

14. sidhujag referenced this in commit de9b077ba3 on Apr 25, 2021
15. promag deleted the branch on Apr 26, 2021
16. gwillen referenced this in commit 3479bcceec on Jun 1, 2022
17. bitcoin-core locked this on Aug 16, 2022

Contributors
promag hebasto jonatack kristapsk

Labels
Refactoring Wallet