qt, refactor: Remove never used default parameter #17943

1. 
   hebasto commented at 6:52 pm on January 16, 2020: member

   In BitcoinGUI::message() slot the bool* ret = nullptr parameter is never used.

   This PR removes it and simplifies connections syntax by replacing lambdas with the &BitcoinGUI::message slot.

2. refactor: Remove never used default parameter 7d0a8f4f53
3. refactor: Simplify connection syntax 1a53b0da60
4. 
   promag commented at 7:22 pm on January 16, 2020: member

   Concept ACK, when was the last usage dropped?
5. DrahtBot added the label GUI on Jan 16, 2020
6. 
   DrahtBot commented at 8:59 pm on January 16, 2020: member

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Conflicts

   Reviewers, this pull request conflicts with the following ones:

   • #16224 (gui: Bilingual GUI error messages by hebasto)

   If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

7. fanquake added the label Refactoring on Jan 16, 2020
8. 
   Empact commented at 10:06 pm on January 16, 2020: member

   The use was introduced in f7f3a96b74bb795d6e184a628adce21c744d234f and I believe removed in 35ecf854c084c248ad640c6af030a9d1ed726c47 but haven’t built the old code to confirm.
9. 
   Empact commented at 10:09 pm on January 16, 2020: member

   Code review ACK https://github.com/bitcoin/bitcoin/pull/17943/commits/1a53b0da60097cd7fd423c519f01ceca0fd0aa14
10. 
    promag commented at 1:20 am on January 17, 2020: member

    Code review ACK 1a53b0da60097cd7fd423c519f01ceca0fd0aa14.
11. 
    Sjors commented at 9:21 am on January 17, 2020: member

    Tested ACK 1a53b0da60097cd7fd423c519f01ceca0fd0aa14

    You can see messages in action by e.g. feeding a malformed BIP20 URI: src/qt/bitcoin-qt -regtest bitcoin://bcrt1q

12. fanquake referenced this in commit 0deba68064 on Jan 17, 2020
13. fanquake merged this on Jan 17, 2020
14. fanquake closed this on Jan 17, 2020

    ---

15. hebasto deleted the branch on Jan 18, 2020
16. 
    hebasto commented at 6:36 pm on January 18, 2020: member

    Sorry, It seems I broke ThreadSafeMessageBox() ((

    bool* ret actually is used here: https://github.com/bitcoin/bitcoin/blob/a654626f076a72416a3d354218d7107571d6caaf/src/qt/bitcoingui.cpp#L1368

    How it could be reverted?

    Ref: #16348

17. 
    promag commented at 7:35 pm on January 18, 2020: member

    Please don’t revert, I think the code you referenced is broken.
18. 
    hebasto commented at 6:09 am on January 19, 2020: member

    @fanquake (from IRC):

    It’s unclear from your comments what is supposedly broken or not.

    0$ ./src/qt/bitcoin-qt 
    1: You need to rebuild the database using -reindex to go back to unpruned mode.  This will redownload the entire blockchain.
    2Please restart with -reindex or -reindex-chainstate to recover.
    3bitcoin-qt: qt/bitcoingui.cpp:1369: bool ThreadSafeMessageBox(BitcoinGUI*, const string&, const string&, unsigned int): Assertion `invoked' failed.
    4Aborted (core dumped)
    

19. laanwj referenced this in commit 0038e536de on Jan 22, 2020
20. PastaPastaPasta referenced this in commit 10185acfe8 on Dec 22, 2021
21. PastaPastaPasta referenced this in commit 8af1abab0e on Dec 22, 2021
22. PastaPastaPasta referenced this in commit 0a10713bb3 on Dec 22, 2021
23. PastaPastaPasta referenced this in commit 70c48b9ca0 on Dec 22, 2021
24. PastaPastaPasta referenced this in commit b6147764d0 on Dec 22, 2021
25. PastaPastaPasta referenced this in commit 3e28512f96 on Dec 22, 2021
26. PastaPastaPasta referenced this in commit 5dbec5163e on Dec 28, 2021
27. PastaPastaPasta referenced this in commit a60f648ed7 on Dec 28, 2021
28. DrahtBot locked this on Feb 15, 2022

Contributors
hebasto promag DrahtBot Empact Sjors

Labels
GUI Refactoring