To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=108345
------- Additional comments from [email protected] Tue Apr 27 08:48:38 +0000 2010 ------- @dtardon: > Yeah, it does, as all its parameters are passed by value ... and I knew it. No > excuse for me! The right thing here would have been to pass the node set > through > boost::ref, i.e. > std::for_each(aProcessedIds.begin(), aProcessedIds.end(), > boost::bind(&SfxItemSet::ClearItem, boost::ref(aNdSet), _1)); well, then boost::bind follows the usual C++ design principles: by default, do something useless that no one actually wants, and then provide an option to do the right thing that has to be specified explicitly. thus creating another pitfall for the unwary that don't use it ten times a day. anyway, thanks for writing the patch :) --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
