summaryrefslogtreecommitdiff
path: root/scripting
AgeCommit message (Collapse)AuthorFilesLines
2015-12-29cppcheck: noExplicitConstructorCaolán McNamara1-1/+1
Change-Id: Ib43e53d5b6c9c130adb765ac9b769f58060ac640
2015-12-22loplugin:unusedfields in scriptingNoel Grandin3-32/+17
Change-Id: Ida8fb31bd85f4807d74daab3d022d8e6340a774d
2015-11-26mark UNO structs as SAL_WARN_UNUSED, where possibleNoel Grandin1-3/+0
Change-Id: Ie3de518f60c9f1313c68df54dbdc1fb2804f1f0d
2015-11-20add mapKeysToSequence/mapValuesToSequence methods to comphelperNoel Grandin1-7/+1
and use them Change-Id: If4dc9df63db37185228aeaaab2979498d61304ec Reviewed-on: https://gerrit.libreoffice.org/20055 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-19use comphelper::containerToSequenceNoel Grandin1-8/+1
in chart2, we remove a local equivalent of the method Change-Id: I25129a3d1ea1dd724eb9cd38a57be37a78b3d100
2015-11-16no need to take a mutex when only dealing with stack-local dataNoel Grandin1-2/+1
Change-Id: Ie45e626aad55a8174a53b769a98601bf54dedf65 Reviewed-on: https://gerrit.libreoffice.org/19979 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-16use initialiser list for Sequence<OUString>Noel Grandin1-3/+1
Change-Id: Ia5e47261d1fc6fac2d046656c05a1c5eedb07e02 Reviewed-on: https://gerrit.libreoffice.org/19978 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser for Sequence<OUString>Noel Grandin1-2/+1
using variations of: git grep -lP 'Sequence.*OUString.*\(\s*1\s*\)' | xargs perl -0777 -pi -e "s/Sequence<OUString> (\w+)\(1\)\; \s*OUString\* pArray.*; .*\[0\]\s*=\s*(\S+)\;/Sequence<OUString> \1 { \2 };/g" Change-Id: I03c64334ff30ee14dce0d17b67f5122a3893bbe3 Reviewed-on: https://gerrit.libreoffice.org/19971 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser syntax for Sequence<OUString>Noel Grandin2-8/+4
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-13cppcheck: noExplicitConstructorCaolán McNamara1-3/+3
Change-Id: Ife26f55c28c4631aec4ba4105225bfca72da8bff
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann11-106/+106
Change-Id: I9b7d8de65780325175d24965057596b88d3c0f05
2015-11-09java: add some @Override annotationsNoel Grandin1-0/+1
Change-Id: I291db1dff5ab3ce40fda6f822a0e3ff1e6ef36c1 Reviewed-on: https://gerrit.libreoffice.org/19855 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-09new loplugin: oncevarNoel Grandin1-2/+1
Change-Id: If57390510dde4d166be3141b9f658a7453755d3f Reviewed-on: https://gerrit.libreoffice.org/19815 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-06loplugin:stringconstant: elide explicit ctor usage (manually due to macros)Stephan Bergmann3-3/+3
Change-Id: Ib023c89256d50ebe15231a96f6106f60198d362e
2015-11-06loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann3-5/+5
Change-Id: If0f1446a25f423b8cbfe6346f7a691f4f2618ad0
2015-11-05java: combine nested if statementsNoel Grandin3-26/+20
Change-Id: I0457b81668e9427a3c8d6a4af93438b7fb2bb7ba
2015-11-05use uno::Reference::set method instead of assignmentNoel Grandin9-24/+21
Change-Id: I2b592fd3327f4bbe4685e84711dc9d6f19222a97
2015-11-04yyyyyNoel Grandin3-5/+4
Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
2015-11-02No need for this debug codeStephan Bergmann1-7/+0
Change-Id: I05a6d15a0c54ca5457d662c2467cb15c14991a0a
2015-11-02cppcheck: noExplicitConstructorCaolán McNamara13-19/+19
Change-Id: I06186e8f0bbc8522f538e8639a68e816093becc2
2015-11-01no need to use OUString constructor in call to createInstanceNoel Grandin1-7/+3
Change-Id: Iaf3d83ba1490cb1d97a5bd4d1f7cd6943d4a7296 Reviewed-on: https://gerrit.libreoffice.org/19704 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-30UNO: no need to use OUString constructor when calling createInstanceNoel Grandin1-2/+2
Change-Id: I37da28539b94706574116d0fff5f008aabfb5526 Reviewed-on: https://gerrit.libreoffice.org/19682 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-28com::sun::star->css in scaddins,sccomp,scriptingNoel Grandin20-589/+584
Change-Id: Id9167341940bac65f055e1e33ff0670f6fa0f6c4 Reviewed-on: https://gerrit.libreoffice.org/19629 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-25cid#1326721 to cid#1326726 Dm: Dubious method usedCaolán McNamara1-11/+11
Change-Id: I5578dc3885945c013064cecf2ca86e7ca734368f
2015-10-25coverity#1326446 Dereference null return valueCaolán McNamara1-1/+1
Change-Id: I18b3e0cc30e05e9a53a0f7ae9997b9f190f192f4
2015-10-22coverity#1326446 Dereference null return valueCaolán McNamara1-0/+6
Change-Id: Ief00b565147bd4a08c4fd6de9df71d9126dc1f75
2015-10-22coverity#1326429 Dereference null return valueCaolán McNamara1-4/+3
Change-Id: Ib047f6cb255610422afb423006b58e0da7eacead
2015-10-20coverity#1326553 Dereference before null checkCaolán McNamara1-1/+1
Change-Id: I1fe2e0477f136f31b854672878d0544d9492d51b
2015-10-20coverity#1326552 Dereference before null checkCaolán McNamara1-1/+1
Change-Id: I5a3fb6b5e9966a8e0c5dd0175e4f4f3292025a2c
2015-10-20Revert "coverity#1327168 see if stock SuppressWarnings is sufficient to"Caolán McNamara1-1/+0
that didn't work This reverts commit 04a7df738aae3b255c1faa80fa79262602eaead7. Change-Id: I5e4dfc28c03668eebb0fe8056beede7ef0b05ee5
2015-10-16cid#1326721 to cid#1326726 Dm: Dubious method usedNoel Grandin1-7/+7
Change-Id: I98e5590f61434ed1b1081b5b592622f9bcf037ec
2015-10-15coverity#1327168 see if stock SuppressWarnings is sufficient toCaolán McNamara1-1/+2
silence UrF: Unread field, some sources online suggest that it might be the case Change-Id: I2287da343e97bec93e0aaf7a82a408c18ab76d9b Reviewed-on: https://gerrit.libreoffice.org/19396 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-10-15coverity#1326240 Explicit null dereferencedCaolán McNamara1-2/+7
Change-Id: Ic84b5fbff0fbd7b8a750a7e40f2dffafe868b5d5
2015-10-15coverity#1326238 Explicit null dereferencedCaolán McNamara1-10/+8
Change-Id: Id02dcf1a1fac22eae0a2e01596521eefee9ecbf3
2015-10-15coverity#1326236 Explicit null dereferencedCaolán McNamara1-4/+3
Change-Id: Ic89c9b39f5f9c260581130da528bc0f514d4ef06
2015-10-15coverity#1326899 HE: Equal objects must have equal hashcodesCaolán McNamara2-0/+10
and... coverity#1326898 HE: Equal objects must have equal hashcodes coverity#1326897 HE: Equal objects must have equal hashcodes coverity#1326896 HE: Equal objects must have equal hashcodes coverity#1326895 HE: Equal objects must have equal hashcodes Change-Id: Ic9cb7542c1bf0608a97e5b4756b737cdf1bb4bd8
2015-10-15cid#1326116 Unchecked return valueNoel Grandin1-1/+3
Change-Id: If33ea9c4e4388cc1b7d15847f32dd2228b047363
2015-10-15cid#1327023 to cid#1327077 SIC: Inner class could be made staticNoel Grandin1-3/+3
Change-Id: I41f89c4feefe4e012d72c663ebb9bbcb4aa7f163
2015-10-14java:using "this" with a static field makes no senseNoel Grandin1-7/+7
Change-Id: I79a38ff2ec50d3a3687cac7ed7688bbbcaf2208c
2015-10-14coverity#1326568 Unused valueCaolán McNamara1-11/+3
Change-Id: Idf1efef484d3a89bd56ba9a6807d4191ef95cb41
2015-10-14coverity#1327113 UrF: Unread fieldCaolán McNamara1-2/+0
Change-Id: I4cc7faaf0ac74c4e1baa2e978d8dea9bc8d7b6d0
2015-10-14coverity#1327178 UuF: Unused fieldCaolán McNamara1-2/+1
Change-Id: Ie9ed1f86a3a51d766987d631605dc561e9748f99
2015-10-14cid#1326361 Data race conditionNoel Grandin1-16/+18
Change-Id: Ia61737232d816bb05f3aedbe76e7905262906a79
2015-10-14cid#1326674 DLS: Dead local storeNoel Grandin1-7/+0
Change-Id: I8168d3703022c9a1baed6e82c43a04ffa0ca52c0
2015-10-14cid#1326345 Check of thread-shared field evades lock acquisitionNoel Grandin1-7/+3
Change-Id: I11fc66358c13c6bb1b6f109980384131b38ef18b
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann18-297/+297
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann1-2/+2
Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
2015-09-30Fix typosAndrea Gelmini5-5/+5
Change-Id: Idd4b0f7b6c422abd6790185b5d48082b294bae64 Reviewed-on: https://gerrit.libreoffice.org/18968 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2015-09-30Script: better way to detect WindowsAndrea Gelmini1-71/+72
Change-Id: I0e99443f3715cde4414d14b2fa507ecab0995c6d Reviewed-on: https://gerrit.libreoffice.org/18937 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-09-18boost->stdCaolán McNamara1-3/+3
Change-Id: Ie490bf2c6921f393bdeed96b1a8815996b701bf0 Reviewed-on: https://gerrit.libreoffice.org/18670 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>