summaryrefslogtreecommitdiff
path: root/testtools
AgeCommit message (Collapse)AuthorFilesLines
2016-09-14Change the shared bridgetest code from static to dynamic libraryStephan Bergmann8-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
2016-09-13loplugin:dllprivateStephan Bergmann1-1/+1
Change-Id: I1fe70a39c50aba8b84c117653185fc37dbbfeab0
2016-09-13loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann5-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
2016-08-09python macros aren't listed under scripting organizeCaolán McNamara1-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
2016-07-15new loplugin unnecessary overrideNoel Grandin1-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>
2016-07-04typo: emtpy → emptyChristian Lohmaier1-1/+1
Change-Id: I60cdcdc7fc38c175243c071f34295efa8739b717
2016-06-27Clean up uses of Any::getValue() in testtoolsStephan Bergmann2-30/+11
Change-Id: I3440328371abfc223f39c6bd2994423bb9b38cf0
2016-05-30Just use Any::operator <<= for sal_Unicode valuesStephan Bergmann2-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
2016-05-04While at it, delete Any functions on sal_Bool*Stephan Bergmann1-3/+2
(at least for LIBO_INTERNAL_ONLY), to help further reduce the occurrences of sal_Bool across the code base Change-Id: I70654a0cb56655984c717b7b894f26c9ab47536e
2016-05-01Fix typos in codeAndrea Gelmini1-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>
2016-04-21Use Sequence ctor taking initializer_listStephan Bergmann1-29/+18
needed adapting loplugin:implicitboolconversion to not warn about Sequence<sal_Bool> arBool({true, false, true}); Change-Id: I971918aab7c958ef8f1e4e0548a84314e95f8325
2016-04-20loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann1-1/+1
Change-Id: If03cdd1fc6943c3783cedf8b735c8a29a3ecd0c7
2016-04-13tdf#94306 replace boost::noncopyable in stoc to xmlsec..Jochen Nitschke2-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>
2016-03-11tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar2-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>
2016-02-09Remove excess newlinesChris Sherlock5-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>
2016-01-29testtools: disable MSVC warning C4503Michael Stahl1-0/+3
This was disabled by boost/config/compiler/visualc.hpp before commit 110d55cc11033d72bce908e65bbb6a9506ab39e5 Change-Id: Ic4757b0ab74c14fb49d7a0ec502253b667fae033
2016-01-10Fix typosAndrea Gelmini3-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>
2015-11-15use initialiser syntax for Sequence<OUString>Noel Grandin2-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>
2015-11-11new loplugin: memoryvarNoel Grandin1-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>
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann3-15/+15
Change-Id: I4024816e38c0196ec502a5d39b8bd50cf6b142fb
2015-11-09com::sun::star->css in testtools/Noel Grandin7-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>
2015-11-05java: remove ((unnecessary)) parenthesesNoel Grandin1-5/+5
Change-Id: I39fbe3a260c8dbfc203662c54eec4db064b88195
2015-11-04yyyyyNoel Grandin3-17/+13
Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
2015-10-30convert Sequence<XInterface> constructions to use initializer listsNoel Grandin1-2/+2
Change-Id: I66475190cc0f18465c56b94af7bc0d5a1ca81242
2015-10-25coverity#1326344 Infinite loop is deliberateCaolán McNamara1-0/+1
Change-Id: Ieeae14607127c442b987afc51f2b59692f5334ca
2015-10-15cppcheck:variableScopeNoel Grandin1-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>
2015-10-14cid#1326679 DLS: Dead local storeNoel Grandin1-1/+0
Change-Id: I87748b6af4731e5c774962a7a26b008385146a5f
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann6-133/+133
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann1-2/+2
Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
2015-09-29Fix typosAndrea Gelmini1-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>
2015-09-09testtools: tdf#88206 replace cppu::WeakImplHelper*Takeshi Abe7-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>
2015-07-10ppc64: using a fp register also consumes a gp register slotCaolan McNamara5-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>
2015-07-08loplugin:stringconstantStephan Bergmann1-3/+3
Change-Id: Ia3275dc10ef18fd5978c7422b33012a5b1e4696c
2015-06-11java:regulatize the order of 'final' and public/privateNoel Grandin2-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>
2015-06-08loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2-4/+4
Change-Id: I13f61e80a27e58fba347173e5d84464553312836
2015-05-26cppcheck: noExplicitConstructorCaolán McNamara4-4/+4
Change-Id: I955b04396d27e88ce228f180275955071da83ec6
2015-05-08Clean up checks of sal_Bool valuesStephan Bergmann1-2/+2
Change-Id: Ia05565077823d6efaeaeb35d3d80b51dd9a10d4c
2015-04-27More loplugin:simplifyboolStephan Bergmann1-1/+1
Change-Id: I8d8d6220ff597ce416042fe03d86a7e7bb0a3518
2015-04-01Replace remaining getCppuType et al with cppu::UnoTypeStephan Bergmann3-32/+32
Change-Id: I02c70b26b82d2d0357111a127aefc57d3be21320
2015-03-31Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann1-1/+1
Change-Id: If19e6222cc7dea2a3a4692d2d4cb43ccc46243a2
2015-03-04V813: Decreased performanceCaolán McNamara1-2/+2
Change-Id: I8a7528366156b288dc422b09cff0d5a32cde3c91
2015-02-27testtools: mark CurrentContextChecker as DLLPRIVATEMichael Stahl1-1/+1
The problem is that currentcontextchecker.cxx is linked statically into 2 dynamic libraries, and with clang now using -fvisibility-ms-compat since commit f0aa1a78fb209310e8baef53c02f365fca518d11 the ASAN complains about ODR violation in testttols uno_test: ==1818==ERROR: AddressSanitizer: odr-violation (0x2aef08e9c600): [1] size=192 'vtable for testtools::bridgetest::CurrentContextChecker' testtools/source/bridgetest/currentcontextchecker.cxx [2] size=192 'vtable for testtools::bridgetest::CurrentContextChecker' testtools/source/bridgetest/currentcontextchecker.cxx Change-Id: Ie705788dbf035f7a9f64878c6dcf3cec9b7e7ba1
2015-02-07loplugin:deletedspecialStephan Bergmann1-2/+2
Change-Id: I9cdb0d7cb2bb9207923f2beedef0bbe44451fb6f
2015-01-20Some more loplugin:cstylecast: testtoolsStephan Bergmann1-4/+4
Change-Id: I28bce2f4bdbadef2d68130adc6fdc09c9840a005
2015-01-02boost::unordered_map->std::unordered_mapCaolán McNamara1-4/+2
you can get debug stl this way Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19
2014-12-18testtools: Use appropriate OUString functions on string constantsStephan Bergmann2-3/+2
Change-Id: I4258287401e76a2f37f88eeeba5aa11d693a01b8
2014-12-16testtools: Use appropriate OUString functions on string constantsStephan Bergmann2-6/+6
Change-Id: Iacc67a087321e9cf0f699f4eecb334eb7372b885
2014-12-12testtools: Use appropriate OUString functions on string constantsStephan Bergmann2-6/+3
Change-Id: I5cb34d95e2a59ff6a3f5189beb40b7dbf574dc29
2014-12-11java: reduce visibility of fields and methodsNoel Grandin3-4/+4
found by PMD Change-Id: Id6737916b68ccbdbdeec5d314747a38410923ac6 Reviewed-on: https://gerrit.libreoffice.org/13409 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-05java: remove some unused fieldsNoel Grandin1-5/+0
Change-Id: I1b6fb6b47439c448ac31983702772e2115c70d56
generated by cgit v1.2.3 (git 2.39.1) at 2024-04-28 14:55:19 +0000