[Libreoffice] [PATCH 01/11] Replace ScChangeTrackMsgStack with std::stack< ScChangeTrackMsgInfo* >

Marcel Metz mmetz at adrian-broher.net
Thu Dec 1 02:08:40 PST 2011


Hello libreoffice devs,

this is my first patch series to the lo project. It replaces the use
of the tools Stack class and corresponding macros with the std::stack
as suggested in in bug #39445 [1] (an easy hack).

I would like to continue with cleaning up the tools module. I've seen
that there are other containers in tools like a Table which probably
could be replaced in most places with a std::table or a std::vector and
the List that could be replaced with std::vector. Should I set up a new
Bug for that or can I just hack on if there aren't any objections on
these changes?

[1] https://bugs.freedesktop.org/show_bug.cgi?id=39445#c1
---
 sc/inc/chgtrack.hxx                        |   10 +---
 sc/source/core/tool/chgtrack.cxx           |   69
+++++++++++++++------------
 sc/source/filter/inc/XclExpChangeTrack.hxx |    1 +
 3 files changed, 42 insertions(+), 38 deletions(-)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Replace-ScChangeTrackMsgStack-with-std-stack-ScChang.patch
Type: text/x-patch
Size: 10472 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20111201/9de8cd86/attachment-0001.bin>


More information about the LibreOffice mailing list