C++11 supports variadic templates: https://en.wikipedia.org/wiki/Variadic_template#C.2B.2B Previously this was worked around with preprocessor macros. If handwritten, those might be hard to find, but some are using the BOOST__PP preprocessor macros and should be easily greppable: http://opengrok.libreoffice.org/search?q=BOOST_PP_REPEAT&project=core&defs=&refs=&path=&hist= git grep BOOST_PP include/ This task to remove the use of those boost macros and use plain C++11 variadic templates instead. There is hope this will also lead to less confusing compiler error messages if those classes are misused.
Daniel Robertson committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=54045cc813c79e53abe608cc1a8d35ee7177465e tdf#94228 vcl: replace BOOST_PP with templates It will be available in 5.1.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.
Daniel Robertson committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=8b00ac9b4627d8d7ed13d352ccb8932be6861d97 tdf#94228 comphelper: replace BOOST_PP It will be available in 5.1.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.
Daniel Robertson committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=6ca355d281133c1e0e54df4e4710a4e99bc38c17 tdf#94228 comphelper: replace BOOST_PP It will be available in 5.1.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.
The last commit should have removed the last use of BOOST_PP in include. I know of at least two other uses of BOOST_PP, but I'm not sure if they should be removed. I'll continue to look for cases in which handwritten macros are used, but I expect my activity on this will slow down significantly.
Migrating Whiteboard tags to Keywords: (EasyHack DifficultyInteresting SkillScript SkillCpp ) [NinjaEdit]
(In reply to Daniel L Robertson from comment #4) > The last commit should have removed the last use of BOOST_PP in include. I > know of at least two other uses of BOOST_PP, but I'm not sure if they should > be removed. I'll continue to look for cases in which handwritten macros are > used, but I expect my activity on this will slow down significantly. Hi Daniel, thanks a lot for your work here. Closing this issue for now as per above, if only to not block people from seeing more interesting easyHacks.
Remove LibreOffice Dev List from CC on EasyHacks (curtailing excessive email to list) [NinjaEdit]