summaryrefslogtreecommitdiffstats
path: root/testtools
Commit message (Collapse)AuthorAgeFilesLines
* use actual UNO enums in test..vbahelperNoel Grandin2017-04-031-1/+1
| | | | | | | Change-Id: I2e38462045efedc31903589a3fa4e6f62d74d8de Reviewed-on: https://gerrit.libreoffice.org/36042 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Fix typosAndrea Gelmini2017-03-251-1/+1
| | | | | | | Change-Id: Ieea40e1b7282267157810f9f58ca083e68ae6715 Reviewed-on: https://gerrit.libreoffice.org/35659 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
* remove some unnecessary OUStringBuffer usageNoel Grandin2017-02-211-23/+5
| | | | | | | Change-Id: Iae9146a3be569107019d9c5af404b94e51e76cd5 Reviewed-on: https://gerrit.libreoffice.org/34469 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Some simplifications, using UNO_QUERY_THROWStephan Bergmann2017-02-152-8/+2
| | | | | | | Change-Id: Iaf7b4695e7930b8e7fa5dd3736d19744afb52fa3 Reviewed-on: https://gerrit.libreoffice.org/34254 Tested-by: Stephan Bergmann <sbergman@redhat.com> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* Add missing #includesStephan Bergmann2017-02-061-0/+1
| | | | | | | | | ...and remove some unncessary using directives/declarations, in preparation of removing now-unnecessary #includes from cppumaker-generated files, post e57ca02849c3d87142ff5ff9099a212e72b8139c "Remove dynamic exception specifications". Change-Id: Iaf1f268871e2ee1d1c76cf90f03557527ebc9067
* Remove dynamic exception specificationsStephan Bergmann2017-01-266-235/+154
| | | | | | | | | | | | | | | | | | | | | | | | | ...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at <https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html> "Dynamic Exception Specifications" for details. Most changes have been done automatically by the rewriting loplugin:dynexcspec (after enabling the rewriting mode, to be committed shortly). The way it only removes exception specs from declarations if it also sees a definition, it identified some dead declarations-w/o-definitions (that have been removed manually) and some cases where a definition appeared in multiple include files (which have also been cleaned up manually). There's also been cases of macro paramters (that were used to abstract over exception specs) that have become unused now (and been removed). Furthermore, some code needed to be cleaned up manually (avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no configurations available that would actually build that code. Missing @throws documentation has not been applied in such manual clean-up. Change-Id: I3408691256c9b0c12bc5332de976743626e13960 Reviewed-on: https://gerrit.libreoffice.org/33574 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* Remove irrelevant SAL_INFOsStephan Bergmann2016-12-091-5/+0
| | | | Change-Id: Ieb58ed113c6a3e85a806ec57a6557e2d37bf9b06
* OSL_TRACE -> SAL in sw..ucbNoel Grandin2016-12-081-4/+2
| | | | | | | Change-Id: I18f5511f70232d91095ac8527a6c5883c36294f5 Reviewed-on: https://gerrit.libreoffice.org/31762 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unnecessaryoverride (dtors) in testtoolsStephan Bergmann2016-12-012-12/+0
| | | | Change-Id: If004c2c58cb92e211cea34b54c3c38c91ccb6e7d
* Change the shared bridgetest code from static to dynamic libraryStephan Bergmann2016-09-148-39/+73
| | | | | | | | | | ...so ASan builds do not complain about ODR violations due to multiple instances of vtable for testtools::bridgetest::CurrentContextChecker after a53808c0ed577468393aced90963af6496706959 "loplugin:dllprivate" removed the SAL_DLLPRIVATE from class CurrentContextChecker (which apparently /did/ have an impact with -fvisibility-ms-compat, as used by Linux Clang ASan/UBSan builds). Change-Id: Ifb1ba1819f7ea989300f4696d44f9599a1436563
* loplugin:dllprivateStephan Bergmann2016-09-131-1/+1
| | | | Change-Id: I1fe70a39c50aba8b84c117653185fc37dbbfeab0
* loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann2016-09-135-7/+7
| | | | | | | | | | | | | | | | | 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
* python macros aren't listed under scripting organizeCaolán McNamara2016-08-091-4/+4
| | | | | | | | | | | | since... commit deb989dd6d1f86e74864131be50ed92d8d43768c Author: Kenneth Koski <mechaxl@gmail.com> Date: Mon Feb 29 22:22:10 2016 -0600 blew away the uno.ByteSequence(str) path Change-Id: I8b73883c4f246ebafd2f810ca61b19da40f833e2
* new loplugin unnecessary overrideNoel Grandin2016-07-151-13/+0
| | | | | | | Change-Id: I88d3e33823d68745b98625050a8a274f9ef04bcb Reviewed-on: https://gerrit.libreoffice.org/27135 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* typo: emtpy → emptyChristian Lohmaier2016-07-041-1/+1
| | | | Change-Id: I60cdcdc7fc38c175243c071f34295efa8739b717
* Clean up uses of Any::getValue() in testtoolsStephan Bergmann2016-06-272-30/+11
| | | | Change-Id: I3440328371abfc223f39c6bd2994423bb9b38cf0
* Just use Any::operator <<= for sal_Unicode valuesStephan Bergmann2016-05-302-4/+3
| | | | | | | | ...now that sal_Unicode no longer clashes with sal_uInt16 on any platform (in LIBO_INTERNAL_ONLY code), after e16fa715c43dcdf836ce8c400b6d54eae87b627d "Handle wchar_t as native C++11 type on windows" Change-Id: Id423dd6235bf14823fa5611b804c0974edbe64b3
* While at it, delete Any functions on sal_Bool*Stephan Bergmann2016-05-041-3/+2
| | | | | | | (at least for LIBO_INTERNAL_ONLY), to help further reduce the occurrences of sal_Bool across the code base Change-Id: I70654a0cb56655984c717b7b894f26c9ab47536e
* Fix typos in codeAndrea Gelmini2016-05-011-1/+1
| | | | | | | Change-Id: I24f7b812fe625f3d91a49fb57701a36a8c68eb36 Reviewed-on: https://gerrit.libreoffice.org/24542 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* Use Sequence ctor taking initializer_listStephan Bergmann2016-04-211-29/+18
| | | | | | | | needed adapting loplugin:implicitboolconversion to not warn about Sequence<sal_Bool> arBool({true, false, true}); Change-Id: I971918aab7c958ef8f1e4e0548a84314e95f8325
* loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann2016-04-201-1/+1
| | | | Change-Id: If03cdd1fc6943c3783cedf8b735c8a29a3ecd0c7
* tdf#94306 replace boost::noncopyable in stoc to xmlsec..Jochen Nitschke2016-04-132-6/+7
| | | | | | | | | | | | | | | Replace with C++11 delete copy-constructur and copy-assignment. Remove boost/noncopyable.hpp includes. Add missing default ctors. With this commit there should be no users of boost::noncopyable left. Change-Id: I6b1e47824912a6a80cc3f00f34938ebc048d8975 Reviewed-on: https://gerrit.libreoffice.org/24051 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar2016-03-112-4/+1
| | | | | | | | | | | | I replaced the OSL_DEBUG_LEVEL > 1 to OSL_DEBUG_LEVEL > 0 conditionals. At some places i removed the logs containing OSL_DEBUG_LEVEL > 1 as OSL_DEBUG_LEVEL > 0 is compiled in production code Change-Id: I4466e7a2025b055f89fc525d7412f10bd4d8a07f Reviewed-on: https://gerrit.libreoffice.org/23067 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* Remove excess newlinesChris Sherlock2016-02-095-30/+0
| | | | | | | | | | | | | | | | | | | A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* testtools: disable MSVC warning C4503Michael Stahl2016-01-291-0/+3
| | | | | | | This was disabled by boost/config/compiler/visualc.hpp before commit 110d55cc11033d72bce908e65bbb6a9506ab39e5 Change-Id: Ic4757b0ab74c14fb49d7a0ec502253b667fae033
* Fix typosAndrea Gelmini2016-01-103-14/+14
| | | | | | | Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* use initialiser syntax for Sequence<OUString>Noel Grandin2015-11-152-6/+3
| | | | | | | | | | | | | | replaced using the script: git grep -lP 'Sequence.*OUString.*\(1\)' | xargs perl -0777 -pi -e "s/Sequence< OUString > (\w+)\(1\); .*\[0\] = (\S+);/Sequence< OUString > \1 { \2 };/g" Change-Id: I23688a91562051a8eed11fc2a85599545c285c34 Reviewed-on: https://gerrit.libreoffice.org/19967 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* new loplugin: memoryvarNoel Grandin2015-11-111-3/+3
| | | | | | | | | | detect when we can convert a new/delete sequence on a local variable to use std::unique_ptr Change-Id: Iecae4e4197eccdfacfce2eed39aa4a69e4a660bc Reviewed-on: https://gerrit.libreoffice.org/19884 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-103-15/+15
| | | | Change-Id: I4024816e38c0196ec502a5d39b8bd50cf6b142fb
* com::sun::star->css in testtools/Noel Grandin2015-11-097-185/+177
| | | | | | | Change-Id: I9a44e44b2478b007fba7f86eefcf4fa04d77b37b Reviewed-on: https://gerrit.libreoffice.org/19852 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* java: remove ((unnecessary)) parenthesesNoel Grandin2015-11-051-5/+5
| | | | Change-Id: I39fbe3a260c8dbfc203662c54eec4db064b88195
* yyyyyNoel Grandin2015-11-043-17/+13
| | | | Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
* convert Sequence<XInterface> constructions to use initializer listsNoel Grandin2015-10-301-2/+2
| | | | Change-Id: I66475190cc0f18465c56b94af7bc0d5a1ca81242
* coverity#1326344 Infinite loop is deliberateCaolán McNamara2015-10-251-0/+1
| | | | Change-Id: Ieeae14607127c442b987afc51f2b59692f5334ca
* cppcheck:variableScopeNoel Grandin2015-10-151-2/+2
| | | | | | | Change-Id: I9b671637fbe0f32e695d22b74bfb40a39a9fe884 Reviewed-on: https://gerrit.libreoffice.org/19364 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* cid#1326679 DLS: Dead local storeNoel Grandin2015-10-141-1/+0
| | | | Change-Id: I87748b6af4731e5c774962a7a26b008385146a5f
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-126-133/+133
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-121-2/+2
| | | | Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
* Fix typosAndrea Gelmini2015-09-291-1/+1
| | | | | | | Change-Id: Ifddd2533b767924791338555031b848c59980c52 Reviewed-on: https://gerrit.libreoffice.org/18989 Reviewed-by: Joren De Cuyper <jorendc@libreoffice.org> Tested-by: Joren De Cuyper <jorendc@libreoffice.org>
* testtools: tdf#88206 replace cppu::WeakImplHelper*Takeshi Abe2015-09-097-18/+16
| | | | | | | | | with the variadic variants. Change-Id: I6c77b9219c772f38005431fed7cd0f4132fab2b9 Reviewed-on: https://gerrit.libreoffice.org/18395 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* ppc64: using a fp register also consumes a gp register slotCaolan McNamara2015-07-105-0/+21
| | | | | | | Change-Id: Idf6f40081f4598c0fa9d1e10bdc208eae49e4cd1 Reviewed-on: https://gerrit.libreoffice.org/16936 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* loplugin:stringconstantStephan Bergmann2015-07-081-3/+3
| | | | Change-Id: Ia3275dc10ef18fd5978c7422b33012a5b1e4696c
* java:regulatize the order of 'final' and public/privateNoel Grandin2015-06-112-6/+6
| | | | | | | | | | Make the order be 'public static' or 'private static' Just makes the code nicer to read. Change-Id: I182424bda45a2d68642e5d04c6091d268ace1fe2 Reviewed-on: https://gerrit.libreoffice.org/16202 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2015-06-082-4/+4
| | | | Change-Id: I13f61e80a27e58fba347173e5d84464553312836
* cppcheck: noExplicitConstructorCaolán McNamara2015-05-264-4/+4
| | | | Change-Id: I955b04396d27e88ce228f180275955071da83ec6
* Clean up checks of sal_Bool valuesStephan Bergmann2015-05-081-2/+2
| | | | Change-Id: Ia05565077823d6efaeaeb35d3d80b51dd9a10d4c
* More loplugin:simplifyboolStephan Bergmann2015-04-271-1/+1
| | | | Change-Id: I8d8d6220ff597ce416042fe03d86a7e7bb0a3518
* Replace remaining getCppuType et al with cppu::UnoTypeStephan Bergmann2015-04-013-32/+32
| | | | Change-Id: I02c70b26b82d2d0357111a127aefc57d3be21320
* Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann2015-03-311-1/+1
| | | | Change-Id: If19e6222cc7dea2a3a4692d2d4cb43ccc46243a2
* V813: Decreased performanceCaolán McNamara2015-03-041-2/+2
| | | | Change-Id: I8a7528366156b288dc422b09cff0d5a32cde3c91