Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


On 09/21/2012 08:12 AM, Markus Mohrhard wrote:
Hey,

I have several patches that are needed to fix bugs in conditional
formats. One of them is a MAB/regression with several duplicates.

They apply cleanly on 3-6 and you may squash them into one large
commit but it makes reviewing them even more complicated. I did not do
that yet because it makes reviewing even more difficult. They allow to
update ScRangeLists correctly when the cells are moved or deleted.
There is one small problem with updating moving only parts of one of
the ScRange entries but this will require more concept work to find
all corner cases and to implement this. These cases are not that
important and these patches here are already an improvement in this
direction.

Please apply them in the following order:

http://cgit.freedesktop.org/libreoffice/core/commit/?id=7222a571d0d458810c1b23871f8b91491db4462d
http://cgit.freedesktop.org/libreoffice/core/commit/?id=a3c4ee1653166ee2ac1f1b9d65ff1065b6288ebc
http://cgit.freedesktop.org/libreoffice/core/commit/?id=4cf0759e7c6bd698c929a11c771d2ab03f1b9536
http://cgit.freedesktop.org/libreoffice/core/commit/?id=e6bca122176cdb2b6e822fc933f159dc3e3c8d46
http://cgit.freedesktop.org/libreoffice/core/commit/?id=7a182026fce922a9f69e8da76d46e87e7188a4e9
http://cgit.freedesktop.org/libreoffice/core/commit/?id=764e7e71038d5ae66061f44bc0cd51ce33ae96ed
http://cgit.freedesktop.org/libreoffice/core/commit/?id=1e3919f040ade5d0f7f9fa854b3ed23366080c0c

Nearly all of these patches are only changing ScRangeList and not
directly the conditional format code. I have some unit tests for
ScRangeList::UpdateReference and ScRangeList::DeleteArea in
sc/qa/unit/rangelst_test.cxx

I've been reviewing these patches since this morning, and I'm only half-way through. I'll need the rest of the day to fully review this patch.

Though I try to approve this change into 3.6, given the size of the change and how close we are to release 3.6.2, I'd be more comfortable if we targeted this for 3.6.3, to give us more time to review.

Best,

Kohei

--
Kohei Yoshida, LibreOffice hacker, Calc

Context


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.