Age | Commit message (Collapse) | Author |
|
Change-Id: If9ce8a094af878497e980cdcfaf11604d613e5b8
|
|
Change-Id: I85cfe8123a6ef1c3a1aa1b1085e2961055dfa907
|
|
Occurred in CSV import without "detect special numbers" activated for data like
,.,
where the . dot resulted in a numeric cell value 0
Change-Id: Ie715d7a8ed02196b59968a92919ad286b3bedf64
|
|
Change-Id: I376040a22b17037893704afbce2b651f8cf2c074
|
|
... and do not test up to three characters if the string is shorter..
Change-Id: I52b74cbde10c14c991cc8c68760c87c1e08ab7e4
|
|
Change-Id: Idb7996e9063955dbb8126243670bdc30bd8dd97a
|
|
Change-Id: Ic23ba93667f5de4b2cdb7868f9e3fc8cf8bacbd3
Reviewed-on: https://gerrit.libreoffice.org/26626
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
Change-Id: I8b0fe9354232a7b60f3605fa6f90f6741f7bf683
Reviewed-on: https://gerrit.libreoffice.org/26537
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: jan iversen <jani@documentfoundation.org>
|
|
This seems to be a good idea based on several discussions
in the project. In the end catching SEH exceptions is just
going to cause strange platform dependent bahavior.
This patch is based on on
http://thread.gmane.org/gmane.comp.documentfoundation.libreoffice.scm/39102/focus=55516
and includes some additional cleanup of the sal signal code.
Change-Id: Iedc998e37e6495afec445eccb60fa1c2b1a7defd
Reviewed-on: https://gerrit.libreoffice.org/26497
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I67eac95686678e6f5a2d60798535b2c65a9ba5d7
|
|
Change-Id: I4ead30fa1736a3a99fb3379e89367934c8df2809
Reviewed-on: https://gerrit.libreoffice.org/26444
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
This reverts commit a82e532ce006c54b2740de74d1da5d11307da7c1. That hack should
no longer be necessary now that the root cause for tdf#38913 has probably been
found and fixed with b5876bfcb69a65c87d602bae687b3c0634c0a1e7 "passing a NULL
pointer to fileno is not allowed".
Conflicts:
sal/osl/w32/salinit.cxx
Change-Id: Id3a3d3163945acc9b5d5e74a8d357dfa2ce63f37
Reviewed-on: https://gerrit.libreoffice.org/26411
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I94e79ae616d5cb9287e69544e283ce4d288a19ef
Reviewed-on: https://gerrit.libreoffice.org/26401
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
Also in:
- CppunitTest_sal_osl_security
- CppunitTest_sal_rtl_crc32
- CppunitTest_sal_osl_thread
- CppunitTest_sal_rtl_locale
- CppunitTest_sal_rtl_process
- CppunitTest_sal_rtl_random
Change-Id: I853457f4ddc387728ea4fa201c278c361369a591
Reviewed-on: https://gerrit.libreoffice.org/26085
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
...to avoid bogus -Werror=unused-result from trunk GCC in the macro expansion.
Change-Id: I227a0edfb22255c31d285609761dbefb4e50e09a
Reviewed-on: https://gerrit.libreoffice.org/26004
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: Ib42627c04b2c06e34f2dd108f8a04eaea916488a
|
|
Change-Id: Ib0594ba236284aa69d180fb72ce60435b679ea05
Reviewed-on: https://gerrit.libreoffice.org/25852
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
Change-Id: I8ff0e104aad045f3835dc8facc760a8339b1d088
|
|
avoid warning whether pid_t is long or int
Change-Id: I533175b53166cb233169902d277906f5ce979421
Signed-off-by: Richard PALO <richard@NetBSD.org>
Reviewed-on: https://gerrit.libreoffice.org/25593
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: If35da46c209746a86687e1c3f60f72f2f08aa681
|
|
Change-Id: I7cd31d5624682bf7bd021586769593499d2481af
Reviewed-on: https://gerrit.libreoffice.org/25204
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I642d7546059f1af993ab15eb3948949109df05c4
Reviewed-on: https://gerrit.libreoffice.org/25203
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I9070bae914d923171b6eaf8209de00922dc87be7
Reviewed-on: https://gerrit.libreoffice.org/25052
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
|
|
The LibreOffice signal handler overwrites the breakpad signal handler.
Just store the old handler and call that handler before we do any work
in our own handler.
Our own handler implementation is just insane as we are even trying to
take the SolarMutex which can easily deadlock.
Change-Id: I2401bba18701115561d00c7fc11138ead2a48205
Reviewed-on: https://gerrit.libreoffice.org/25022
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: Icfc4b73d5664c205ad642c660af5227f403d20ff
Reviewed-on: https://gerrit.libreoffice.org/25021
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Fix "fallthrough annotation in unreachable code". Clang is clever
enough to know that code after assert(false) is unreachable in a
non-NDEBUG build.
Seriously, micro-optimisation of the case that should never be
reached?
Change-Id: I10d7ae60895437be52c27aa93539d17e0f091ac4
|
|
Change-Id: Id8262e735b9f95d8b476a4939c362648e2a4328a
|
|
Change-Id: Ib67f7cb71255a32810f09a6b7e528f024bbef9df
|
|
...which (in LIBO_INTERNAL_ONLY) for Clang expands to [[clang::fallthrough]] in
preparation of enabling -Wimplicit-fallthrough. (This is only relevant for
C++11, as neither C nor old C++ has a way to annotate intended fallthroughs.)
Could use BOOST_FALLTHROUGH instead of introducing our own SAL_FALLTHROUGH, but
that would require adding back in dependencies on boost_headers to many
libraries where we carefully removed any remaining Boost dependencies only
recently. (At least make SAL_FALLTHROUGH strictly LIBO_INTERNAL_ONLY, so its
future evolution will not have any impact on the stable URE interface.) C++17
will have a proper [[fallthroug]], eventually removing the need for a macro
altogether.
Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
|
|
Change-Id: If0d87b6679765fc6d1f9300c6972845cf3742b9c
Reviewed-on: https://gerrit.libreoffice.org/24674
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I6e86cb7ffbd8c8cbf3cccaef656cf7b7e755e24f
|
|
are actually pointer vars.
Also convert from regex to normal code, so we can enable this
plugin all the time.
Change-Id: Ie36a25ecba61c18f99c77c77646d6459a443cbd1
Reviewed-on: https://gerrit.libreoffice.org/24391
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I0164df28e2d05f4f73bd640d3ca3cc3220b57df6
|
|
...to be set before including Carbon/Carbon.h? Was originally introduced with
c34f2e30bbad1a6b92308d0daabe413bd1bc161b "INTEGRATION: CWS pj51: #i61959# patch
applied", but might just be cargo cult these days.
Change-Id: I20cdc938d28e075614f6711a06ab785700833aff
Reviewed-on: https://gerrit.libreoffice.org/24293
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
(that started to cause loplugin:salbool after
13758a3d154e8e450fdfe8bcdeb6b3a03996c53a "SourceManager::isMacroArgExpansion has
only one param in older Clang")
Change-Id: Ibbdf4552d81385cf952e2d47d1fe23daf78824b5
|
|
check for local variables which follow our member field naming
convention, which is highly confusing
Change-Id: Idacedf7145d09843e96a584237b385f7662eea10
|
|
Change-Id: I6c656f991999791469015500aff1905fdb16ba65
|
|
Searched source for using declarations.
Checked if those symbols reappear in the source file,
even in comments or dead code but not in #include statements.
If they don't reappear, remove the declaration.
Remove includes whose symbol got removed.
Change-Id: Ibb77163f63c1120070e9518e3dc0a78c6c59fab0
Reviewed-on: https://gerrit.libreoffice.org/24148
Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Ic89d1c3be1939c8989ad5ee806980e8904f8e44d
Reviewed-on: https://gerrit.libreoffice.org/24069
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...when SIG_DFL happens to be defined as nullptr, by initializing the Signals
array to SIG_DFL in the first place
Change-Id: Ie9183540e2e8fd795aa288bf30266bc5d9af3e69
|
|
remove <boost/noncopyable.hpp> in pch and
remove boost from makefile if it was the only boost entry.
Change-Id: Icb945ae59c137571f4f63807601738eea5c3e831
Reviewed-on: https://gerrit.libreoffice.org/24061
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I0a8dc9591c00188d0a31feb784d14ec53f389f27
|
|
... in modules reportdesign to sdext
Replace with C++11 delete copy-constructur and
copy-assignment.
Remove boost/noncopyable.hpp includes.
Make some overloaded ctors explicit
(most in sd slidesorter).
Add deleted copy-assignment in sc/inc/chart2uno.hxx.
Change-Id: I21d4209f0ddb00063ca827474516a05ab4bb2f9a
Reviewed-on: https://gerrit.libreoffice.org/23970
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I496fdbab3e17c806c2d16513412c4c1b9120bc3c
|
|
The file contains their definitions before they are used anyway.
Change-Id: Iaa541ec9a12e4971c0cd4ba7cfb4790d328b7b63
|
|
3979afceecb98780549fda60c7f75c8015bb33d8 broke the build on Windows 7.
Change-Id: I142208c062d56d6a3daa8213a877411edde45e55
|
|
Change-Id: I7029f37639ead7a8b597ff39d1abcd295fab0551
|
|
Change-Id: I7c96181399f4d7e62d4aceca404b22d68f903513
Reviewed-on: https://gerrit.libreoffice.org/23754
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Including no keywords from extern "C" blocks
Change-Id: I87f2ed75888b51ec9e0cb75566bf7c2351b479b4
Reviewed-on: https://gerrit.libreoffice.org/23675
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: If5d3cb2242e08e559380a4e2c86c96dc6cdc0c71
|