summaryrefslogtreecommitdiffstats
path: root/l10ntools
Commit message (Collapse)AuthorAgeFilesLines
* Remove excess newlinesChris Sherlock2016-02-094-4/+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>
* loplugin: unused return valuesNoel Grandin2016-01-182-4/+2
| | | | | | | Change-Id: I9c61a46c57894bc63a57740206c0bcb4a16553af Reviewed-on: https://gerrit.libreoffice.org/21571 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedfieldsNoel Grandin2016-01-182-7/+4
| | | | | | | Change-Id: I321b594f1190cdd997b4769161e1c02d31f46213 Reviewed-on: https://gerrit.libreoffice.org/21566 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Fix set but unused variable warningPeter Foley2016-01-161-2/+0
| | | | | | | Change-Id: I23fb9abf2b9462c939d4d9caf699e48ffded3bb4 Reviewed-on: https://gerrit.libreoffice.org/21505 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* loplugin:unusedmethods unused return value in l10ntools,linguisticNoel Grandin2016-01-157-42/+35
| | | | Change-Id: I11571c25e527787a2054bc8106a27c44b8a96116
* 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>
* Remove some includes (stdio/stdlib)Julien Nabet2015-12-272-5/+0
| | | | | | | Change-Id: I4aa649961efac521e72c363efb5d0a3192125da5 Reviewed-on: https://gerrit.libreoffice.org/20966 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
* Add snide commentTor Lillqvist2015-12-201-0/+2
| | | | Change-Id: Ie8c0d47a7a82458ec1ed0695cea13c136e2fab90
* loplugin:unusedfields variousNoel Grandin2015-11-252-5/+3
| | | | Change-Id: I59d9f7f73677358b4ae57efda965d43718bdf0d5
* coverity#1339308 Uncaught exceptionCaolán McNamara2015-11-143-10/+7
| | | | Change-Id: Id63bbdd11af66de6be8a3e9dfca1c7efbe74b08a
* new loplugin: memoryvarNoel Grandin2015-11-111-8/+7
| | | | | | | | | | 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-1014-68/+68
| | | | Change-Id: I87dbff06345f24064b87c375e5381cb27118e614
* coverity#1338269 Uncaught exceptionCaolán McNamara2015-11-061-15/+23
| | | | Change-Id: I32bfeb0ded37da05eecc97b92d9f724d59123fce
* coverity#1338268 Uncaught exceptionCaolán McNamara2015-11-061-12/+22
| | | | Change-Id: Id954d556c2b65772025b8aaabdbfa9ed4ae7a9b7
* coverity#1338261 Uncaught exceptionCaolán McNamara2015-11-061-59/+67
| | | | Change-Id: I3e818ed5ccf395f0fbaf82f8e62870b847fc41c9
* coverity#1338256 Uncaught exceptionCaolán McNamara2015-11-061-14/+21
| | | | Change-Id: Iadfb2e59fd40aa178123dbfc20be093cc8f128d2
* start next find at previous endpos, not one after previous startposEike Rathke2015-10-211-1/+1
| | | | | | | | The portion from startpos to endpos was handled, no need to recheck things in between and find nothing; or worse, find something and duplicate a part of a portion.. Change-Id: I37b58d2433514ffa8f31a0fa06f84c6a8aaee947
* properly check for ICU status so known tags get actually recognizedEike Rathke2015-10-211-1/+1
| | | | | | | | | | | | | | nIcuErr after the second RegexMatcher::find() call was U_STRING_NOT_TERMINATED_WARNING hence a check for U_ZERO_ERROR failed and a content string of "foo <emph>bar</emph> baz" became "foo <emph>bar&lt;/emph&gt; baz" which later provoked a missing ending tag error. This should had used U_SUCCESS(nIcuErr) instead and only worked by chance. Apparently the warning wasn't issued in earlier ICU versions and only introduced with 55 or 56. Change-Id: I36482c928544fc6e8c0403997af33084a47d4bea
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-123-19/+19
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* cppcheck:variableScopeNoel Grandin2015-10-121-3/+2
| | | | | | | 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-011-2/+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-302-2/+2
| | | | | | | Change-Id: I44bdb63bef76e9686e995661a24a3bef7cd50ab7 Reviewed-on: https://gerrit.libreoffice.org/18959 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* remove unused #include <sal/alloca.h>Michael Stahl2015-09-291-1/+0
| | | | Change-Id: I490841fd2c526e4ca6d6c94cc7b9dba318c932a1
* boost->stdCaolán McNamara2015-09-172-4/+4
| | | | Change-Id: Ifde84627578283bd057d7393eb7e5578ef5c029a
* Generated code is compiled with WaE disabled, anywayStephan Bergmann2015-09-033-21/+0
| | | | | | | ...and at least in the __GNUC__ case, the list of ignored warnings only covered a fraction of the actually emitted warnings, so just don't bother Change-Id: I962f50be7da3ae1e215318c6b7dd2f534cbdd57c
* tdf#39440 reduce scope of local variablesMichael Weghorn2015-08-221-2/+1
| | | | | | | | | This addresses some cppcheck warnings. Change-Id: I6ab170c4426494ab9fc53017f88e47fbfdbd9aad Reviewed-on: https://gerrit.libreoffice.org/17920 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* disentangle the Dzongkha mess a littleEike Rathke2015-08-221-1/+1
| | | | | | As per [MS-LCID] rev. 7.0 2015-06-30 Change-Id: If4980d2b1149d93b39e5b57f5741bbdfb5680516
* implicit conversion of NULL constant to 'nullptr_t'Noel Grandin2015-08-201-1/+1
| | | | Change-Id: I98f8df3f27fbd520f0bfbbb39c0e3cc5fa892b0d
* new loplugin automemNoel Grandin2015-08-208-85/+56
| | | | | | find places where we should be using std::unique_ptr Change-Id: I5b9defe778fdc4738ecea381215396874db59e66
* new loplugin: defaultparamsNoel Grandin2015-08-112-2/+2
| | | | | | | | find places where we do not need to be passing a parameter to a function, because that function has a default value which matches the value we are passing. Change-Id: I04d1fd6275204dd4925e6563282464f461123632
* -Werror,-Wunused-private-fieldStephan Bergmann2015-08-033-7/+3
| | | | Change-Id: I10206d2a9673296e1c3c7e813e4696d80cbb2bfc
* loplugin:unusedmethodsNoel Grandin2015-07-304-21/+3
| | | | | | | Change-Id: Ib4d77ee01e7362f5951f81fceeca3c489872d971 Reviewed-on: https://gerrit.libreoffice.org/17378 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Replace boost::scoped_array<T> with std::unique_ptr<T[]>Takeshi Abe2015-06-171-2/+2
| | | | | | | | | | | | | | | | | | | | | | This may reduce some degree of dependency on boost. Done by running a script like: git grep -l '#include *.boost/scoped_array.hpp.' \ | xargs sed -i -e 's@#include *.boost/scoped_array.hpp.@#include <memory>@' git grep -l '\(boost::\)\?scoped_array<\([^<>]*\)>' \ | xargs sed -i -e 's/\(boost::\)\?scoped_array<\([^<>]*\)>/std::unique_ptr<\2[]>/' ... and then killing duplicate or unnecessary includes, while changing manually m_xOutlineStylesCandidates in xmloff/source/text/txtimp.cxx, extensions/source/ole/unoconversionutilities.hxx, and extensions/source/ole/oleobjw.cxx. Change-Id: I3955ed3ad99b94499a7bd0e6e3a09078771f9bfd Reviewed-on: https://gerrit.libreoffice.org/16289 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* remove unnecessary check for null when calling deleteNoel Grandin2015-06-152-9/+7
| | | | | | | | 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
* loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2015-06-081-4/+4
| | | | Change-Id: I6088ea3fd106aa9563eb71afaf00977cc7c06e20
* new clang plugin: loopvartoosmallNoel Grandin2015-05-284-9/+9
| | | | | | | | | | | | | Idea from bubli - look for loops where the index variable is of such size that it cannot cover the range revealed by examining the length part of the condition. So far, I have only run the plugin up till the VCL module. Also the plugin deliberately excludes anything more complicated than a straightforward incrementing for loop. Change-Id: Ifced18b01c03ea537c64168465ce0b8287a42015
* l10ntools: support x-comments in ulf formatLászló Németh2015-05-221-1/+2
| | | | Change-Id: I9e2812b02348538aa5bb6a130530d12b31a374dd
* loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann2015-05-111-1/+1
| | | | Change-Id: Ia2d6f3e05398395af5e12ce9c1b35484f06a13b4
* Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann2015-03-311-1/+1
| | | | Change-Id: I8fc20c94ba0233a91b63ff35f91dc9c8b51a2aba
* Clean up C-style casts from pointers to voidStephan Bergmann2015-03-281-5/+5
| | | | Change-Id: Id9c24eea7b52fa28e0fdde39f28dc625aa82ffc2
* loplugin:staticfunctionNoel Grandin2015-03-275-16/+16
| | | | Change-Id: I10c30ef28e7321882a720d1e7869a75a89febfc5
* const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann2015-03-261-1/+1
| | | | Change-Id: I0f8379517d8e97c24b32ca58749f1c0f2c7e8f49
* new loplugin: change virtual methods to non-virtualNoel Grandin2015-01-262-21/+16
| | | | | | | | | | Where we can prove that the virtual method is never overriden. In the case of pure-virtual methods, we remove the method entirely. Sometimes this leads to entire methods and fields being eliminated. Change-Id: I138ef81c95f115dbd8c023a83cfc7e9d5d6d14ae
* add Guarani (gug) languageAndras Timar2015-01-251-3/+4
| | | | Change-Id: Icf1612f88447e9ae348ef9ad333607a3f6dc8d32
* Some more loplugin:cstylecast: l10ntoolsStephan Bergmann2015-01-202-16/+16
| | | | Change-Id: Ifa1c715b5569cb7481a6f1ecaa98306ca7e2d7d8
* brute-force find-and-remove of unused #define constants.Noel Grandin2015-01-081-2/+0
| | | | Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
* attempt to compare a singular iterator to a singular iteratorCaolán McNamara2015-01-021-3/+12
| | | | | | falls over during build with debug stl Change-Id: Id7241e1275067f827e4a663eb901c41e4c9873f5
* boost::unordered_map->std::unordered_mapCaolán McNamara2015-01-024-11/+11
| | | | | | you can get debug stl this way Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19
* fdo#87754: duplicated strings in an ItemList are not translatedZolnai Tamás2014-12-281-8/+11
| | | | Change-Id: I72c6e234ff999a6dbed48cc62fe7d39aeb798f35
* fdo#39440 reduce scope of local variablesMichael Weghorn2014-12-182-20/+11
| | | | | | | | | This addresses some cppcheck warnings. Change-Id: Ib16e4ecc0e0c43b2c1fb527eb0668a960ae9756e Reviewed-on: https://gerrit.libreoffice.org/13513 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>