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


Hey,

2012/1/27 Eike Rathke <erack@redhat.com>:
Hi Kohei,

On Thursday, 2012-01-26 22:37:14 -0500, Kohei Yoshida wrote:

So, I took a second look at this, and came up with the attached version
which I think is better and more correct than the first one.  This one
will correctly honor the delete note flag, and also maintains the option
of deleting the note cell if the right conditions are met (i.e. delete
note flag is set and nobody is listening to it).

Yup, that does it. Perfect :)

As for the pivot table change, that code block is indeed necessary; when
non-empty cells exist that overlaps the range where the new table output
gets written, those cells may survive if they happen to be where the
table content is empty.  So I reverted that part of my initial change.

Good you sorted that out.

I reverted the previous af70bc00c6714eb8695babdf5af07416552f7034 and
committed the new patch to master, and to 3-5 with my sign-off.

http://cgit.freedesktop.org/libreoffice/core/commit/?h=libreoffice-3-5&id=08adeedce926bac1a010c26f2a4ae2d5fb761376

Two more reviews needed for 3-5-0.


Looks good to me. One more review needed for 3-5-0.

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.