diff options
author | Michael Stahl <michael.stahl@allotropia.de> | 2022-07-13 16:11:29 +0200 |
---|---|---|
committer | Michael Stahl <michael.stahl@allotropia.de> | 2022-07-14 13:51:01 +0200 |
commit | d7827f712ddd21a6c1e151f54dc6eba5c12690da (patch) | |
tree | 3ab3c5bc3c9a825979ae60866921f339a3aaa1c5 /compilerplugins/clang/test/refcounting.cxx | |
parent | 306510f0850518dc3a8162769e4aeb83795486be (diff) |
tdf#149978 sw: ODF import: fix nondeterministic automatic styles
The problem is that in SwpHints::TryInsertHint() there is a check for
IsInXMLImport() that is presumably an optimization to avoid the
potentially expensive call to BuildPortions().
While LO would only produce 1 text:span referencing an automatic style
around any given character content, this is not required by ODF, and so
other producers may legitimately produce such nested text:span elements.
Unfortunately the current SwpHints::Insert() isn't deterministic, the
RES_TXTATR_AUTOFMT with same start/end will be compared by address in
CompareSwpHtStart() (whereas RES_TXTATR_CHARFMT has a sort number for
this), so the result is going to be a random order.
Change-Id: Id62a7ff5fb85dbe42b7e1a27b0d8b36b74cf1100
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137033
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Diffstat (limited to 'compilerplugins/clang/test/refcounting.cxx')
0 files changed, 0 insertions, 0 deletions