summaryrefslogtreecommitdiffstats
path: root/rsc
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:constantparamNoel Grandin2016-04-052-6/+5
| | | | | | | Change-Id: I821ed77a6c6014c9a82d31a4b117ed6f1b4abf18 Reviewed-on: https://gerrit.libreoffice.org/23832 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#97966 Drop 'static' keywordsWastack2016-04-011-3/+3
| | | | | | | | | Including no keywords from extern "C" blocks Change-Id: I87f2ed75888b51ec9e0cb75566bf7c2351b479b4 Reviewed-on: https://gerrit.libreoffice.org/23675 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* -Werror=unused-but-set-variableStephan Bergmann2016-03-231-3/+0
| | | | Change-Id: I7c0f1e37f3a53dfed09c4e13d10826022b3ca777
* loplugin:constantparam in rscNoel Grandin2016-03-236-71/+28
| | | | Change-Id: I54f80c9c759eec4b73d1fd194e3efc8156bd3e2d
* Revert "tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionals"Norbert Thiebaud2016-03-139-37/+52
| | | | | | | This reverts commit 0c3ff8ca07265304f9ea0a7aca8c83703cff01a1. that was breaking debug and dbgutil build on windows (at least) Change-Id: Ide58bd4551029bb4e1ef791bb65ad191fbb25cae
* WaE: loplugin:stringconcatCaolán McNamara2016-03-121-2/+2
| | | | Change-Id: I28a106e978cdfd70847737784f9bff6ae365fda7
* tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar2016-03-129-52/+37
| | | | | | | | | | I replaced OSL_DEBUG_LEVEL > 1 with OSL_DEBUG_LEVEL > 0 conditionals and in some places i used SAL_INFO(..)s Change-Id: I87e2e4d73be22630dbcce6df2650b924cba9f8ec Reviewed-on: https://gerrit.libreoffice.org/23005 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* -Werror,-Wunused-macrosStephan Bergmann2016-03-041-4/+0
| | | | | | Appears to no (longer?) be relevant here, as reported by clang-cl. Change-Id: I339f8b04eb733a9c6a97fcd4b9b4758e6a84d5c3
* loplugin:unuseddefaultparam in rscNoel Grandin2016-03-024-6/+6
| | | | Change-Id: Ib4cbc48295e7c16683765d1c1420fcf73235517d
* sal_uIntPtr/sal_uLong to Index (aka sal_uInt32) in UniqueIndexMatteo Casalin2016-03-017-67/+68
| | | | Change-Id: I212cb3bb9d920741629fc4564bbd28b393e8fe00
* new loplugin: commaoperatorNoel Grandin2016-02-231-2/+4
| | | | Change-Id: I03f24e61f696b7619855e3c7010aa0d874e5a4ff
* use consistent #define checks for the Windows platformNoel Grandin2016-02-173-3/+3
| | | | | | | | | | | | 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>
* Fix typosAndrea Gelmini2016-02-051-2/+2
| | | | | | | Change-Id: Ice72f8d9971e15dd6ef365e64cd567b8581a92d3 Reviewed-on: https://gerrit.libreoffice.org/21797 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* Compiler warning battleStephan Bergmann2016-02-021-1/+8
| | | | | | | | ...where 05642b446c39c1551d2d703be18158c7ae0f0c36 "-Werror,-Wconstant-conversion (from 255 to -1)" caused MSVC to counter with "warning C4310: cast truncates constant value" Change-Id: I7d80825a01e49817d7989e00774fed1f162dea85
* -Werror,-Wconstant-conversion (from 255 to -1)Stephan Bergmann2016-02-021-1/+1
| | | | Change-Id: I9cce09bf729398a475d9d07fdfa002f325611546
* ditch loading of SplitWindows from resource filesCaolán McNamara2016-01-303-22/+0
| | | | Change-Id: Ic56aa8d3dcafc24e1a1a1d494d20b0d23b41677d
* loplugin:unusedmethods unused return value in rscNoel Grandin2016-01-1512-41/+25
| | | | Change-Id: I6301aed80083c21e26851817220de6e135eb4da7
* Fix typosAndrea Gelmini2016-01-101-1/+1
| | | | | | | 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 variousNoel Grandin2015-11-251-1/+0
| | | | Change-Id: I59d9f7f73677358b4ae57efda965d43718bdf0d5
* new loplugin: memoryvarNoel Grandin2015-11-111-10/+6
| | | | | | | | | | 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-1031-234/+234
| | | | Change-Id: I8df6c1c7d2312a4864750180561a3e63fe37003f
* -Werror,-WformatStephan Bergmann2015-10-261-1/+1
| | | | Change-Id: I9668268f15fbcd4af7432d4d0d403b74263fbb18
* remove extra spaces around ::Noel Grandin2015-10-267-33/+33
| | | | | | just makes it harder to grep for methods Change-Id: I0925db2f9284972fcf7e1e3adf90cbe21f69eb15
* cppcheck: invalidPrintfArgType_sintCaolán McNamara2015-10-241-1/+1
| | | | Change-Id: I4103c8e1549db44055fbfcb8ced6ec36d27fc126
* loplugin:defaultparamsStephan Bergmann2015-10-201-2/+2
| | | | Change-Id: I80c422a961170f25d2a674bb347b9d330b6c7fc0
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-1210-168/+168
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* cppcheck:variableScopeNoel Grandin2015-10-121-36/+22
| | | | | | | Change-Id: I037feb335499629300309851dcda3bb661f03d4f Reviewed-on: https://gerrit.libreoffice.org/19316 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedmethodsNoel Grandin2015-10-012-8/+0
| | | | | | | | | | | | - improvements to the plugin to find more method calls - improvements to python script to remove more false+ - fix the FORCE_COMPILE_ALL build flag to include code in the $WORKDIR Change-Id: I4d6015dcb9b9d60c26f0bcee8abad807177a7836 Reviewed-on: https://gerrit.libreoffice.org/19064 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Fix typosAndrea Gelmini2015-09-307-9/+9
| | | | | | | Change-Id: I8f4500fc7a901c5cc73634ba6da6b9541452e5ae Reviewed-on: https://gerrit.libreoffice.org/18966 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* Renamed wrongly prefixed boolean variablesStefan Heinemann2015-09-292-2/+2
| | | | | | | | | Fixed tdf#94269 Change-Id: I63109cc4e095bad680d7637a065080ea368860ae Reviewed-on: https://gerrit.libreoffice.org/18851 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
* loplugin:removeunusedmethods, remove unused stuffNoel Grandin2015-09-292-41/+0
| | | | Change-Id: I35456b2a3ad2a84a1d045f09cdfb29e4c19b8350
* convert Link<> to typedNoel Grandin2015-09-234-31/+31
| | | | | | | Change-Id: I80bd798ea5d36fa94d6b3348be713dbe2cbeab14 Reviewed-on: https://gerrit.libreoffice.org/18803 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Use a C99 flexible array memberStephan Bergmann2015-09-182-2/+2
| | | | | | | | | ...so the following strcpy does not cause a false abort under _FORTIFY_SOURCE=2 Change-Id: I395136f11020064766db76f87ed8f5b01c083c3a Reviewed-on: https://gerrit.libreoffice.org/18681 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin:mergeclass, merge BiNode with NameNode, Obj0Type with ObjkTypeNoel Grandin2015-09-082-36/+26
| | | | Change-Id: Icbc0dfc6096a6e2c651dad4fe9f78d176f389390
* cppcheck: fix some redundantCondition warningsJulien Nabet2015-09-051-2/+1
| | | | | | | Change-Id: If53d33fb4d9ca9dc1babb9043bd07c5d29fe74c2 Reviewed-on: https://gerrit.libreoffice.org/18353 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
* tdf#39468: Translate German comments in core/rsc/incLucas Satabin2015-08-3013-166/+157
| | | | | | | Change-Id: I5d878192bde816e59ef570731113b526e8098d85 Reviewed-on: https://gerrit.libreoffice.org/18065 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* tdf#39468: Translate German comments in core/rsc/sourceLucas Satabin2015-08-2813-250/+242
| | | | | | | Change-Id: Ia2d94a88d217a3b0ec9c78096b8d74a38216fbc9 Reviewed-on: https://gerrit.libreoffice.org/18066 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* loplugin:unusedmethodsNoel Grandin2015-08-0513-60/+1
| | | | | | | Change-Id: I6801618efb5a66d24156fa429e026acb6ca03aba Reviewed-on: https://gerrit.libreoffice.org/17506 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* sal,rsc,oox: inline some use-once typedefsNoel Grandin2015-08-042-9/+5
| | | | Change-Id: I91bebe06c5c5f3ebe5fbeec83dcf3f5bdf563201
* convert WindowAlign to scoped enumNoel Grandin2015-07-242-4/+6
| | | | Change-Id: I028d108b3a83bd5541d873c4d03b87339cd3be52
* formatting of public/private/protected section qualifiersNoel Grandin2015-07-171-2/+2
| | | | | | | | | | make it consistent across the codebase, no space between keyword and the colon Change-Id: Idca61ddfc74ad2460fb05fe417499324b05e5de5 Reviewed-on: https://gerrit.libreoffice.org/17148 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* remove unnecessary check for null when calling deleteNoel Grandin2015-06-152-4/+2
| | | | | | | | Idea originally from caolan. Found using the following command: find . -name *.cxx | xargs /opt/local/bin/grep -zlP '(?m)if\s*\(\s*\w+\s*\)\s*delete\s+\w+\;' Change-Id: I3338f4e22193a6dfd6219c8c75835224a3392763
* cppcheck: variableScopeCaolán McNamara2015-06-091-5/+1
| | | | Change-Id: If0c56c89e2722d47a401363f3854331229856490
* loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2015-06-088-20/+20
| | | | Change-Id: I27516cc532e46b6e43a2f2f292246f1414e9f9b3
* -fsanitize=nonnull-attribute in memcpy callStephan Bergmann2015-06-021-1/+5
| | | | Change-Id: I4fc491906311781b9941c8f02d99d254542027ae
* loplugin:cstylecast: deal with those that are (technically) const_castStephan Bergmann2015-06-021-1/+1
| | | | Change-Id: Ie100fc1048b076f94e7ef33b632df4264d37cf51
* loplugin:loopvartoosmallNoel Grandin2015-05-291-2/+2
| | | | Change-Id: I1e9768c08af0bc7caac6a39c13842ee9d8ad962c
* loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann2015-05-111-10/+10
| | | | Change-Id: I1e62523e08e6f1e7ec926b1a5e7c45d7c688f781
* Bin empty eval statementMatúš Kukan2015-05-021-3/+0
| | | | Change-Id: I52755c373f86490ba3e8910e71e3fab88b1ca38c
* Gradually typed LinkStephan Bergmann2015-04-302-2/+2
| | | | | | | | | | | | | | | | | Turn the Link class into a template abstracting over the link's argument and return types, but provide default template arguments that keep the generic, unsafe "void* in, sal_IntPtr out" behvior. That way, individual uses of the Link class can be updated over time. All the related macros are duplicated with ..._TYPED counterparts, that additionally take the RetType (except for LINK_TYPED, which manages to infer the relevant types from the supplied Member). (It would have been attractive to change the "untyped" LinkStubs from taking a void* to a properly typed ArgType parameter, too, but that would cause -fsanitize=function to flag uses of "untyped" Link::Call.) Change-Id: I3b0140378bad99abbf240140ebb4a46a05d2d2f8