scripted-diff: Remove BOOST_FOREACH, Q_FOREACH and PAIRTYPE #10502

1. 
   jtimon commented at 5:25 pm on June 1, 2017: contributor

   Separated from #10193 since the reverse iterator isn’t working there, as suggested in #10193 (comment) (but without removing #include <boost/foreach.hpp> everywhere except where BOOST_REVERSE_FOREACH is needed).

   Seems to compete with #10497

2. 
   practicalswift commented at 6:07 pm on June 1, 2017: contributor

   utACK 9eafbf2

   Excellent work! I ❤️ modern C++

3. jtimon force-pushed on Jun 1, 2017
4. 
   jtimon commented at 8:41 pm on June 1, 2017: contributor

   Added one small preparation commit since not all environments where passing travis.
5. fanquake added the label Refactoring on Jun 1, 2017
6. fanquake added the label Scripts and tools on Jun 1, 2017
7. fanquake added this to the "In progress" column in a project

   ---

8. jtimon force-pushed on Jun 2, 2017
9. 
   jtimon commented at 1:31 am on June 2, 2017: contributor

   Needed rebase after #10195. Just run the sed scripts again. Review should be as simple as it was, more or less.
10. Small preparations for Q_FOREACH, PAIRTYPE and #include <boost/foreach.hpp> removal a5410ac5ec
11. scripted-diff: Fully remove BOOST_FOREACH

    -BEGIN VERIFY SCRIPT-
    sed -i 's/BOOST_FOREACH *(\(.*\),/for (\1 :/' ./src/*.h ./src/*.cpp ./src/*/*.h ./src/*/*.cpp ./src/*/*/*.h ./src/*/*/*.cpp ;
    -END VERIFY SCRIPT-

    7c00c26726
12. scripted-diff: Remove Q_FOREACH

    -BEGIN VERIFY SCRIPT-
    sed -i 's/Q_FOREACH *(\(.*\),/for (\1 :/' ./src/*.h ./src/*.cpp ./src/*/*.h ./src/*/*.cpp ./src/*/*/*.h ./src/*/*/*.cpp ;
    -END VERIFY SCRIPT-

    18dc3c3962
13. scripted-diff: Remove PAIRTYPE

    -BEGIN VERIFY SCRIPT-
    sed -i 's/PAIRTYPE(\([^,]*\), \([^\)]*\))/std::pair<\1, \2>/' ./src/*.h ./src/*.cpp ./src/*/*.h ./src/*/*.cpp ./src/*/*/*.h ./src/*/*/*.cpp ;
    sed -i ':a;N;$!ba;s/#define std::pair<t1, t2>    std::pair<t1, t2>\n//' ./src/utilstrencodings.h ;
    -END VERIFY SCRIPT-

    1238f13cf6
14. jtimon force-pushed on Jun 5, 2017
15. 
    jtimon commented at 6:15 pm on June 5, 2017: contributor

    Needed rebase again(2)
16. 
    sipa commented at 6:20 pm on June 5, 2017: member

    Concept ACK. Nit: you can add a commit that removes the #include <boost/foreach.hpp> directives.
17. 
    jtimon commented at 7:04 pm on June 5, 2017: contributor

    you can add a commit that removes the #include <boost/foreach.hpp> directives.

    But not in the whole project yet, because I’m not able to remove BOOST_REVERSE_FOREACH yet, see #10193 on top of this one. I can remove it only on the files that I can for now.

18. 
    ryanofsky commented at 7:16 pm on June 5, 2017: member

    utACK 1238f13cf6ccf1177b66df735f360c61ae1dc20b. Seems fine to remove includes in a followup pr as long as some of them are still needed for reverse iteration.
19. 
    theuni commented at 8:01 pm on June 13, 2017: member

    nice, utACK 1238f13cf6ccf1177b66df735f360c61ae1dc20b.
20. sipa merged this on Jun 14, 2017
21. sipa closed this on Jun 14, 2017

    ---

22. sipa referenced this in commit 1ad3d4e126 on Jun 14, 2017
23. 
    sipa commented at 1:06 am on June 14, 2017: member

    utACK 1238f13cf6ccf1177b66df735f360c61ae1dc20b
24. fanquake moved this from the "In progress" to the "Done" column in a project

    ---

25. jtimon deleted the branch on Jun 14, 2017
26. PastaPastaPasta referenced this in commit c70a050b0d on Jul 5, 2019
27. PastaPastaPasta referenced this in commit 387da0a591 on Jul 5, 2019
28. PastaPastaPasta referenced this in commit bd9e9ee38a on Jul 5, 2019
29. PastaPastaPasta referenced this in commit 63800a55cc on Jul 5, 2019
30. PastaPastaPasta referenced this in commit ea1e4157d4 on Jul 5, 2019
31. PastaPastaPasta referenced this in commit 162daca146 on Jul 5, 2019
32. PastaPastaPasta referenced this in commit 56a09e8d21 on Jul 6, 2019
33. PastaPastaPasta referenced this in commit 126024f006 on Jul 6, 2019
34. PastaPastaPasta referenced this in commit e3d2723aca on Jul 6, 2019
35. PastaPastaPasta referenced this in commit a053b909e6 on Jul 8, 2019
36. PastaPastaPasta referenced this in commit 22c3c7b675 on Jul 8, 2019
37. PastaPastaPasta referenced this in commit db21a2ae09 on Jul 8, 2019
38. PastaPastaPasta referenced this in commit f4712c09ee on Jul 9, 2019
39. PastaPastaPasta referenced this in commit 770b52ad12 on Jul 9, 2019
40. PastaPastaPasta referenced this in commit 55bcf2a4a6 on Jul 9, 2019
41. PastaPastaPasta referenced this in commit a15d7405e3 on Jul 9, 2019
42. PastaPastaPasta referenced this in commit 0d52db844c on Jul 9, 2019
43. PastaPastaPasta referenced this in commit e81a28604c on Jul 9, 2019
44. barrystyle referenced this in commit b8aa4de7b4 on Jan 22, 2020
45. barrystyle referenced this in commit ddbe697693 on Jan 22, 2020
46. barrystyle referenced this in commit 9ed5a69d50 on Jan 22, 2020
47. zkbot referenced this in commit 77c2a5f810 on Dec 4, 2020
48. MarcoFalke locked this on Sep 8, 2021

Contributors
jtimon practicalswift sipa ryanofsky theuni

Labels
Refactoring Scripts and tools