summaryrefslogtreecommitdiff
path: root/l10ntools
AgeCommit message (Collapse)AuthorFilesLines
2016-02-09Remove excess newlinesChris Sherlock4-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>
2016-01-18loplugin: unused return valuesNoel Grandin2-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>
2016-01-18loplugin:unusedfieldsNoel Grandin2-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>
2016-01-16Fix set but unused variable warningPeter Foley1-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>
2016-01-15loplugin:unusedmethods unused return value in l10ntools,linguisticNoel Grandin7-42/+35
Change-Id: I11571c25e527787a2054bc8106a27c44b8a96116
2016-01-10Fix typosAndrea Gelmini1-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>
2015-12-27Remove some includes (stdio/stdlib)Julien Nabet2-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>
2015-12-20Add snide commentTor Lillqvist1-0/+2
Change-Id: Ie8c0d47a7a82458ec1ed0695cea13c136e2fab90
2015-11-25loplugin:unusedfields variousNoel Grandin2-5/+3
Change-Id: I59d9f7f73677358b4ae57efda965d43718bdf0d5
2015-11-14coverity#1339308 Uncaught exceptionCaolán McNamara3-10/+7
Change-Id: Id63bbdd11af66de6be8a3e9dfca1c7efbe74b08a
2015-11-11new loplugin: memoryvarNoel Grandin1-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>
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann14-68/+68
Change-Id: I87dbff06345f24064b87c375e5381cb27118e614
2015-11-06coverity#1338269 Uncaught exceptionCaolán McNamara1-15/+23
Change-Id: I32bfeb0ded37da05eecc97b92d9f724d59123fce
2015-11-06coverity#1338268 Uncaught exceptionCaolán McNamara1-12/+22
Change-Id: Id954d556c2b65772025b8aaabdbfa9ed4ae7a9b7
2015-11-06coverity#1338261 Uncaught exceptionCaolán McNamara1-59/+67
Change-Id: I3e818ed5ccf395f0fbaf82f8e62870b847fc41c9
2015-11-06coverity#1338256 Uncaught exceptionCaolán McNamara1-14/+21
Change-Id: Iadfb2e59fd40aa178123dbfc20be093cc8f128d2
2015-10-21start next find at previous endpos, not one after previous startposEike Rathke1-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
2015-10-21properly check for ICU status so known tags get actually recognizedEike Rathke1-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
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann3-19/+19
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12cppcheck:variableScopeNoel Grandin1-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>
2015-10-01loplugin:unusedmethodsNoel Grandin1-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>
2015-09-30Fix typosAndrea Gelmini2-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>
2015-09-29remove unused #include <sal/alloca.h>Michael Stahl1-1/+0
Change-Id: I490841fd2c526e4ca6d6c94cc7b9dba318c932a1
2015-09-17boost->stdCaolán McNamara2-4/+4
Change-Id: Ifde84627578283bd057d7393eb7e5578ef5c029a
2015-09-03Generated code is compiled with WaE disabled, anywayStephan Bergmann3-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
2015-08-22tdf#39440 reduce scope of local variablesMichael Weghorn1-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>
2015-08-22disentangle the Dzongkha mess a littleEike Rathke1-1/+1
As per [MS-LCID] rev. 7.0 2015-06-30 Change-Id: If4980d2b1149d93b39e5b57f5741bbdfb5680516
2015-08-20implicit conversion of NULL constant to 'nullptr_t'Noel Grandin1-1/+1
Change-Id: I98f8df3f27fbd520f0bfbbb39c0e3cc5fa892b0d
2015-08-20new loplugin automemNoel Grandin8-85/+56
find places where we should be using std::unique_ptr Change-Id: I5b9defe778fdc4738ecea381215396874db59e66
2015-08-11new loplugin: defaultparamsNoel Grandin2-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
2015-08-03-Werror,-Wunused-private-fieldStephan Bergmann3-7/+3
Change-Id: I10206d2a9673296e1c3c7e813e4696d80cbb2bfc
2015-07-30loplugin:unusedmethodsNoel Grandin4-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>
2015-06-17Replace boost::scoped_array<T> with std::unique_ptr<T[]>Takeshi Abe1-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>
2015-06-15remove unnecessary check for null when calling deleteNoel Grandin2-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
2015-06-08loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann1-4/+4
Change-Id: I6088ea3fd106aa9563eb71afaf00977cc7c06e20
2015-05-28new clang plugin: loopvartoosmallNoel Grandin4-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
2015-05-22l10ntools: support x-comments in ulf formatLászló Németh1-1/+2
Change-Id: I9e2812b02348538aa5bb6a130530d12b31a374dd
2015-05-11loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann1-1/+1
Change-Id: Ia2d6f3e05398395af5e12ce9c1b35484f06a13b4
2015-03-31Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann1-1/+1
Change-Id: I8fc20c94ba0233a91b63ff35f91dc9c8b51a2aba
2015-03-28Clean up C-style casts from pointers to voidStephan Bergmann1-5/+5
Change-Id: Id9c24eea7b52fa28e0fdde39f28dc625aa82ffc2
2015-03-27loplugin:staticfunctionNoel Grandin5-16/+16
Change-Id: I10c30ef28e7321882a720d1e7869a75a89febfc5
2015-03-26const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann1-1/+1
Change-Id: I0f8379517d8e97c24b32ca58749f1c0f2c7e8f49
2015-01-26new loplugin: change virtual methods to non-virtualNoel Grandin2-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
2015-01-25add Guarani (gug) languageAndras Timar1-3/+4
Change-Id: Icf1612f88447e9ae348ef9ad333607a3f6dc8d32
2015-01-20Some more loplugin:cstylecast: l10ntoolsStephan Bergmann2-16/+16
Change-Id: Ifa1c715b5569cb7481a6f1ecaa98306ca7e2d7d8
2015-01-08brute-force find-and-remove of unused #define constants.Noel Grandin1-2/+0
Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
2015-01-02attempt to compare a singular iterator to a singular iteratorCaolán McNamara1-3/+12
falls over during build with debug stl Change-Id: Id7241e1275067f827e4a663eb901c41e4c9873f5
2015-01-02boost::unordered_map->std::unordered_mapCaolán McNamara4-11/+11
you can get debug stl this way Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19
2014-12-28fdo#87754: duplicated strings in an ItemList are not translatedZolnai Tamás1-8/+11
Change-Id: I72c6e234ff999a6dbed48cc62fe7d39aeb798f35
2014-12-18fdo#39440 reduce scope of local variablesMichael Weghorn2-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>