Age | Commit message (Collapse) | Author |
|
(cherry picked from commit 9461b0e1385b87d2490bf9402729fb9e912395e9)
Change-Id: Iead5b4ff6064395fd5abbcdea73db38415dbb37c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144514
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>
|
|
Follow-up. I didn't want to include it in the main deprecation patch.
No good reason - android is just too different and untestable by me.
Change-Id: I661211aea619af256e38dc685fdd0662e5aa7272
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/140284
Reviewed-by: Justin Luth <jluth@mail.com>
Tested-by: Jenkins
|
|
Change-Id: Idc8cc3c24d061537a76a37f4fa84951a41a42657
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144470
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Id243f28f267a5ea22625e277d6272f8514d33ad9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144469
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...like in other code accessing XMLElementWrapper_XmlSecImpl. (See the upcoming
commit introducing loplugin:unocast on why such dynamic_casts from UNO types are
dangerous.)
Change-Id: Ib67332dd2d6a1aa31f5515940fdd6f4a9f638bbb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144508
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...like in similar code introduced in the initial
88ac77c0bc21accfd6f5c404217c3c2aaef2d674 "gpg4libre: List and view GPG keys",
but which was then apparently forgotten in this code later introduced in
06929219aeb0cb3550bf0cf8b5de7f1164c00a8d "gpg4libre: Show whether pgp keys are
trusted or not". (See the upcoming commit introducing loplugin:unocast on why
such dynamic_casts from UNO types are dangerous.)
Change-Id: I6606d0fa0be515a45a190d2da7e16d5b712bfc4d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144507
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...when building against recent Clang 16 trunk,
> In file included from compilerplugins/clang/sharedvisitor/sharedvisitor.cxx:30:
> In file included from compilerplugins/clang/cppunitassertequals.cxx:13:
> compilerplugins/clang/compat.hxx:40:14: error: 'value' is deprecated: std::optional::value is throwing. Use *X instead [-Werror,-Wdeprecated-declarations]
> return o.value();
> ^~~~~
> *X
> compilerplugins/clang/pointerbool.cxx:118:52: note: in instantiation of function template specialization 'compat::value<llvm::APSInt>' requested here
> if (compat::has_value(ret) && (compat::value(ret) == 1 || compat::value(ret) == 0))
> ^
> ~/llvm/inst/include/llvm/ADT/Optional.h:273:3: note: 'value' has been explicitly marked deprecated here
> LLVM_DEPRECATED("std::optional::value is throwing. Use *X instead", "*X")
> ^
> ~/llvm/inst/include/llvm/Support/Compiler.h:143:50: note: expanded from macro 'LLVM_DEPRECATED'
> #define LLVM_DEPRECATED(MSG, FIX) __attribute__((deprecated(MSG, FIX)))
> ^
Change-Id: I4320c279a464e52ad608df2a340f62795558a7b8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144472
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
<https://github.com/llvm/llvm-project/commit/8f0df9f3bbc6d7f3d5cbfd955c5ee4404c53a75d>
"[clang] Convert OptionalFileEntryRefDegradesToFileEntryPtr to std::optional"
Change-Id: I60bf4a770d6037edcfe0129591fd46941c8903db
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144471
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Plus some bonus cleanup.
I want these files to be clean of junk.
How I missed these SAL_CALLs in my previous clean attempts,
I don't even dare to speculate.
Change-Id: I6056b61bbbdb088fe83752496df00d0fa319e637
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144137
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
|
|
Similar to font size & font color.
Change-Id: I0987c50235dc317dd4b778a5b9946a0415558eeb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144502
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|
|
EditEngine stuff uses CharBackColor, while SW used BackColor.
Deprecate the SW-only SID.
What does this do?
It allows the toolbar "Character Highlighting Color"
to work with both body text (previously BackColor)
and comment text(CharBackColor).
Otherwise, the toolbar item was disabled when moving into
a comment. (Shapes/Textboxes got their own formatting toolbar,
so those are unaffected.)
AFAIK, after this commit, SID_ATTR_CHAR_COLOR_BACKGROUND
should still work fine for API calls, or for customized toolbars.
That's why I didn't try to remove it completely - just deprecate.
Do I know what I am doing here?
No, although I'm fairly confident nontheless,
and I've had some good help from reviewers.
Change-Id: I0afcbdb1b91c872f6d47f3c82c573f3b9fea6dfc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137334
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
|
|
ScDocShell: :GetViewData() can return nullptr
Change-Id: I39bb2dce1ab233f39b0ec687ed71089b35537f9c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144467
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
|
|
* Update helpcontent2 from branch 'master'
to e2388e894212c7be710371c589263c45e994a304
- set id attribute from paragraphs as type ID for uniqueness constraint
so that the xmllint job on jenkins can detect those
Change-Id: I45058e114fbce907cb4fc26ec5a8d06974eb8145
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/144465
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
use more ImplInheritanceHelper to reduce boilerplate
Change-Id: I3ecc79ac5a1ae7a005845a7f1460e3b5037e6d53
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144442
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
use more ImplInheritanceHelper to reduce boilerplate
Change-Id: I209e013f84a83339d0f4a6dfd90b462a47b6beeb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144441
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
* Update helpcontent2 from branch 'master'
to b91a0a30115e414982bd4b3f710e0218661b4e18
- Fix typo
Change-Id: Iede879decee335d13e50df7244326dbc44728145
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/144503
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: Ibd4381e8e1f38588706a9cb093a63a9673776d52
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144504
Tested-by: Julien Nabet <serval2412@yahoo.fr>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: I6d04185f14c3ec72b40426bd881cac80528cc736
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143244
Tested-by: Jenkins
Reviewed-by: Hossein <hossein@libreoffice.org>
|
|
seen during import of:
forums/forum-mso-en4-187408.xls
forums/forum-mso-en4-187900.xls
forums/forum-mso-en4-187890.xls
Change-Id: Id15e9c1ea98d761225d41850b9b2aa58d9c9e407
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144466
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
* Update helpcontent2 from branch 'master'
to 1445fa8fda93a2f0ffab873c926390adb4937397
- Fix duplicate IDs in non-localized content
Change-Id: I759b05a2ba10671700c9222f54e61edf39f5baff
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/144538
Tested-by: Jenkins
Tested-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
|
|
* Update translations from branch 'master'
to 7d5ae8c4582675a8e102e4d330658a588ef015cf
- update translations for 7.5.0 rc1/master
and force-fix errors using pocheck
Change-Id: I3c9458097e8ea70e7931e1646095b0da3f5a04eb
|
|
Searching for content controls can be filtered by tag, title, or Id.
The function was erroneously returning the last control in the
list if the specified control did not exist.
Change-Id: I0cfbccadc134b64456d703f1e19db6e142f162fd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144346
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
|
|
Change-Id: I50c8f9579a9fb07431ad0d6eebc5f0df8e40ebd4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144463
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ia8cdf5d446647e0f3d60c14e1aae315db63769c7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144462
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
The problem was that the SwTextControl could be deleted
by the user or in VBA and my references (or pointers
for that matter) to m_rCC would be invalid.
Instead, use the shared_ptr, which will always be valid,
from the SwContentControl. Thankfully, there is
backwards access to the Format and Text controls.
This is nicer anyways. It has reduced code lines,
and simpler access to the most used aspect.
Change-Id: Ie382b33ecda5c3d0024f0eabcb8ff158db942f52
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144345
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
|
|
Change-Id: Ib467fa5eec2560943e10adc2c440fdbf8f853a2f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144446
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
(and would also be macOS 13 for the intel builds if the old macMini
would be on the officially supported list)
Change-Id: I4205f30274bdd43d80d02d6cda863ec83bdda63e
|
|
(See the upcoming commit introducing that loplugin:unocast on why such
dynamic_casts from UNO types are dangerous.)
Change-Id: I11bc363447c44319bc47f7eebb7084f64ea85511
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144400
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Also fix the bad Undo label, where the invisible IAA character
became visible.
Choosing one of the spelling suggestions of the context menu
of a bad word fixed the spelling of the original word, but
also deleted its inword comment. Keep the comment at the end
of the fixed word.
Fix also the bad Undo label, which contained the original
word with the Interlinear Annotation Anchor (IAA) character.
Now it shows only the word, but not the Unicode replacement
glyph of IAA.
Regression from commit b0f170d7df9cff12535d2ecfd146b32b745a8ef8
"0xFFF9 is a better choice for CH_TXTATR_INWORD than 0x0002".
Change-Id: I5bfa214a060c446e9897e806832bedfe54241e1e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144397
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
The background tiling images become larger when
opening and closing Area tab: reopening the image
properties in the Area tab shows that the Width
and Height got reset to the original values of the
image.
Regression from commit 686349476e03f951f4a9ff9755b9f71951b64ea5
"[GSoC] Move all fill style tabs inside area tab".
Change-Id: Ibc4dfd73dedd81b1b7fe17574198dd4999aab967
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143819
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
by using bitmap image sizes instead of the original ones.
Change-Id: Id6f2777bf6803bca7252878203a12ab796ac33dc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143766
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
If the object is missing, it's still possible to keep its shape
by exporting its preview graphic, as before the regression.
Regression from commit adc042f95d3dbd65b778260025d59283146916e5
"tdf#124333 PPTX import: fix Z-order of embedded OLE objects".
Change-Id: Ib2fd00f53a80572cfc9acaefea55015780c57da8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144040
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
The bugdoc had a numbering, where the paragraph marker was explicitly
formatted to a custom sans font, and this font is also used for the numbering
portion's font. This was imported from DOCX correctly, but once you save
to ODT and reload, the font used in the numbering portion changed from
sans to serif, which is incorrect.
The reason for this seems to be that earlier
5ba30f588d6e41a13d68b1461345fca7a7ca61ac (tdf#64222 sw: better DOCX
import/export of paragraph marker formatting, 2019-09-06) introduced
support for storing this paragraph marker formatting in
RES_PARATR_LIST_AUTOFMT, but this was lost on ODT export / import.
Fix the problem by 1) adding the autostyle to the autostyle pool, so the
font gets written when writing automatic char styles 2) extending
SwXTextPortion to have a mode where it exposes the
RES_PARATR_LIST_AUTOFMT of the current text node and 3) improving
lcl_CreatePortions() to expose a trailing empty text portion that works
with these properties.
This also required adjusting CppunitTest_sw_ooxmlexport2's
testFdo64238_b, which explicitly asserted that the format of the
paragraph marker is not exposed in the UNO API text portion enumeration.
An additional improvement would be to go with a more explicit markup in
the ODT output for RES_PARATR_LIST_AUTOFMT, but we would need to decide
what name to use there, since currently we call this paragraph marker
formatting RES_PARATR_LIST_AUTOFMT / ListAutoFormat, and somewhat
confusingly this is primarily about paragraph marker formatting, not
lists / numberings (that is just a consequence).
Change-Id: I19c7eed19c6fc85c251ef87a5181c0719a0a382c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144447
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|
|
Regression from commit eec48130271188cab63665acedbabf1ff5e850a2
"tdf#148926 tdf#151678 PPTX import: position of standard connector
- part1".
Change-Id: Ib9381403f7c0edf9a06e3d98965067a51b4797d8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144000
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
... ContentControlsEnumWrapper, ListEntriesEnumWrapper and
SwHTMLTextFootnote
See tdf#94879 for motivation.
Change-Id: I1efbf1d7e3def37868ef0e36aeb4ec184d31eb46
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144445
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|
|
* Update helpcontent2 from branch 'master'
to 318ec1f0e6248eac2536256bf26e1a183fc59541
- janitor: update comment for the bookmarks.part rule
the remarks about the indicator files are obsolete after the rule has
been rewritten to work in batch and not on single files
Change-Id: I8a2509aed7a38b9cff0dacec3d4161bc020fde31
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/144468
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
Choosing one of the spelling suggestions in the spelling
dialog, or rewriting the bad word removed also its
inword comment. Keep the comment at the end of the fixed word.
Regression from commit b0f170d7df9cff12535d2ecfd146b32b745a8ef8
"0xFFF9 is a better choice for CH_TXTATR_INWORD than 0x0002".
Change-Id: Iaf083453fad4e0ca99e254f53b20ead86b3ba15e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144458
Tested-by: Jenkins
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
When a column was filtered for values that included 0,
the all string entries were always checked in the Autofilter
dropdown (because their conversion to number, i.e. value 0
was used to check their existence in the filtered values
instead of their string value).
Regression from commit f6b143a57d9bd8f5d7b29febcb4e01ee1eb2ff1d
"tdf#142910 sc filter: fix "greater than" or "smaller than" etc".
Change-Id: Ib659aba9d6f3d6bc3557547b1a27963f51e3eca3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143777
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
Bad import of the Empty filter operator resulted shifted
filter settings, e.g. selected "A" and "B" instead of
"empty" and "A" from the range "empty", "A" and "B".
Regression from commit f6b143a57d9bd8f5d7b29febcb4e01ee1eb2ff1d
"tdf#142910 sc filter: fix "greater than" or "smaller than" etc".
Change-Id: I33082d77fb9e707aa474fb22753a001960341d66
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143776
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
those are only meant to be used by the build system (primarily
RepositoryExternal.mk) - of course no rule without exception:
concat-deps reads SYSTEM_BOOST from env, so set it in the corresponding
recipe lines
Change-Id: I83c88fc6ad4467b429771c43650fe7fc0ccde407
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144343
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
use more ImplInheritanceHelper to reduce boilerplate
Change-Id: I2b4937ebee2e8cc16759df0eacdeb160343a7001
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144440
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
The shapes got extra 3D shape properties by the default template,
which caused visible difference in MSO.
Follow-up to commit I02f1edea84b50dbb9b0e0a59b6062a29894afde4
(tdf#152186 PPTX export: fix shadow of the shape in theme)
Follow-up to commit 25e9766e9b87342b6f1b29dc146d5f33e12157a7
(tdf#92222 PPTX export: fix line width of the shape in theme)
Change-Id: I84f931e45e7656c456fe910cbb5e09aabeb8924f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143516
Reviewed-by: László Németh <nemeth@numbertext.org>
Tested-by: László Németh <nemeth@numbertext.org>
|
|
* Update helpcontent2 from branch 'master'
to e739da903b92597316aeac26ec190a8d7c7bec28
- convertfilters.py: Don't add name attribute to links
Change-Id: I556ee26594fada9722ef898ef9e36df4644e4f37
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/144461
Tested-by: Jenkins
Tested-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
|
|
* Update helpcontent2 from branch 'master'
to c8611c6e320bf5db95a6d1fa54d3ff6e13f9eaa7
- Remove name attribute from links
Change-Id: I999712c68026ddd40578c1f5e92a994c24a11eae
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/144460
Tested-by: Jenkins
Tested-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
|
|
The nlpsolver module is "Solver for Nonlinear Programming", which was
mistakenly called "New Linear Programming", and this is now fixed.
The change will appear within a week in:
https://docs.libreoffice.org/nlpsolver.html
Change-Id: I05b14d1e4056d8d0797728905886edef867f29e1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144408
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Also the first footnote can be at arbitrary
place in footnote.xml.
Follow-up to commit 09ae3c01940bbc25ffde51963683b04e3cb4bb6a
"tdf#152203 DOCX import: fix mixed footnotes/endnotes".
Change-Id: Iab356f7373483d812f0e802a994357fdad831d9d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144380
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
to try and untangle the dependency mess a little.
This means
(*) tunneling a piece of information from VCLXWindow via
the XWindowPeer::getProperty method, so
we do not need to directly call VCLXWindow.
(*) moving some formpdf code from framework "down" to vcl,
where the drawinglayer code can use it
Change-Id: I2c1ff3e34941b3147e66cf0a2d0fdb13293e3aa2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144341
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I650aeebb4d021911c1f14d4867c5beee84020155
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143880
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>
|
|
==324198== Invalid read of size 8
==324198== at 0x2D840D75: std::__uniq_ptr_impl<SwField, std::default_delete<SwField> >::_M_ptr() const (unique_ptr.h:191)
==324198== by 0x2D840D54: std::unique_ptr<SwField, std::default_delete<SwField> >::get() const (unique_ptr.h:462)
==324198== by 0x2D932B9B: SwFormatField::GetField() (fmtfld.hxx:122)
==324198== by 0x2E740E7B: SwXTextField::Impl::GetField() const (unofield.cxx:2634)
==324198== by 0x2E7487EB: SwXTextField::setPropertyValue(rtl::OUString const&, com::sun::star::uno::Any const&) (unofield.cxx:2150)
==324198== by 0x1100C123: XMLPropertyBackpatcher<short>::ResolveId(rtl::OUString const&, short) (XMLPropertyBackpatcher.cxx:73)
==324198== by 0x1100BB0F: XMLTextImportHelper::InsertFootnoteID(rtl::OUString const&, short) (XMLPropertyBackpatcher.cxx:180)
==324198== by 0x10FF7E71: XMLFootnoteImportContext::startFastElement(int, com::sun::star::uno::Reference<com::sun::star::xml::sax::XFastAttributeList> const&) (XMLFootnoteImportContext.cxx:99)
==324198== by 0x10C40361: SvXMLImport::startFastElement(int, com::sun::star::uno::Reference<com::sun::star::xml::sax::XFastAttributeList> const&) (xmlimp.cxx:812)
==324198== by 0x28FBFE71: (anonymous namespace)::Entity::startElement((anonymous namespace)::Event const*) (fastparser.cxx:470)
==324198== by 0x28FBED4E: sax_fastparser::FastSaxParserImpl::callbackStartElement(unsigned char const*, unsigned char const*, unsigned char const*, int, unsigned char const**, int, unsigned char const**) (fastparser.cxx:1304)
==324198== by 0x28FBCCAF: (anonymous namespace)::call_callbackStartElement(void*, unsigned char const*, unsigned char const*, unsigned char const*, int, unsigned char const**, int, int, unsigned char const**) (fastparser.cxx:333)
==324198== Address 0x259e2b68 is 328 bytes inside a block of size 344 free'd
==324198== at 0x4848669: operator delete(void*) (vg_replace_malloc.c:923)
==324198== by 0x2E4EC2B1: SwFormatField::~SwFormatField() (atrfld.cxx:132)
==324198== by 0x8635DA2: SfxItemPool::Remove(SfxPoolItem const&) (itempool.cxx:802)
==324198== by 0x2E5BB014: SwTextAttr::Destroy(SwTextAttr*, SfxItemPool&) (txatbase.cxx:63)
==324198== by 0x2E594EE2: SwTextNode::DestroyAttr(SwTextAttr*) (thints.cxx:1302)
==324198== by 0x2E55C4EC: SwTextNode::~SwTextNode() (ndtxt.cxx:238)
==324198== by 0x2E55CB38: SwTextNode::~SwTextNode() (ndtxt.cxx:227)
==324198== by 0x2DEE048F: SwNodes::RemoveNode(o3tl::strong_int<int, Tag_SwNodeOffset>, o3tl::strong_int<int, Tag_SwNodeOffset>, bool) (nodes.cxx:2325)
==324198== by 0x2DEE50C2: SwNodes::DelNodes(SwNodeIndex const&, o3tl::strong_int<int, Tag_SwNodeOffset>) (nodes.cxx:1384)
==324198== by 0x2DC87B02: sw::DocumentContentOperationsManager::DeleteSection(SwNode*) (DocumentContentOperationsManager.cxx:2118)
==324198== by 0x2DD2BB66: sw::DocumentLayoutManager::DelLayoutFormat(SwFrameFormat*) (DocumentLayoutManager.cxx:285)
==324198== by 0x2DB0900E: DelFlyInRange(SwNodeIndex const&, SwNodeIndex const&, SwIndex const*, SwIndex const*) (docedt.cxx:254)
maybe this is a problem since the various changes associated with:
commit 2c9298e4f667a5dd7606d79890fcedfff0f66e26
Date: Fri Apr 5 23:14:47 2019 +0200
SwXTextField::Impl FieldTypeClient: SwClient no more
commit e18359445fabad9ba1a704600e9ee327112cc6ae
Date: Sun Apr 14 13:33:35 2019 +0200
[API CHANGE] SwXTextField: no more SwModify/SwClient
Change-Id: I52aa0a70604ea5ce96d2b55a9a2ad363a889f58e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136515
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
save SwDoc* before calling DeleteAndJoin, so SetFormatField doesn't
use m_pImpl->m_pDoc potentially cleared by it
Change-Id: Ia130f224fcccd35e1dfafbbaf5c461c9bd032f9d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144407
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|