summaryrefslogtreecommitdiff
path: root/l10ntools
AgeCommit message (Collapse)Author
2015-11-06coverity#1338269 Uncaught exceptionCaolán McNamara
Change-Id: I32bfeb0ded37da05eecc97b92d9f724d59123fce
2015-11-06coverity#1338268 Uncaught exceptionCaolán McNamara
Change-Id: Id954d556c2b65772025b8aaabdbfa9ed4ae7a9b7
2015-11-06coverity#1338261 Uncaught exceptionCaolán McNamara
Change-Id: I3e818ed5ccf395f0fbaf82f8e62870b847fc41c9
2015-11-06coverity#1338256 Uncaught exceptionCaolán McNamara
Change-Id: Iadfb2e59fd40aa178123dbfc20be093cc8f128d2
2015-10-21start next find at previous endpos, not one after previous startposEike Rathke
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 Rathke
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 Bergmann
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12cppcheck:variableScopeNoel Grandin
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 Grandin
- 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 Gelmini
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 Stahl
Change-Id: I490841fd2c526e4ca6d6c94cc7b9dba318c932a1
2015-09-17boost->stdCaolán McNamara
Change-Id: Ifde84627578283bd057d7393eb7e5578ef5c029a
2015-09-03Generated code is compiled with WaE disabled, anywayStephan Bergmann
...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 Weghorn
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 Rathke
As per [MS-LCID] rev. 7.0 2015-06-30 Change-Id: If4980d2b1149d93b39e5b57f5741bbdfb5680516
2015-08-20implicit conversion of NULL constant to 'nullptr_t'Noel Grandin
Change-Id: I98f8df3f27fbd520f0bfbbb39c0e3cc5fa892b0d
2015-08-20new loplugin automemNoel Grandin
find places where we should be using std::unique_ptr Change-Id: I5b9defe778fdc4738ecea381215396874db59e66
2015-08-11new loplugin: defaultparamsNoel Grandin
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 Bergmann
Change-Id: I10206d2a9673296e1c3c7e813e4696d80cbb2bfc
2015-07-30loplugin:unusedmethodsNoel Grandin
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 Abe
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 Grandin
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 Bergmann
Change-Id: I6088ea3fd106aa9563eb71afaf00977cc7c06e20
2015-05-28new clang plugin: loopvartoosmallNoel Grandin
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émeth
Change-Id: I9e2812b02348538aa5bb6a130530d12b31a374dd
2015-05-11loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann
Change-Id: Ia2d6f3e05398395af5e12ce9c1b35484f06a13b4
2015-03-31Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann
Change-Id: I8fc20c94ba0233a91b63ff35f91dc9c8b51a2aba
2015-03-28Clean up C-style casts from pointers to voidStephan Bergmann
Change-Id: Id9c24eea7b52fa28e0fdde39f28dc625aa82ffc2
2015-03-27loplugin:staticfunctionNoel Grandin
Change-Id: I10c30ef28e7321882a720d1e7869a75a89febfc5
2015-03-26const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann
Change-Id: I0f8379517d8e97c24b32ca58749f1c0f2c7e8f49
2015-01-26new loplugin: change virtual methods to non-virtualNoel Grandin
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 Timar
Change-Id: Icf1612f88447e9ae348ef9ad333607a3f6dc8d32
2015-01-20Some more loplugin:cstylecast: l10ntoolsStephan Bergmann
Change-Id: Ifa1c715b5569cb7481a6f1ecaa98306ca7e2d7d8
2015-01-08brute-force find-and-remove of unused #define constants.Noel Grandin
Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
2015-01-02attempt to compare a singular iterator to a singular iteratorCaolán McNamara
falls over during build with debug stl Change-Id: Id7241e1275067f827e4a663eb901c41e4c9873f5
2015-01-02boost::unordered_map->std::unordered_mapCaolán McNamara
you can get debug stl this way Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19
2014-12-28fdo#87754: duplicated strings in an ItemList are not translatedZolnai Tamás
Change-Id: I72c6e234ff999a6dbed48cc62fe7d39aeb798f35
2014-12-18fdo#39440 reduce scope of local variablesMichael Weghorn
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 Grandin
Change-Id: Id2eb27132fd6c1734e50c02617ce1bbb75e294a0
2014-11-28Fold URE: Linux ure/lib/* -> program/Stephan Bergmann
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 Srivastava
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 Bergmann
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 Grandin
Change-Id: I771004b7ccab3344a67e827e45bc34c22ffa5f77
2014-11-05markup with event type not checker typeCaolán McNamara
Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
2014-10-02remove checks of __GCC_MINOR__ that are always true in GCC 4.6 or laterMichael Stahl
Change-Id: I4412a0c1e4dee94cd717f24b8df5e257d895f721
2014-10-02remove HAVE_GCC_PRAGMA_DIAGNOSTIC_MODIFY check and macroMichael Stahl
This has been supported by GCC and clang for a very long time. Change-Id: I410a2b39004c932003f8cbefe935aedb109b1163
2014-09-30l10ntools: std::auto_ptr -> std::unique_ptrStephan Bergmann
Change-Id: I595506b8c6d127abaa8d86bd0ed263edba210bb5
2014-09-19l10ntools/source/export.cxx: be more std::Douglas Mencken
Change-Id: Ia5f96f7d2bf366a6156d198e73e7987a471f28da Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-09-14fdo#80650 src transl. merge must be case sensitiveDavid Tardon
This makes the desired changes in workdir/SrsPartMergeTarget/starmath/source/symbol.src. The only other merged file that is changed is workdir/SrsPartMergeTarget/sw/source/ui/misc/numberingtypelistbox.src, with changes like < "1, 2, 3, ..." ; 4/*SVX_NUM_ARABIC - < "a, b, c, ..." ; 0/*SVX_NUM_CHARS_UPPER_LETTER */; > ; + < "A, B, C, ..." ; 0/*SVX_NUM_CHARS_UPPER_LETTER */; > ; < "a, b, c, ..." ; 1/*SVX_NUM_CHARS_LOWER_LETTER */; > ; This looks okay to me too :-) Change-Id: I729075209027ed1f3fec311c05b631c0f681708b
2014-09-09l10ntools/source/export.cxx: add sanity check to aOutput.mSimpleDouglas Mencken
Change-Id: I05074760f189dfc00ce14ec33002609a56797a6d Reviewed-on: https://gerrit.libreoffice.org/11321 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>