This has now been pushed. (Commit:
http://cgit.freedesktop.org/libreoffice/writer/commit/?h=libreoffice-3-4&id=746fff15ad34616635e09125f6d0656da39a05ad
)
But in general, this bug, and I guess similar ones Michael was investigating recently, brings up a
serious question: How broken might the code actually be after all the DECLARE_LIST removal Easy
Hacks completed without testing? Do we now have lots of crashers caused by misuse of iterators etc
in various seldom invoked code paths? Time will tell, I am sure.
Maybe we should ask each person who did one or several of those DECLARE_LIST removals to try to
find out how to exercise the affected code paths? Would that be a good follow-up Easy Hack? After
all, now we have pre-built code out that contains the removals, so it is easier to test when you
don't need to build the beast yourself.
--tml
Context
- [Libreoffice] DECLARE_LIST removal fallout, was: Review request: [PUSHED] http://pastebin.com/bgVxQKug · Tor Lillqvist
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.