Since C++11 non-member std::begin() and std::end() are available.
It would be nice:
- to have these being able to handle UNO Sequences too
- and then remove ::comphelper::stl_begin / ::comphelper::stl_end and use the std::begin() std::end() instead
It's worth checking that we can in fact use C++11 in LibreOffice. I currently see zero hits for std::begin() in the code, so this seems a tad risky to me; Bjoern ?
(In reply to comment #0)
> Since C++11 non-member std::begin() and std::end() are available.
But we cannot rely on them being available in our code base, which still needs to compile with non-C++11 compilers. And breaking contracts by hacking nonmember begin/end functions into those compilers' std namespaces in order to get rid of comphelper::stl_begin/end is arguably not worth the trouble.
wontfix for now then.
We could use boost::begin() and boost::end().
indeed, that gives a nice migration-path to C++11 and is still standard compliant.
adding LibreOffice developer list as CC to unresolved EasyHacks for better visibility.
see e.g. http://nabble.documentfoundation.org/minutes-of-ESC-call-td4076214.html for details
Marcos Paulo de Souza committed a patch related to this issue.
It has been pushed to "master":
fdo#63020: Replace ::comphelper::stl_begin()...
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:
Affected users are encouraged to test the fix and report feedback.
Migrating Whiteboard tags to Keywords: (EasyHack DifficultyBeginner SkillCpp )
Remove LibreOffice Dev List from CC on EasyHacks
(curtailing excessive email to list)