summaryrefslogtreecommitdiff
path: root/testtools
diff options
context:
space:
mode:
authorArmin Le Grand (allotropia) <armin.le.grand.extern@allotropia.de>2024-01-12 11:22:04 +0100
committerArmin Le Grand <Armin.Le.Grand@me.com>2024-01-12 15:22:36 +0100
commitd22a86089edfcadbef5231525a2947b954f4784e (patch)
tree089a815a6ef4dd7bedad50381a7b4db6d15fef68 /testtools
parentae3a730a4b64fdb9698c753313504bbe174f1519 (diff)
tdf#158783 Correct compares of SwTOXMark Items
That item was never 'pooled', so operator== was not really ever used. It just compared the 'type', so pretty many instances were assumed to be equal, what is wrong. We discussed to implement it (there is quite some content), but we came to the point that it's only safe to say instances are equal when same instance -> fallback to ptr compare. This came into play since I identified/changed many (160?) places where SfxPoolItems were ptr-compared when doing that paradigm change in Items. This leads to the two methods 'areSfxPoolItemPtrsEqual' which just makes ptr compare and 'SfxPoolItem::areSame' which also will use op==. For the initial adaption I chose the wrong function adapting places where SwTOXMark were involved. Change-Id: I7df029ad4542719681b1455de17ed5990d248395 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161963 Reviewed-by: Michael Stahl <michael.stahl@allotropia.de> Tested-by: Armin Le Grand <Armin.Le.Grand@me.com>
Diffstat (limited to 'testtools')
0 files changed, 0 insertions, 0 deletions