Commit Graph

424940 Commits

Author SHA1 Message Date
Andrea Gelmini
c44668a6c1 Fix typos
Change-Id: I1b966b636f67a549718ca19c00c2820e8d168c7a
Reviewed-on: https://gerrit.libreoffice.org/56589
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2018-06-28 14:37:39 +02:00
Vasily Melenchuk
d77f8eef16 tdf#86612: statusbar: hide some elements if width is not sufficient
new statusbar element property mandatory=true/false to determine
if this element can be hidden if total statusbar width is not
enough to fit all elements.

marked some calc and draw statusbar elements as not mandatory.

Change-Id: I20e26d3c4bd865e94ea48632a1e97d55f3fa712f
Reviewed-on: https://gerrit.libreoffice.org/56443
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2018-06-28 14:02:46 +02:00
Miklos Vajna
f4c73f90da tdf#117866 cui personalization: fix a possible race
The case when the thread is re-scheduled exactly after checking for
m_bExecute but before taking the solar mutex.

Change-Id: I494c123a6225da60407391771622208c055ae49b
Reviewed-on: https://gerrit.libreoffice.org/56585
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins
2018-06-28 13:30:43 +02:00
Nickson Thanda
0e0ceebc0d tdf#36867 - Undo autocapitalization when inserting a cell in Calc
Change-Id: Ia70144d6c86b1b08e14425bf537e9c39d585a614
Reviewed-on: https://gerrit.libreoffice.org/55820
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
2018-06-28 13:28:48 +02:00
Samuel Mehrbrodt
b80207f5ac tdf#118429 Provide unsigned signature image
when creating signature line.
OOXML Export expects this.

Change-Id: Ib6ab1f879c5ef27e8347002576ba95104117cadd
Reviewed-on: https://gerrit.libreoffice.org/56576
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2018-06-28 12:50:29 +02:00
Stephan Bergmann
a1f5b1f30a Clean up ownership tracking of SalVisual's visual (in X11 Visual base class)
GCC trunk towards GCC 9 emits -Wdeprecated-copy because SalVisual has implicit
copy functions but a user-declared dtor.  The implicitly-defined copy functions
are actually used (so cannot be deleted), but in fragile interaction with the
semantics of the user-provided dtor in the SalColormap(sal_uInt16) ctor.

Changing SalVisual into a move-only class (moving the "this instance must delete
its visual member" information) doesn't work well, as SalDisplay::ScreenData is
used as a supply of SalVisual instances (but which will never be of the "this
instance must delete its visual member" variety) that are then copied into
SalColormap::m_aVisual.

As only SalColormap creates SalVisual instances of the "this instance must
delete its visual member" variety, what actually works is to track that
information in SalColormap instead of directly in SalVisual, and make
SalColormap a move-only class.

Change-Id: Ib968a38c40b660ce91981b3c7b281f4add6ece6b
Reviewed-on: https://gerrit.libreoffice.org/56579
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2018-06-28 11:12:15 +02:00
Mike Kaganski
4600b07c1d tdf#102195: don't exclude everything between first and last subset
When subsets are non-contiguous, we need to include the parts between
subsets.

Change-Id: I28214dccc75e6a6af5c65397b2126049a65bf79f
Reviewed-on: https://gerrit.libreoffice.org/56571
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2018-06-28 10:03:48 +02:00
Noel Grandin
1113f93db3 loplugin:useuniqueptr in ScAccessiblePreviewTable
Change-Id: I75b37dab51f3510f6e7155c191a85af99b0d350f
Reviewed-on: https://gerrit.libreoffice.org/56547
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 09:51:17 +02:00
Gabor Kelemen
f4dafe050a tdf#42949 Fix IWYU warnings in include/osl/*h
Found with bin/find-unneeded-includes
Only removal proposals are dealt with here.

Change-Id: Ib2301526d7aa6982af6c8c79ed7e9a4c34b7bbf7
Reviewed-on: https://gerrit.libreoffice.org/56491
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2018-06-28 09:08:15 +02:00
Miklos Vajna
64acf709a6 pdfium: replace FPDFPath_GetDrawMode() patch with backport
Change-Id: I86c7c0f73a21f670676716d7c22f519ed82da145
Reviewed-on: https://gerrit.libreoffice.org/56544
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2018-06-28 09:07:12 +02:00
Noel Grandin
a30e805bf7 loplugin:useuniqueptr in ScConsolidateDlg
Change-Id: I80cd0b051a137f51aa03ebc573369c945ca502da
Reviewed-on: https://gerrit.libreoffice.org/56553
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:34:10 +02:00
Noel Grandin
e53ed47e9d loplugin:useuniqueptr in ScUndoDBData
Change-Id: I6107079f8474a2ddc76a9cb6e93552d95861dc6c
Reviewed-on: https://gerrit.libreoffice.org/56563
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:34:00 +02:00
Noel Grandin
9954df6905 loplugin:useuniqueptr in ScAutoFmtPreview
Change-Id: I748382e73a2e2b80292e8c396cb61ca805c03549
Reviewed-on: https://gerrit.libreoffice.org/56562
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:33:54 +02:00
Noel Grandin
0d41681eef loplugin:useuniqueptr in ScUndoWidthOrHeight
Change-Id: I224c5d7b2562cc0ad5d03dc7dffc5476c83a5f54
Reviewed-on: https://gerrit.libreoffice.org/56561
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:33:48 +02:00
Noel Grandin
2037e85b79 loplugin:useuniqueptr in ScAccessibleDocument
Change-Id: I199b5f3bd0d8555dc01d6f51208ebe4526a32f9a
Reviewed-on: https://gerrit.libreoffice.org/56559
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:33:42 +02:00
Noel Grandin
4baf5adc8f loplugin:useuniqueptr in ScFilterDlg
Change-Id: I8d7561df19d433db5454bcf4d60e1cb81c44f031
Reviewed-on: https://gerrit.libreoffice.org/56558
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:33:36 +02:00
Noel Grandin
c25db26f4c loplugin:useuniqueptr in ScNavigatorDlg
Change-Id: I27ef6e27c8eef01a5de0f117ee03364ca545b27a
Reviewed-on: https://gerrit.libreoffice.org/56560
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:33:28 +02:00
Noel Grandin
0bd2c35937 loplugin:useuniqueptr in ScModule
Change-Id: I6433050af217668800c7257433c11bfec37d9634
Reviewed-on: https://gerrit.libreoffice.org/56557
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:33:08 +02:00
Noel Grandin
f35ddce379 loplugin:useuniqueptr in ScAccessibleDocumentPagePreview
Change-Id: I0b90d59e627691c6c187f34591992301102afcfe
Reviewed-on: https://gerrit.libreoffice.org/56556
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:33:00 +02:00
Noel Grandin
b1b490ee35 loplugin:useuniqueptr in ScAccessiblePreviewCell
Change-Id: I99b9809b3924285b4f0d58905352777b948ee592
Reviewed-on: https://gerrit.libreoffice.org/56546
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:30:40 +02:00
Noel Grandin
ceff8f7359 loplugin:useuniqueptr in ScDocShell
Change-Id: I6d4d4899670d8c3254f8c4337a14ba2bb937a2bf
Reviewed-on: https://gerrit.libreoffice.org/56555
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:29:28 +02:00
Noel Grandin
02e02fc032 loplugin:useuniqueptr in ScSpecialFilterDlg
Change-Id: I864e09c6c86a852bc79d9e2ea6b50c568fd0155f
Reviewed-on: https://gerrit.libreoffice.org/56554
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:29:03 +02:00
Noel Grandin
dee73a9af5 loplugin:useuniqueptr in ScZoomSliderWnd
Change-Id: If6cc62d08fbd589f332514e01a42aae8f919e68f
Reviewed-on: https://gerrit.libreoffice.org/56552
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:28:12 +02:00
Noel Grandin
e528b7fd3a loplugin:useuniqueptr in ScAccessibleNoteTextData
Change-Id: I38101b3046c4354f48b6c9d34a016351659e9795
Reviewed-on: https://gerrit.libreoffice.org/56551
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:28:04 +02:00
Noel Grandin
00505cad5e loplugin:useuniqueptr in ScTransferObj
Change-Id: Ie934c2d0ad0c71b22d84976661b19d07e2eb64d4
Reviewed-on: https://gerrit.libreoffice.org/56550
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:27:44 +02:00
Noel Grandin
4dbfc74325 loplugin:useuniqueptr in ScCheckListBox
Change-Id: Ic3059e266a9276786e33ba5adf846e51551bdd6f
Reviewed-on: https://gerrit.libreoffice.org/56549
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:27:37 +02:00
Noel Grandin
1156f2ed90 loplugin:useuniqueptr in ScAccessiblePreviewHeaderCell
Change-Id: Id1ccd3657ffb0a5ae4762ea6ce25f8ce2c77db92
Reviewed-on: https://gerrit.libreoffice.org/56548
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:27:25 +02:00
Noel Grandin
291f125e84 loplugin:useuniqueptr in ScAccessiblePageHeaderArea
Change-Id: Ic32996b3aaf415109c1fdc336db138ec12159ce2
Reviewed-on: https://gerrit.libreoffice.org/56545
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-28 08:27:11 +02:00
Zdeněk Crhonek
681b3facd3 uitest ProtectSheet, protectSpreadsheet, clearCells, tdf#101904
Change-Id: I18f5563be60fa7826d8f720288abcfaf9d93313e
Reviewed-on: https://gerrit.libreoffice.org/56564
Tested-by: Jenkins
Reviewed-by: Zdenek Crhonek <zcrhonek@gmail.com>
2018-06-28 07:05:40 +02:00
Zdeněk Crhonek
df579bea05 repair uitest solver.py
Change-Id: I64b5d78e9a63c3b78b6ac87123bf09903556f8d9
Reviewed-on: https://gerrit.libreoffice.org/56543
Tested-by: Jenkins
Reviewed-by: Zdenek Crhonek <zcrhonek@gmail.com>
2018-06-28 07:05:04 +02:00
Luboš Luňák
7e1cabd965 blacklist ocGetPivotData for calc's threading
There appears to be a lot of caching involved in the code invoked
by ScGetPivotData(). Testcase is e.g. tdf#89186-1 (with minimal
formula size disabled again).

Change-Id: If7e2a972898c358dc8f80cf238f8d5ccf541f16d
Reviewed-on: https://gerrit.libreoffice.org/56525
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
2018-06-28 01:13:50 +02:00
Stephan Bergmann
dc952845ca Suspiciously, copy ops of Bundle-derived classes didn't copy Bundle::mnColor
Lets assume that that was an oversight, and change all those user-provided
copy assignment ops into implicitly-defined ones (that will now also copy
Bundle::mnColor).  Avoids -Wdeprecated-copy with GCC trunk towards GCC 9, as the
copy ctors were already left implicitly-defined (and so already copied
Bundle::mnColor).

Change-Id: I43e323b55d059c7afecaddab0d777b6df47182c8
Reviewed-on: https://gerrit.libreoffice.org/56539
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2018-06-27 22:19:01 +02:00
Arkadiy Illarionov
f17eb4c84d tdf#96099 Remove *HandlerPtr typedefs in writerfilter
See https://bugs.documentfoundation.org/show_bug.cgi?id=96099#c44

Change-Id: Ib2d828a66bc14faa4f341eec9210e1effdebd7c1
Reviewed-on: https://gerrit.libreoffice.org/56541
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-27 22:10:33 +02:00
Tor Lillqvist
33af2a8e97 This assert() seems superfluous
The assertion fails when one reproduces tdf#69460 but removing the
assert() does not cause any problems.

Change-Id: Idaa5d41a0cf76073e406d7152fc3b4e9ebd3e876
Reviewed-on: https://gerrit.libreoffice.org/56538
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
2018-06-27 21:12:46 +02:00
Eike Rathke
d9388ab070 Check string length before copying, tdf#115007 follow-up
Change-Id: I50b842afc505ce603225fb2d25281cc8e9240200
Reviewed-on: https://gerrit.libreoffice.org/56537
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
2018-06-27 20:08:08 +02:00
Stephan Bergmann
40e5ad1179 Break HashedStyle::RefCount out into new RefCountedHashdStyle
...as that member is only used in m_aIdToStyle, and it was confusing how the
user-provided HashedStyle copy ctor initializes it to zero while the implicitly
defined copy assignment op copies it.  (And existence of only one of those
copy functions also triggered -Wdeprecated-copy with GCC trunk towards GCC 9.)

Change-Id: I176ef95af8780fefe21e53f61fd66f5e2d9156ab
Reviewed-on: https://gerrit.libreoffice.org/56528
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2018-06-27 18:39:33 +02:00
Miklos Vajna
1e533f69f0 tdf#102960 vcl opengl: fix missing support for polygon track frames
Which is used in e.g. the Calc cell border dialog. The approach is
similar to commit 60790935cc (tdf#96657 -
vcl opengl: implement invert: Track Frame., 2016-01-09) but that one was
for rectangles, this one is for polygons.

Change-Id: Ib1feebab2d14f4450fee0afe96afcea906965fdb
Reviewed-on: https://gerrit.libreoffice.org/56534
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins
2018-06-27 17:45:10 +02:00
Tor Lillqvist
fa7ebc82bb This should be a const NSString* so that we can pass such variables to it
Change-Id: I017c100f83faac64ad3a8430010c226b438e6a07
Reviewed-on: https://gerrit.libreoffice.org/56529
Tested-by: Jenkins
Reviewed-by: Tor Lillqvist <tml@collabora.com>
2018-06-27 17:17:53 +02:00
andreas kainz
a29b101897 tdf#114425 add Page Settings to writer and calc NB
add to NB grouped (full and compact) in Edit group
cause Format is only Format selected text but not Format page.
Edit group is most generic group that fit's for page selection

Change-Id: I49af8ff5b560655d43c55818da391a768d26ef04
Reviewed-on: https://gerrit.libreoffice.org/56430
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-06-27 16:01:24 +02:00
Tomaž Vajngerl
1a651c3fff vcl: prefer OpenType font feature description over Graphite
If the fon supports Graphite features we can get the feature
descriptions from the font, but this descriptions are mostly
limited (available in a limited number of languages), so it is
better to only use this descriptions only if the feature does not
have an OpenType description available.

Change-Id: Ic0b81cb8cc22e1e73f30e6a960167cfa982fbfc7
Reviewed-on: https://gerrit.libreoffice.org/56318
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2018-06-27 15:11:00 +02:00
Tor Lillqvist
387f00311a tdf#53282: Leave (our own) full-screen mode when the AquaSalFrame is dying
Change-Id: I85243dff96b9090de0fd55564b44abb6142d9ac2
Reviewed-on: https://gerrit.libreoffice.org/56524
Tested-by: Jenkins
Reviewed-by: Tor Lillqvist <tml@collabora.com>
2018-06-27 14:52:01 +02:00
Noel Grandin
e5c8f2ba40 loplugin:useuniqueptr in ScUnoAddInCollection
Change-Id: Ic38eab88f68952549af3417c3e85020877c95446
Reviewed-on: https://gerrit.libreoffice.org/56498
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-27 13:51:58 +02:00
Henry Castro
b965280309 tdf#118308: Incorrect Paste Special dialog when ...
pasting content from another cell

In tiled rendering case (headless) each window has its own clipboard,
otherwise exists a unique clipboard (UNO service).

Change-Id: I7d5d0b085faeaffa3fc0a80914fbe9349f4aa402
Reviewed-on: https://gerrit.libreoffice.org/56508
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
2018-06-27 13:44:07 +02:00
Tomaž Vajngerl
b262e6e5da xmloff: prevent embedding same font files with checksumming
When embedding fonts to ODF documents we can get into a situation
where the same font is embedded multiple times and this wastes
space. To prevent this we calculate the checksum of the font first
and track the fonts that are embedded. When the font has the same
checksum, the path to already embedded font is returned.

Change-Id: I087c42d9a63b1697b9b4ea985dbce2fda760a996
Reviewed-on: https://gerrit.libreoffice.org/56507
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2018-06-27 12:57:22 +02:00
Tomaž Vajngerl
eeef4ab2f4 xmloff: cleanup XMLFontAutoStylePool, use range for, std::pair
Change-Id: I5c775b8bc5cef3e14404f119e1ffb06334022003
Reviewed-on: https://gerrit.libreoffice.org/56506
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2018-06-27 12:57:04 +02:00
Noel Grandin
a980305186 loplugin:useuniqueptr in ScConsData
Change-Id: I0f9392d95ec2887ee62d1486f63600693a8b4dca
Reviewed-on: https://gerrit.libreoffice.org/56497
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-27 11:37:01 +02:00
László Németh
4fc072bf18 locale hu_HU.xml: add date formats with suffixes, articles
and spelling out using the new NatNum12 modifier
to support common "in", "on", "from", "to", etc. date
suffix variants, "dash-free" spell out format of years
of the new Hungarian orthographical reform, and months
with articles.

For example, "in 2018" is "2018-ban", "in 2019" is "2019-ben"
in Hungarian because of "vowel harmony", the word stem
dependent fluctuation of the suffix variants, a frequent
linguistic feature of other agglutinative languages, too,
including Estonian, Finnish and Turkish.

Note: some of the new date formats will work correctly only
with the upcoming update of the external libnumbertext.

Note 2: add also alternative (real) abbreviated month names,
because the default abbreviated month names are Roman
numbers.

Change-Id: Ibb33ff6a627b8e27fd02388653e3b33ebd446a10
Reviewed-on: https://gerrit.libreoffice.org/55637
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
2018-06-27 11:23:48 +02:00
Noel Grandin
add887972d loplugin:useuniqueptr in OutlineBulletDlg and SdPresLayoutTemplateDlg
Change-Id: I4bd0a9dd7be5b6ad7c1307f73c89ab0ee915ce9f
Reviewed-on: https://gerrit.libreoffice.org/56495
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-06-27 11:14:12 +02:00
Stephan Bergmann
47e334edd0 Null deref in ScDocument::IsClipboardSource
Appears to be a regression introduced with
b6b1ded0a0 "sc: replace ScCaptionPtr with
std::shared_ptr, tdf#117997, tdf#117228".  Reported e.g. as
<https://ci.libreoffice.org//job/lo_ubsan/954/console> during
CppunitTest_sc_subsequent_export_test and CppunitTest_sc_ucalc:

> /sc/source/core/data/document.cxx:2574:28: runtime error: member call on null pointer of type 'ScDocShell'
>     #0 0x2b9971542d76 in ScDocument::IsClipboardSource() const /sc/source/core/data/document.cxx:2574:28
>     #1 0x2b99722b7762 in ScFormulaCell::ScFormulaCell(ScFormulaCell const&, ScDocument&, ScAddress const&, ScCloneFlags) /sc/source/core/data/formulacell.cxx:844:14
>     #2 0x2b996fee35b0 in ScCellValue::assign(ScCellValue const&, ScDocument&, ScCloneFlags) /sc/source/core/data/cellvalue.cxx:395:29
>     #3 0x2b997312db22 in ScChangeTrack::Clone(ScDocument*) const /sc/source/core/tool/chgtrack.cxx:4565:25
>     #4 0x2b99cd1728be in XclExpChangeTrack::CreateTempChangeTrack() /sc/source/filter/xcl97/XclExpChangeTrack.cxx:1561:12
>     #5 0x2b99cd169a74 in XclExpChangeTrack::XclExpChangeTrack(XclExpRoot const&) /sc/source/filter/xcl97/XclExpChangeTrack.cxx:1418:39
>     #6 0x2b99cab087c0 in ExcDocument::ReadDoc() /sc/source/filter/excel/excdoc.cxx:788:41
>     #7 0x2b99cadddc2e in ExportBiff5::Write() /sc/source/filter/excel/expop2.cxx:112:5
>     #8 0x2b99cab8ee83 in lcl_ExportExcelBiff(SfxMedium&, ScDocument*, SvStream*, bool, unsigned short) /sc/source/filter/excel/excel.cxx:181:16
>     #9 0x2b99cab8d342 in ScFormatFilterPluginImpl::ScExportExcel5(SfxMedium&, ScDocument*, ExportFormatExcel, unsigned short) /sc/source/filter/excel/excel.cxx:219:16
>     #10 0x2b99768df9d2 in ScDocShell::ConvertTo(SfxMedium&) /sc/source/ui/docshell/docsh.cxx:2419:30
>     #11 0x2b998a7e2550 in SfxObjectShell::SaveTo_Impl(SfxMedium&, SfxItemSet const*) /sfx2/source/doc/objstor.cxx:1513:19
>     #12 0x2b998a7fe5e2 in SfxObjectShell::DoSaveAs(SfxMedium&) /sfx2/source/doc/objstor.cxx:1900:17
>     #13 0x2b9985a31bb6 in ScBootstrapFixture::saveAndReload(ScDocShell*, rtl::OUString const&, rtl::OUString const&, rtl::OUString const&, SfxFilterFlags) /sc/qa/unit/helper/qahelper.cxx:664:5
>     #14 0x2b9985a34593 in ScBootstrapFixture::saveAndReload(ScDocShell*, int) /sc/qa/unit/helper/qahelper.cxx:686:28
>     #15 0x2b9956fca835 in ScExportTest::testTrackChangesSimpleXLSX() /sc/qa/unit/subsequent_export-test.cxx:2618:29
>     #16 0x2b9957203058 in void std::_Mem_fn_base<void (ScExportTest::*)(), true>::operator()<, void>(ScExportTest*) const /home/tdf/lode/opt_private/lib/gcc/x86_64-unknown-linux-gnu/5.2.0/../../../../include/c++/5.2.0/functional:600:11
>     #17 0x2b9957202d63 in void std::_Bind<std::_Mem_fn<void (ScExportTest::*)()> (ScExportTest*)>::__call<void, , 0ul>(std::tuple<>&&, std::_Index_tuple<0ul>) /home/tdf/lode/opt_private/lib/gcc/x86_64-unknown-linux-gnu/5.2.0/../../../../include/c++/5.2.0/functional:1073:11
>     #18 0x2b9957202771 in void std::_Bind<std::_Mem_fn<void (ScExportTest::*)()> (ScExportTest*)>::operator()<, void>() /home/tdf/lode/opt_private/lib/gcc/x86_64-unknown-linux-gnu/5.2.0/../../../../include/c++/5.2.0/functional:1131:11
>     #19 0x2b9957201149 in std::_Function_handler<void (), std::_Bind<std::_Mem_fn<void (ScExportTest::*)()> (ScExportTest*)> >::_M_invoke(std::_Any_data const&) /home/tdf/lode/opt_private/lib/gcc/x86_64-unknown-linux-gnu/5.2.0/../../../../include/c++/5.2.0/functional:1871:2
>     #20 0x2b9957203ad0 in std::function<void ()>::operator()() const /home/tdf/lode/opt_private/lib/gcc/x86_64-unknown-linux-gnu/5.2.0/../../../../include/c++/5.2.0/functional:2271:14
>     #21 0x2b99571ff885 in CppUnit::TestCaller<ScExportTest>::runTest() /workdir/UnpackedTarball/cppunit/include/cppunit/TestCaller.h:175:7
>     #22 0x2b990c338e7d in CppUnit::TestCaseMethodFunctor::operator()() const /workdir/UnpackedTarball/cppunit/src/cppunit/TestCase.cpp:32:5
>     #23 0x2b99294cb4d6 in (anonymous namespace)::Protector::protect(CppUnit::Functor const&, CppUnit::ProtectorContext const&) /test/source/vclbootstrapprotector.cxx:48:14
>     #24 0x2b990c2f7947 in CppUnit::ProtectorChain::ProtectFunctor::operator()() const /workdir/UnpackedTarball/cppunit/src/cppunit/ProtectorChain.cpp:20:12
>     #25 0x2b991bcad7a6 in (anonymous namespace)::Prot::protect(CppUnit::Functor const&, CppUnit::ProtectorContext const&) /unotest/source/cpp/unobootstrapprotector/unobootstrapprotector.cxx:89:12
>     #26 0x2b990c2f7947 in CppUnit::ProtectorChain::ProtectFunctor::operator()() const /workdir/UnpackedTarball/cppunit/src/cppunit/ProtectorChain.cpp:20:12
>     #27 0x2b9917d39e13 in (anonymous namespace)::Prot::protect(CppUnit::Functor const&, CppUnit::ProtectorContext const&) /unotest/source/cpp/unoexceptionprotector/unoexceptionprotector.cxx:63:16
>     #28 0x2b990c2f7947 in CppUnit::ProtectorChain::ProtectFunctor::operator()() const /workdir/UnpackedTarball/cppunit/src/cppunit/ProtectorChain.cpp:20:12
>     #29 0x2b990c2732df in CppUnit::DefaultProtector::protect(CppUnit::Functor const&, CppUnit::ProtectorContext const&) /workdir/UnpackedTarball/cppunit/src/cppunit/DefaultProtector.cpp:15:12
>     #30 0x2b990c2f7947 in CppUnit::ProtectorChain::ProtectFunctor::operator()() const /workdir/UnpackedTarball/cppunit/src/cppunit/ProtectorChain.cpp:20:12
>     #31 0x2b990c2edb70 in CppUnit::ProtectorChain::protect(CppUnit::Functor const&, CppUnit::ProtectorContext const&) /workdir/UnpackedTarball/cppunit/src/cppunit/ProtectorChain.cpp:86:18
>     #32 0x2b990c3c2ccb in CppUnit::TestResult::protect(CppUnit::Functor const&, CppUnit::Test*, std::string const&) /workdir/UnpackedTarball/cppunit/src/cppunit/TestResult.cpp:182:10
>     #33 0x2b990c335efd in CppUnit::TestCase::run(CppUnit::TestResult*) /workdir/UnpackedTarball/cppunit/src/cppunit/TestCase.cpp:91:5
>     #34 0x2b990c33c0b3 in CppUnit::TestComposite::doRunChildTests(CppUnit::TestResult*) /workdir/UnpackedTarball/cppunit/src/cppunit/TestComposite.cpp:64:5
>     #35 0x2b990c33b01d in CppUnit::TestComposite::run(CppUnit::TestResult*) /workdir/UnpackedTarball/cppunit/src/cppunit/TestComposite.cpp:23:3
>     #36 0x2b990c33c0b3 in CppUnit::TestComposite::doRunChildTests(CppUnit::TestResult*) /workdir/UnpackedTarball/cppunit/src/cppunit/TestComposite.cpp:64:5
>     #37 0x2b990c33b01d in CppUnit::TestComposite::run(CppUnit::TestResult*) /workdir/UnpackedTarball/cppunit/src/cppunit/TestComposite.cpp:23:3
>     #38 0x2b990c402686 in CppUnit::TestRunner::WrappingSuite::run(CppUnit::TestResult*) /workdir/UnpackedTarball/cppunit/src/cppunit/TestRunner.cpp:47:5
>     #39 0x2b990c3c0e19 in CppUnit::TestResult::runTest(CppUnit::Test*) /workdir/UnpackedTarball/cppunit/src/cppunit/TestResult.cpp:149:3
>     #40 0x2b990c403b27 in CppUnit::TestRunner::run(CppUnit::TestResult&, std::string const&) /workdir/UnpackedTarball/cppunit/src/cppunit/TestRunner.cpp:96:3
>     #41 0x515d6d in (anonymous namespace)::ProtectedFixtureFunctor::run() const /sal/cppunittester/cppunittester.cxx:315:13
>     #42 0x51071f in sal_main() /sal/cppunittester/cppunittester.cxx:465:14
>     #43 0x50e832 in main /sal/cppunittester/cppunittester.cxx:372:1
>     #44 0x2b990e038444 in __libc_start_main (/lib64/libc.so.6+0x22444)
>     #45 0x433494 in _start (/workdir/LinkTarget/Executable/cppunittester+0x433494)

Change-Id: Id6cbc92844c89d48daecf8fad0b8899fcd087eaa
Reviewed-on: https://gerrit.libreoffice.org/56513
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2018-06-27 11:06:06 +02:00
Stephan Bergmann
6ebc026e34 salhelper::SingletonRef copy ctor is broken, mark as deleted
If that (implicitly defined) ctor had ever been used, it would have failed to
increment m_nRef, so could have caused m_pInstance to be deleted too early.  So
better mark it as deleted.  (LO itself apparently didn't call it.  If 3rd party
code would want to call it after all, it could be changed from deleted to
properly user-provided, incrementing m_nRef, in the future.)

The implicitly defined copy assignment op happens to already do the right thing,
and /is/ used, e.g., during

>         m_aNode = _rhs.m_aNode;

in DriversConfig::operator= (connectivity/source/commontools/DriversConfig.cxx).
Mark it as defaulted to avoid -Wdeprecated-copy with GCC trunk towards GCC 9.

Change-Id: I527123d9de837c311d30a270feb09dc173d9f411
Reviewed-on: https://gerrit.libreoffice.org/56512
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2018-06-27 10:57:59 +02:00