[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 Julien Nabet changed: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED Assignee|libreoffice-b...@lists.free |serval2...@yahoo.fr |desktop.org | -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 --- Comment #9 from Commit Notification --- Julien Nabet committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=2a44e785da22fe08da295208c2819af813e63447 Related fdo#86929: Kill FOREACHPAM_START for good (final part) It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 --- Comment #8 from Commit Notification --- Julien Nabet committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=8286e92417794e68f6a53f887a426d5708fef0f6 Related fdo#86929: Kill FOREACHPAM_START for good (part1) It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 --- Comment #7 from Commit Notification --- Julien Nabet committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=bbf59cb82eabee984926fb11dc5281dc84e62fb6 Related fdo#86929: Kill FOREACHPAM_START for good (part2) It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 Julien Nabet changed: What|Removed |Added CC||serval2...@yahoo.fr --- Comment #6 from Julien Nabet --- I submitted for review these: - https://gerrit.libreoffice.org/#/c/13465/ - https://gerrit.libreoffice.org/#/c/13466/ but I don't know how to do for those in sw/source/core/doc/, see http://opengrok.libreoffice.org/search?q=FOREACHPAM_START&project=core&defs=&refs=&path=&hist= -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 Björn Michaelsen changed: What|Removed |Added Depends on|75757 | --- Comment #5 from Björn Michaelsen --- With some additional thinking, I found a way to get C++11 for(:) iteration right away, without using the BOOST_FOREACH hack as an intermediate solution. See: http://cgit.freedesktop.org/libreoffice/core/commit/?id=02ef3abd83f2d5f702349a8cd47928621ee5c620 for an example change. Thus also not depending on fdo#75757 anymore. This should make this Easy Hack even simpler. -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 --- Comment #4 from Commit Notification --- Bjoern Michaelsen committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=02ef3abd83f2d5f702349a8cd47928621ee5c620 fdo#86929: Kill FOREACHPAM_START for good It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 Commit Notification changed: What|Removed |Added Whiteboard|EasyHack SkillCpp |EasyHack SkillCpp |DifficultyBeginner |DifficultyBeginner |TopicCleanup|TopicCleanup target:4.5.0 -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 Björn Michaelsen changed: What|Removed |Added Depends on||75757 -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 --- Comment #3 from Björn Michaelsen --- We cant for now have sw::Ring<> implement STL-like begin()/end() functions, see: https://gerrit.libreoffice.org/gitweb?p=core.git;a=blob;f=sw/inc/ring.hxx;h=d58733f1459fda302a99d1ea9d56da82c55f9ca9;hb=2dd7cc5b925d0b4c62553eeba9f6524ce7b6217b#l105 So for now, BOOST_FOREACH should still be an improvement over the old const-eating homegrown macros. Once none of the derived classes of sw::Ring<> are deriving from any other STL-container, we should be able to trivially replace BOOST_FOREACH with a proper C++11 style for() iteration. -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 --- Comment #2 from Miklos Vajna --- FWIW, C++11 range-based for-loops are OK on libreoffice-4-4 and master, best to not use BOOST_FOREACH in new code, I guess. -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Bug 86929] Kill FOREACHPAM_START for good
https://bugs.freedesktop.org/show_bug.cgi?id=86929 Björn Michaelsen changed: What|Removed |Added CC||libreoffice@lists.freedeskt ||op.org --- Comment #1 from Björn Michaelsen --- adding LibreOffice developer list as CC to unresolved Writer EasyHacks for better visibility. see e.g. http://nabble.documentfoundation.org/minutes-of-ESC-call-td4076214.html for details -- You are receiving this mail because: You are on the CC list for the bug. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice