summaryrefslogtreecommitdiff
path: root/l10ntools
AgeCommit message (Collapse)AuthorFilesLines
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>
2014-12-05fdo#38835 strip out OString globalsNoel Grandin3-4/+4
Change-Id: Id2eb27132fd6c1734e50c02617ce1bbb75e294a0
2014-11-28Fold URE: Linux ure/lib/* -> program/Stephan Bergmann1-1/+1
The ../../../program/ links in the URE jar Class-Paths are a temporary kludge (and juh.jar had lacked adaption for Mac OS X). Change-Id: I2542d8a582866485dd61c05df3fc6b4b39a8403d
2014-11-14fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava1-2/+2
Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-11-06Revert "use the new OUString::fromUtf8 method"Stephan Bergmann5-10/+13
This reverts commit 05050cdb23de586870bf479a9df5ced06828d498, not all places that use e.g. OStringToOUString to convert potential UTF-8 are guaranteed to fulfil the prerequisites necessary to use fromUtf8 (and some places like e.g. in codemaker are happy with the best-effort effect of OStringToOUString's OSTRING_TO_OUSTRING_CVTFLAGS).
2014-11-06use the new OUString::fromUtf8 methodNoel Grandin5-13/+10
Change-Id: I771004b7ccab3344a67e827e45bc34c22ffa5f77
2014-11-05markup with event type not checker typeCaolán McNamara2-0/+2
Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
2014-10-02remove checks of __GCC_MINOR__ that are always true in GCC 4.6 or laterMichael Stahl3-6/+0
Change-Id: I4412a0c1e4dee94cd717f24b8df5e257d895f721
2014-10-02remove HAVE_GCC_PRAGMA_DIAGNOSTIC_MODIFY check and macroMichael Stahl3-3/+3
This has been supported by GCC and clang for a very long time. Change-Id: I410a2b39004c932003f8cbefe935aedb109b1163