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.
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.