summaryrefslogtreecommitdiff
path: root/stoc
AgeCommit message (Collapse)AuthorFilesLines
2016-08-05loplugin:countusersofdefaultparams in sfx2..svgioNoel Grandin1-1/+1
Change-Id: Ia01f3a9aa21c88df5fe5242ad4a3c0acbe68fda0 Reviewed-on: https://gerrit.libreoffice.org/27903 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-27improve passstuffbyref return analysisNoel Grandin1-1/+1
Change-Id: I4258bcc97273d8bb7a8c4879fac02a427f76e18c Reviewed-on: https://gerrit.libreoffice.org/27317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-18don't write past the end of the arrayMarkus Mohrhard1-1/+1
Found through http://crashreport.libreoffice.org/stats/crash_details/6ba2edbd-d268-4074-8b87-590a5694cc26 Change-Id: Ic95d472fd117c2d286fabb775ef49e287a2874fa
2016-07-11loplugin:nullptr: Better heuristic to determine code shared between C and C++Stephan Bergmann1-1/+1
Change-Id: I51e1c5fa4639e51fac90f92adf3d87d12960d589
2016-07-07loplugin:passstuffbyref also for {css::uno,rtl}::ReferenceStephan Bergmann1-2/+2
Change-Id: I5ef89d0f0f97dace7dc15dc07b3328177aab462b
2016-07-04remove deprecated Link functions from RegistryNoel Grandin1-37/+11
Change-Id: Id678d4b6263df86993b289e8439faac847dd7c5f Reviewed-on: https://gerrit.libreoffice.org/26855 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-21Clean up uses of Any::getValue() in stocStephan Bergmann7-108/+95
Change-Id: I455b708bfcc9d5aa71bb0ffc21efec205302169e
2016-06-19loplugin:salbool: Implicit conversions from non-Boolean fundamental typesStephan Bergmann1-22/+25
Change-Id: I67eac95686678e6f5a2d60798535b2c65a9ba5d7
2016-06-06remove some manual ref-countingNoel Grandin1-82/+76
triggered when I noticed a class doing acquire() in the constructor and then release() in the destructor. found mostly by git grep -n -B5 -e '->release()' Change-Id: I96e43a3d30ffd9ae9a34275f24cd914d8f7b026f Reviewed-on: https://gerrit.libreoffice.org/25806 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-30Some clean up of uses of css::uno::Any::setValueStephan Bergmann1-3/+2
Change-Id: I04e8aef35a6083b61d775c8eb3f96757da2b31bd
2016-05-30Just use Any::operator <<= for sal_Unicode valuesStephan Bergmann1-4/+4
...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-24Revert "remove some manual ref-counting"Noel Grandin1-76/+82
until I have a better understanding of the UNO reference counting. This reverts commit 111de438ea3e512a541281dc0716cc728ea8d152.
2016-05-24remove some manual ref-countingNoel Grandin1-82/+76
triggered when I noticed a class doing acquire() in the constructor and then release() in the destructor. found mostly by git grep -n -B5 -e '->release()' Change-Id: Ie1abeaed75c1f861df185e3bde680272dbadc97f Reviewed-on: https://gerrit.libreoffice.org/25363 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-19loplugin:unusedmethods in slideshow to svtoolsNoel Grandin1-15/+0
Change-Id: Icf0056e13c88d7d347e668adaeddd4ed72af85cf Reviewed-on: https://gerrit.libreoffice.org/25141 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-11clang-tidy modernize-loop-convert in scripting to svtoolsNoel Grandin1-6/+3
Change-Id: I98229d14109cf243839d632feabde1391ea9bad5 Reviewed-on: https://gerrit.libreoffice.org/24847 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-10Insert explicit "break" when falling through to empty next caseStephan Bergmann1-2/+3
...which itself only contains a "break" (or nothing at all at the end of the "switch"), as otherwise Clang -Wimplicit-fallthrough would warn about these. Change-Id: I25c1cf2ca74dfeba7ca0385ca8f1c1bf30bbf91b
2016-05-04While at it, delete Any functions on sal_Bool*Stephan Bergmann1-6/+3
(at least for LIBO_INTERNAL_ONLY), to help further reduce the occurrences of sal_Bool across the code base Change-Id: I70654a0cb56655984c717b7b894f26c9ab47536e
2016-05-01tdf#97499 Fixed containers parameters clearing #9tymyjan1-2/+2
Change-Id: I097106e1887be283320fc518ddafda76608b46ee Reviewed-on: https://gerrit.libreoffice.org/24521 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-04-26update loplugin stylepolice to check local pointers varsNoel Grandin1-4/+4
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>
2016-04-21loplugin:salbool: sal_Bool[] -> bool[]Stephan Bergmann1-13/+13
Change-Id: I3c5bf7a53c9ae173f8fce885ecf022f092aa43a9
2016-04-20loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann11-26/+26
Change-Id: I3a3295b538e79186914390b6da73cc8bc25419a5
2016-04-18clang-tidy performance-unnecessary-copy-initializationNoel Grandin2-12/+7
probably not much performance benefit, but it sure is good at identifying leftover intermediate variables from previous refactorings. Change-Id: I3ce16fe496ac2733c1cb0a35f74c0fc9193cc657 Reviewed-on: https://gerrit.libreoffice.org/24026 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-14clang-tidy performance-unnecessary-value-param in variousNoel Grandin2-3/+3
Change-Id: I7168d44dab8e6a8e37bb7920d744ff32f5e52907 Reviewed-on: https://gerrit.libreoffice.org/24019 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-14loplugin:passstuffbyref in variousNoel Grandin1-1/+1
Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
2016-04-13tdf#94306 replace boost::noncopyable in stoc to xmlsec..Jochen Nitschke8-32/+49
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-04-12Remove stray spaces from start of log messages, reduxStephan Bergmann1-1/+1
Change-Id: I6d8e330665860d376629ff6df8897d3826d0883b
2016-04-12Remove stray spaces from start of log messagesStephan Bergmann3-6/+6
Change-Id: I0bf6945a030867b3575fcef81c0d32addb85b992
2016-04-12tdf#91794 removed OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar5-60/+14
I removed all the OSL_DEBUG_LEVEL > 1 conditionals by SAL_INFOs or OSL_DEBUG_LEVEL > 0 Change-Id: I261644a179e763612204bbb0e54a869fa1a62298 Reviewed-on: https://gerrit.libreoffice.org/23035 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-04-01tdf#97966 Drop 'static' keywordsWastack6-24/+24
Including no keywords from extern "C" blocks Change-Id: I364aa7d67c2bf867588798e07df95c3f3f2bd7f3 Reviewed-on: https://gerrit.libreoffice.org/23676 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-30loplugin:nullptr: Find some more cases in templatesStephan Bergmann2-8/+8
Change-Id: I1f127d56e40b04f2b4df85c0afbcfd424d68a8cc
2016-03-17loplugin:constantparam in registryNoel Grandin1-1/+1
Change-Id: Ic3c60325a722422912d06fe21d072c5f5c4bccb1 Reviewed-on: https://gerrit.libreoffice.org/23322 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-10Fix memory leaksStephan Bergmann1-24/+5
Change-Id: If9f7dc4a28d5e005959f0d4a0a2ed317b699f292
2016-03-10No more need for jfw_freeJavaInfoStephan Bergmann1-3/+3
Change-Id: I2426a76936b4099a243ce8c102da867e7868aac3
2016-03-10Avoid reserved identifierStephan Bergmann1-5/+5
Change-Id: Ifc4929a5ef43194af33b0aa2816aaf21bc61cd0e
2016-03-09include/jvmfwk/*.h -> .hxx (only ever included from C++ code)Stephan Bergmann1-1/+1
Change-Id: Ia912e937d5a48afb6f8f5345b20bb7bd517fc4f1
2016-03-07coverity#1353440 try and refactor to avoid Argument cannot be negativeCaolán McNamara1-14/+7
Change-Id: I3a75e27ba1b721a99e0547b418df883e2c1753b4
2016-03-07return early and drop elseCaolán McNamara1-56/+54
Change-Id: I4887f01c94e6d7d716d2a074d503fca505a4aff8
2016-03-07!(==) -> !=Caolán McNamara1-7/+7
Change-Id: Ie1c3c61dea2052e85179157e3e0b6c07dd60bb48
2016-02-20coverity#1353440 Argument cannot be negativeCaolán McNamara1-5/+5
Change-Id: I0576264c006a70a6783e31fb77b380928558cf45
2016-02-18WaE: replace OUString constructed from a string literal directly ...Tor Lillqvist1-1/+1
... with the string literal [loplugin:stringconstant] Change-Id: I38fff65c8b5da320ad82419f5c7b46f35da2aa62
2016-02-18Resolves: rhbz#1285356 force swing not to use gtk2 if gtk3 is loadedCaolán McNamara1-1/+8
Change-Id: I6347bf4c25ce649073afdfe4225182ab2dc84af1 Reviewed-on: https://gerrit.libreoffice.org/22433 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-17use consistent #define checks for the Windows platformNoel Grandin1-1/+1
stage 2 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro In this stage we focus on replacing usage of the WIN macro Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84 Reviewed-on: https://gerrit.libreoffice.org/22393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-16use consistent #define checks for the Windows platformNoel Grandin2-2/+2
stage 1 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro Change-Id: Iece73abdee530937e0737190b1aa97a46cd3075f Reviewed-on: https://gerrit.libreoffice.org/22390 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-02-15Rename rtl::isValidCodePoint -> rtl::isUnicodeCodePointStephan Bergmann1-3/+3
...and fix its documentation, and use it throughout the code base. Change-Id: I349bc2009b1b0aa7115ea90bc6ecd0a812f63698
2016-02-15uno::Sequence->std::vector in stoc/introspectionNoel Grandin1-172/+130
Change-Id: I6427bc26d77e4ace2a827b9ae81716d62b5050ba
2016-02-12tdf#95857 Sort out German plurals ...danielt9981-3/+3
Made a start in removing the incorrect 'Infos' German plural Change-Id: Ie989351a7473fc35b563e63ce6a4fb229093af60 Reviewed-on: https://gerrit.libreoffice.org/22301 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2016-02-09Remove excess newlinesChris Sherlock24-111/+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-10Fix typosAndrea Gelmini2-2/+2
Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2015-12-21loplugin:unusedfields in sfx2,slideshow,starmath,stocNoel Grandin4-10/+3
Change-Id: If3622b23c45dd3a2a4e9869452142d1f6b47919e
2015-12-21tdf#96223: introspection queries for XPropertySet againOliver Specht1-15/+1
this patch removes the search by name for css::beans::XPropertySet because that fails if that is indirectly inherited e.g. with css::util::XSearchDescriptor Change-Id: I7f40b1d109d9bddc0987bea8eb3eda5b7f1aaaa9 Reviewed-on: https://gerrit.libreoffice.org/20792 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>