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

Markus Mohrhard markus.mohrhard at googlemail.com
Thu Mar 1 03:50:46 PST 2012


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