[REVIEW] don't try to update range names twice in ScDocument::TransferTab

Markus Mohrhard markus.mohrhard at googlemail.com
Thu Mar 1 05:40:53 PST 2012


Missed the attachment, You can find the commit at
http://cgit.freedesktop.org/libreoffice/core/commit/?id=1b93146d442bb8396126d6106f05a5bb0ee9fffa

2012/3/1 Markus Mohrhard <markus.mohrhard at googlemail.com>:
> Hey,
>
> the attached patch prevents double updating of the index for copied range names.
>
> The new code updating range names is in cell.cxx adjustRangeName which
> does the job directly in the formula tokens. The removed code did
> something similar for global range names but was incomplete and
> contained some errors. In some cases the removed code did stilla djust
> the range name while in most cases it did not.
>
> There is no need for this code anymore since all this functionality is
> now covered by adjustRangeName and the old code could not deal with
> local range names and some more advanced adjustments while copying
> between different documents.
>
> Regards,
> Markus


More information about the LibreOffice mailing list