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

pull jtimon wants to merge 4 commits into bitcoin:master from jtimon:b15-boost-foreach changing 63 files +296 −300
  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

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-22 12:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me