[PUSHED 3-6][REVIEW 3-6-1 2/3] Fix for fdo#53640 - Pivot table from database source
Eike Rathke
erack at redhat.com
Wed Aug 22 12:50:31 PDT 2012
Hi,
On Wednesday, 2012-08-22 20:22:19 +0100, Michael Meeks wrote:
> On Wed, 2012-08-22 at 15:05 -0400, Kohei Yoshida wrote:
> > https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdiff;h=f4ed9a5adbea3de88870ff4f5cb202bd606e0746
> > cherry-picked to 3-6 and preferably to 3-6-1 as well. This fixes
>
> Looks sensible - I'm sure clang would have something to say about this
> sort of incrementing of dead variables ;-)
Michael cherry-picked that to 3-6 as
https://gerrit.libreoffice.org/gitweb?p=core.git;a=commit;h=a28d9cad672739cbf8dc6cf1f9db7d0f8becfee1
which has my +1
> > Also, optionally I'd like
> > https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdiff;h=88979ef6ffe7921200311513d6601f9c5d582a42
> >
> > backported as well. Leaving this code in would result in re-populating
> > the pivot cache twice unnecessarily, which would degrade performance.
>
> I pushed that - though it looks more scary.
In 3-6 as
https://gerrit.libreoffice.org/gitweb?p=core.git;a=commit;h=257a5fb833c2c03a3a82b19b9ab5a7cb3b5848ef
I can't say much about that without digging into cache/table refreshs
other than having to trust Kohei's judgement. +0.5? ;-)
Eike
--
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3 9E96 2F1A D073 293C 05FD
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20120822/59a8ee6f/attachment.pgp>
More information about the LibreOffice
mailing list