Age | Commit message (Collapse) | Author |
|
Change-Id: If326175d571d15752efd1b63df45b2bc785f7541
Reviewed-on: https://gerrit.libreoffice.org/61653
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
This mostly affects explicitly defaulted functions that had recently been user-
declared to silence new GCC trunk -Wdeprecated-copy. It is not entirely clear
to me what the best approach is overall, see my mail
<http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20181001/245321.html>
"Re: r343285 - [cxx2a] P0641R2: (Some) type mismatches on defaulted functions
only", but lets just explicitly delete those functions for now.
Change-Id: If8c72f612f67a8feb8b03c2fb988c807e704ef03
Reviewed-on: https://gerrit.libreoffice.org/61259
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
we've been using the normal memory allocator instead of the sal slab
allocator ever since
commit bc6a5d8e79e7d0e7d75ac107aa8e6aa275e434e9
Date: Wed Nov 15 16:52:44 2017 +0530
Disable custom allocator
Change-Id: I3383962cedb85d56fbec695398901f6ff7057651
Reviewed-on: https://gerrit.libreoffice.org/58577
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
mostly this seems to be fallout from
commit 089a4f245325a5be5cd5951d85305d791b4d9cb6
Date: Mon Aug 6 14:55:04 2018 +0200
remove Calc's software interpreter
Change-Id: Ib48696e724969a28a7d41155ba1573fd9cd58be6
Reviewed-on: https://gerrit.libreoffice.org/58869
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
...by explicitly defaulting the copy/move functions (and, where needed in turn,
also a default ctor) for classes that have a user-declared dtor that does
nothing other than an implicitly-defined one would do, but needs to be user-
declared because it is virtual and potentially serves as a key function to
emit the vtable, or is non-public, etc.; and by removing explicitly user-
provided functions that do the same as their implicitly-defined counterparts,
but may prevent implicitly declared copy functions from being defined as non-
deleted in the future. (Even if such a user-provided function was declared
non-inline in an include file, the apparently-used implicitly-defined copy
functions are already include, so why bother with non-inline functions.)
Change-Id: I4efe3eb088e5f9096be87dd8240504768755112b
Reviewed-on: https://gerrit.libreoffice.org/58096
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Found with bin/find-unneeded-includes
Only removal proposals are dealt with here
and a bit of fallout management.
Change-Id: I347377d7bb2a0cf8acda1ad28b5cc79bb954ab7d
Reviewed-on: https://gerrit.libreoffice.org/55726
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
|
|
from header to source file.
Revert "no matching function for call to 'intrusive_ptr_add_ref'"
This reverts commit a49be6bd585ac4610cbf04ca3525f2d90a770367.
Also reverts small part of
commit fd3fc84e590fc84f62ce3bace668fe40a25f54b5
Change-Id: I5b48cf7e9f830ec411202c5364d0caac76449095
Reviewed-on: https://gerrit.libreoffice.org/47680
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...with clang-cl, in instantiation of member function
'boost::intrusive_ptr<const ScMatrix>::intrusive_ptr' requested at
sc/inc/token.hxx(291,20):
class SC_DLLPUBLIC ScMatrixCellResultToken : public formula::FormulaToken
Change-Id: Id8485d937744759ddbe36e8508c54e2a996df4d2
Reviewed-on: https://gerrit.libreoffice.org/47672
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
remove default-able copy-ctors/assignments
Change-Id: I40255393899275a5825ffef4df3f08c6ce5348c2
Reviewed-on: https://gerrit.libreoffice.org/47140
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
ScJumpMatrixToken unconditionally deletes the ScJumpMatrix pointer it
receives. But it's copy constructor also just copies that pointer,
meaning that we could end up freeing that pointer twice.
ScJumpMatrix has no copy constructor, so I just managed it via
shared_ptr.
Change-Id: I9cf13312afb4f2869fdc878e5f34060614e31842
Reviewed-on: https://gerrit.libreoffice.org/41728
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I8abc44b9f0bcd5e39fd8a94abafaf80aadcd26ca
Reviewed-on: https://gerrit.libreoffice.org/41496
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Icd077a82ab0c951667a6566d5c5d47084c46d20d
|
|
always use a 8bit enum for StackVar
correct StackVar types for opcode opSep and opClose in unit test
and test them.
Change-Id: I89fac97d791eb12667ce5218ff31e798c8711e10
Reviewed-on: https://gerrit.libreoffice.org/36000
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
...from function definitions occurring within class definitions. Done with
a rewriting Clang plugin (to be pushed later).
Change-Id: I9c6f2818a57ccdb361548895a7743107cbacdff8
Reviewed-on: https://gerrit.libreoffice.org/34874
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I877bc0927aff7d175c1978fdf3db55d2e048d3b9
Reviewed-on: https://gerrit.libreoffice.org/33510
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
So also "empty" result cells with a number format applied are displayed empty,
instead of 1899-12-30 for example.
Change-Id: I5280d0213b8a809a04c5cb152ded58028706493a
|
|
The issue of 362d4f0cd4e50111edfae9d30c90602c37ed65a2 "Explicitly mark
overriding destructors as 'virtual'" appears to no longer be a problem with
MSVC 2013.
(The little change in the rewriting code of compilerplugins/clang/override.cxx
was necessary to prevent an endless loop when adding "override" to
OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager();
in chart2/source/inc/LifeTime.hxx, getting stuck in the leading
OOO_DLLPUBLIC_CHARTTOOLS macro. Can't remember what that
isAtEndOfImmediateMacroExpansion thing was originally necessary for, anyway.)
Change-Id: I534c634504d7216b9bb632c2775c04eaf27e927e
|
|
Change-Id: Ic378126a30be853d10fe174c451cee3c6ded404f
Reviewed-on: https://gerrit.libreoffice.org/25109
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
replace the old declare and don't implement pattern
with C++11 delete keyword
Change-Id: I96c71d512d8dab4ad7c806c2e871604163fc49c1
Reviewed-on: https://gerrit.libreoffice.org/24399
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: David Tardon <dtardon@redhat.com>
|
|
Change-Id: I75c483972db787ea9c941eaac0df023f1929b09c
|
|
Change-Id: I0d62536caa6eb455473a755067abc585662cd9a5
|
|
Change-Id: I765d2a600f9c57da50c85354688e3ae796750d94
|
|
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
|
|
Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
|
|
To diustinguish from formula::FormulaToken::IsGlobal() virtual dummy
implementation error message and spot code that possibly makes false
assumptions about svIndex tokens.
Change-Id: I5c4a62dc5fdf3a97b7a4d143ce1e419f15a8fdb0
|
|
... including reference update as named expressions don't hold RPN.
Change-Id: Ieea19e24f1dcf71d489ff0249bc7d3644e4f0d2b
|
|
Change-Id: I541577e0b99b0144a755e5755adc890c0ca8d204
|
|
Change-Id: Iaf76cf298dd2f9f9b8d9fb56ecb9e7b29fdc6ca1
|
|
According to https://msdn.microsoft.com/en-us/library/dd906358.aspx
Take them by their words.
Change-Id: I832c860695833ad04338ebd56c5b56805f1aa0a0
|
|
Change-Id: Iab74e027d9929bc100ad92c2126c17fca1e2e46a
|
|
Change-Id: I7f8cdba40d1d455927557c31867b822afb9dff7e
|
|
Change-Id: Id02fb58bbb9419d7c70d8265cb990ffc8ebc68ec
|
|
Change-Id: I507b6dfc0ce1a7b0f28142b05e15834ad6e6ac80
|
|
Change-Id: I4a43ea664e9db2bb44e0e7f089a71abf6f572b1e
|
|
Change-Id: Ie35d717fd55e6bbde958d3f2be4dac5f489467f9
|
|
Change-Id: Id6f7f0fbc120072cf6a61229838e39c4a53aeee7
|
|
Change-Id: Ieeca9fe957e7bc6a4cf9d7d6ac57f9ed150aab78
|
|
Change-Id: Ib5ed5a6bd598df660c3b865782e6a639e2c657ee
|
|
Change-Id: I153abe3a9c7ff784da16785fd9da8045c3581d8a
|
|
comment it's unnecessary
Change-Id: Iebf2d1dbed1f9c1e2e09144f47e73d9628355f2e
|
|
...now that ScToken has dissolved into FormulaToken
Change-Id: Idd28842125698fde62df7f70079f73105b918bc0
|
|
...to keep UBSan from complaining about illegal downcasts from FormulaToken to
ScToken (which were practically harmless, given that ScToken did not add any
data members).
Change-Id: I8fdc026eec363442cc4b720b71d65c972b4a4446
|
|
Change-Id: I03daf7bb68ed33d9610b7ef19d3bd6c18dc8170f
|
|
Change-Id: I20eb3b44d540f459cd1902a7a3b6a5c867e5bc07
|
|
...so that default implementations can return nullptr without further ado
Change-Id: Idc8778fe868efa00b1de597a54aa07a2e6fa3bc0
|
|
Change-Id: I2c81f24a27ede9922ed8ff16cf0ea4af824e9e59
|
|
Change-Id: I94f330d31f142bbeea2274597c855481cb311e0c
|
|
...and rename to extendRangeReference to avoid confusion with
formula::FormulaCompiler::ExtendRangeReference.
Change-Id: Ifcad309c14e04a0e37c80ca44462da587387241d
|
|
Change-Id: I5f27054ef2f9fdc4d0be0afe91ad31d053976bcf
|
|
Change-Id: Ib720bc4b3bfff8e428d14d65ff73144db82c60e0
|