[Qt] minor optimisations in peertablemodel #6317

1. 
   Diapolo commented at 11:11 am on June 21, 2015: none

   • remove an unneeded include of net.h in peertablemodel.cpp
   • add const after size() in PeerTablePriv
   • remove 2x unneeded else in functions
   • replace a (int) typecast by (QVariant) to use Qt style
2. 
   paveljanik commented at 5:27 am on June 22, 2015: contributor

   Can you please remove { and } in a one-line ifs? utACK
3. 
   Diapolo commented at 2:21 pm on June 22, 2015: none

   @paveljanik @laanwj most of the time prefers { and } even for one-line ifs :).
4. 
   jonasschnelli commented at 3:42 pm on June 23, 2015: contributor

   I agree with @paveljanik: most one line ifs are without brackets and i think this is according to our code style / clang style.
5. 
   Diapolo commented at 6:58 pm on June 23, 2015: none

   @paveljanik @jonasschnelli Changed into one-line ifs :).
6. laanwj added the label GUI on Jul 2, 2015
7. 
   Diapolo commented at 6:17 am on July 6, 2015: none

   @paveljanik @jonasschnelli @laanwj Do I get some ACKs, this should be easy to merge :).
8. 
   paveljanik commented at 7:23 am on July 6, 2015: contributor

   utACK I think such changes should go to trivial…
9. 
   jonasschnelli commented at 7:26 am on July 6, 2015: contributor

   utACK
10. 
    jonasschnelli commented at 7:24 pm on July 14, 2015: contributor

    ACK
11. 
    Diapolo commented at 9:25 am on July 29, 2015: none

    @laanwj Can you please merge this trivial stuff :)?
12. [Qt] minor optimisations in peertablemodel

    - remove an unneeded include of net.h in peertablemodel.cpp
    - add const after size() in PeerTablePriv
    - remove 2x unneeded else in functions
    - replace a (int) typecast by (QVariant) to use Qt style
    - remove unneeded include of peertablemodel.h from rpcconsole.cpp

    92684bb872
13. 
    Diapolo commented at 12:13 pm on August 10, 2015: none

    @laanwj Ping once more.
14. 
    Diapolo commented at 7:57 pm on September 9, 2015: none

    @jonasschnelli IMHO, if we still support the GUI, we need another person wo is willing or able to merge GUI pulls… it’s currently frustrating!
15. 
    MarcoFalke commented at 12:54 pm on September 10, 2015: member

    merge GUI pulls

    It seems cosmetic-only changes to the source code usually get rejected? (#6511)

    Still, utACK from me.

16. jgarzik merged this on Sep 16, 2015
17. jgarzik closed this on Sep 16, 2015

    ---

18. jgarzik referenced this in commit ca5e2a1864 on Sep 16, 2015
19. Diapolo deleted the branch on Sep 16, 2015
20. str4d referenced this in commit 2e4ce09eba on Jan 31, 2017
21. str4d referenced this in commit 1afb84314b on Feb 8, 2017
22. DrahtBot locked this on Sep 8, 2021

Contributors
Diapolo paveljanik jonasschnelli MarcoFalke

Labels
GUI