Hi Kohei,

On Friday, 2012-01-27 00:03:29 +0100, Eike Rathke wrote:

> > Then the second part is in ScDPOutput::Output(), where the old content
> > gets removed before writing out a new one.  The thing is, this method
> > only gets called from ScDPObject::Output(), and that method already
> > deletes the old content which makes the removal code in ScDPOutput
> > unnecessary.  So, when updating or refreshing the pivot table output,
> > calc was essentially deleting the range twice.
> 
> That's indeed unnecessary.

Or is it? Before that, CalcSizes() is called that modifies nTabEndCol
and nTabEndRow that are passed to pDoc->DeleteAreaTab(), didn't check if
that actually may grow the area that was cleared before.

  Eike

-- 
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3  9E96 2F1A D073 293C 05FD

Attachment: pgp0VZ8bxftq0.pgp
Description: PGP signature

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to