summaryrefslogtreecommitdiffstats
path: root/stoc
Commit message (Collapse)AuthorAgeFilesLines
* tdf#100726 - Improve readability of OUString concatanationsGökhan Gurbetoğlu2016-08-251-22/+14
| | | | | | | | | | Improved readability of OUString concatanations. Also removed unused OUStrings "sColor" and "sEntry" from the code. Change-Id: Ie9792f499cd880be72229f8a8c71f05ff8e258b6 Reviewed-on: https://gerrit.libreoffice.org/28375 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* loplugin:countusersofdefaultparams in sfx2..svgioNoel Grandin2016-08-051-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>
* improve passstuffbyref return analysisNoel Grandin2016-07-271-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>
* don't write past the end of the arrayMarkus Mohrhard2016-07-181-1/+1
| | | | | | | Found through http://crashreport.libreoffice.org/stats/crash_details/6ba2edbd-d268-4074-8b87-590a5694cc26 Change-Id: Ic95d472fd117c2d286fabb775ef49e287a2874fa
* loplugin:nullptr: Better heuristic to determine code shared between C and C++Stephan Bergmann2016-07-111-1/+1
| | | | Change-Id: I51e1c5fa4639e51fac90f92adf3d87d12960d589
* loplugin:passstuffbyref also for {css::uno,rtl}::ReferenceStephan Bergmann2016-07-071-2/+2
| | | | Change-Id: I5ef89d0f0f97dace7dc15dc07b3328177aab462b
* remove deprecated Link functions from RegistryNoel Grandin2016-07-041-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>
* Clean up uses of Any::getValue() in stocStephan Bergmann2016-06-217-108/+95
| | | | Change-Id: I455b708bfcc9d5aa71bb0ffc21efec205302169e
* loplugin:salbool: Implicit conversions from non-Boolean fundamental typesStephan Bergmann2016-06-191-22/+25
| | | | Change-Id: I67eac95686678e6f5a2d60798535b2c65a9ba5d7
* remove some manual ref-countingNoel Grandin2016-06-061-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>
* Some clean up of uses of css::uno::Any::setValueStephan Bergmann2016-05-301-3/+2
| | | | Change-Id: I04e8aef35a6083b61d775c8eb3f96757da2b31bd
* Just use Any::operator <<= for sal_Unicode valuesStephan Bergmann2016-05-301-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
* Revert "remove some manual ref-counting"Noel Grandin2016-05-241-76/+82
| | | | | | | until I have a better understanding of the UNO reference counting. This reverts commit 111de438ea3e512a541281dc0716cc728ea8d152.
* remove some manual ref-countingNoel Grandin2016-05-241-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>
* loplugin:unusedmethods in slideshow to svtoolsNoel Grandin2016-05-191-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>
* clang-tidy modernize-loop-convert in scripting to svtoolsNoel Grandin2016-05-111-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>
* Insert explicit "break" when falling through to empty next caseStephan Bergmann2016-05-101-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
* While at it, delete Any functions on sal_Bool*Stephan Bergmann2016-05-041-6/+3
| | | | | | | (at least for LIBO_INTERNAL_ONLY), to help further reduce the occurrences of sal_Bool across the code base Change-Id: I70654a0cb56655984c717b7b894f26c9ab47536e
* tdf#97499 Fixed containers parameters clearing #9tymyjan2016-05-011-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>
* update loplugin stylepolice to check local pointers varsNoel Grandin2016-04-261-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>
* loplugin:salbool: sal_Bool[] -> bool[]Stephan Bergmann2016-04-211-13/+13
| | | | Change-Id: I3c5bf7a53c9ae173f8fce885ecf022f092aa43a9
* loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann2016-04-2011-26/+26
| | | | Change-Id: I3a3295b538e79186914390b6da73cc8bc25419a5
* clang-tidy performance-unnecessary-copy-initializationNoel Grandin2016-04-182-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>
* clang-tidy performance-unnecessary-value-param in variousNoel Grandin2016-04-142-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>
* loplugin:passstuffbyref in variousNoel Grandin2016-04-141-1/+1
| | | | Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
* tdf#94306 replace boost::noncopyable in stoc to xmlsec..Jochen Nitschke2016-04-138-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>
* Remove stray spaces from start of log messages, reduxStephan Bergmann2016-04-121-1/+1
| | | | Change-Id: I6d8e330665860d376629ff6df8897d3826d0883b
* Remove stray spaces from start of log messagesStephan Bergmann2016-04-123-6/+6
| | | | Change-Id: I0bf6945a030867b3575fcef81c0d32addb85b992
* tdf#91794 removed OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar2016-04-125-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>
* tdf#97966 Drop 'static' keywordsWastack2016-04-016-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>
* loplugin:nullptr: Find some more cases in templatesStephan Bergmann2016-03-302-8/+8
| | | | Change-Id: I1f127d56e40b04f2b4df85c0afbcfd424d68a8cc
* loplugin:constantparam in registryNoel Grandin2016-03-171-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>
* Fix memory leaksStephan Bergmann2016-03-101-24/+5
| | | | Change-Id: If9f7dc4a28d5e005959f0d4a0a2ed317b699f292
* No more need for jfw_freeJavaInfoStephan Bergmann2016-03-101-3/+3
| | | | Change-Id: I2426a76936b4099a243ce8c102da867e7868aac3
* Avoid reserved identifierStephan Bergmann2016-03-101-5/+5
| | | | Change-Id: Ifc4929a5ef43194af33b0aa2816aaf21bc61cd0e
* include/jvmfwk/*.h -> .hxx (only ever included from C++ code)Stephan Bergmann2016-03-091-1/+1
| | | | Change-Id: Ia912e937d5a48afb6f8f5345b20bb7bd517fc4f1
* coverity#1353440 try and refactor to avoid Argument cannot be negativeCaolán McNamara2016-03-071-14/+7
| | | | Change-Id: I3a75e27ba1b721a99e0547b418df883e2c1753b4
* return early and drop elseCaolán McNamara2016-03-071-56/+54
| | | | Change-Id: I4887f01c94e6d7d716d2a074d503fca505a4aff8
* !(==) -> !=Caolán McNamara2016-03-071-7/+7
| | | | Change-Id: Ie1c3c61dea2052e85179157e3e0b6c07dd60bb48
* coverity#1353440 Argument cannot be negativeCaolán McNamara2016-02-201-5/+5
| | | | Change-Id: I0576264c006a70a6783e31fb77b380928558cf45
* WaE: replace OUString constructed from a string literal directly ...Tor Lillqvist2016-02-181-1/+1
| | | | | | ... with the string literal [loplugin:stringconstant] Change-Id: I38fff65c8b5da320ad82419f5c7b46f35da2aa62
* Resolves: rhbz#1285356 force swing not to use gtk2 if gtk3 is loadedCaolán McNamara2016-02-181-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>
* use consistent #define checks for the Windows platformNoel Grandin2016-02-171-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>
* use consistent #define checks for the Windows platformNoel Grandin2016-02-162-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>
* Rename rtl::isValidCodePoint -> rtl::isUnicodeCodePointStephan Bergmann2016-02-151-3/+3
| | | | | | ...and fix its documentation, and use it throughout the code base. Change-Id: I349bc2009b1b0aa7115ea90bc6ecd0a812f63698
* uno::Sequence->std::vector in stoc/introspectionNoel Grandin2016-02-151-172/+130
| | | | Change-Id: I6427bc26d77e4ace2a827b9ae81716d62b5050ba
* tdf#95857 Sort out German plurals ...danielt9982016-02-121-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>
* Remove excess newlinesChris Sherlock2016-02-0924-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>
* Fix typosAndrea Gelmini2016-01-102-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>
* loplugin:unusedfields in sfx2,slideshow,starmath,stocNoel Grandin2015-12-214-10/+3
| | | | Change-Id: If3622b23c45dd3a2a4e9869452142d1f6b47919e