summaryrefslogtreecommitdiffstats
path: root/lingucomponent
Commit message (Collapse)AuthorAgeFilesLines
* update loplugin stylepolice to check local pointers varsNoel Grandin2016-04-261-14/+14
| | | | | | | | | | | | are actually pointer vars. Also convert from regex to normal code, so we can enable this plugin all the time. Change-Id: Ie36a25ecba61c18f99c77c77646d6459a443cbd1 Reviewed-on: https://gerrit.libreoffice.org/24391 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:salboolStephan Bergmann2016-04-221-3/+3
| | | | Change-Id: I186376ceeb53126bc31d63d00a988be6cb7a5637
* new plugin stylepoliceNoel Grandin2016-04-211-11/+11
| | | | | | | check for local variables which follow our member field naming convention, which is highly confusing Change-Id: Idacedf7145d09843e96a584237b385f7662eea10
* loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann2016-04-201-3/+3
| | | | Change-Id: Ia4afa4cc65215053ac44cc0b48e528064837c49e
* try next hunspell 1.4.0 prerelease milestoneCaolán McNamara2016-04-181-1/+0
| | | | | | | | try using reverse iterators here then and remove dictmgr which we don't seem to use or need Change-Id: I981fccc4ca7562d5ab707dc2f605e409169dec07
* clang-tidy performance-unnecessary-copy-initializationNoel Grandin2016-04-181-2/+1
| | | | | | | | | | | probably not much performance benefit, but it sure is good at identifying leftover intermediate variables from previous refactorings. Change-Id: I3ce16fe496ac2733c1cb0a35f74c0fc9193cc657 Reviewed-on: https://gerrit.libreoffice.org/24026 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* clean-up: unused using declarations and includesJochen Nitschke2016-04-182-3/+0
| | | | | | | | | | | | | | Searched source for using declarations. Checked if those symbols reappear in the source file, even in comments or dead code but not in #include statements. If they don't reappear, remove the declaration. Remove includes whose symbol got removed. Change-Id: Ibb77163f63c1120070e9518e3dc0a78c6c59fab0 Reviewed-on: https://gerrit.libreoffice.org/24148 Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:passstuffbyref in variousNoel Grandin2016-04-141-2/+2
| | | | Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
* tdf#94306 replace boost::noncopyable ...Jochen Nitschke2016-04-111-3/+3
| | | | | | | | | | | | | ... in modules editeng to oox. Replace with C++11 delete copy-constructur and copy-assignment. Remove boost/noncopyable.hpp includes and one unused boost/checked_delete.hpp include in linguistic. Change-Id: I5a38d8e5ac1b4286bdeb3858d56490a53d13fe80 Reviewed-on: https://gerrit.libreoffice.org/23928 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* cleanup: remove unused Reference.h(xx) includesJochen Nitschke2016-04-021-1/+0
| | | | | | | | | and unused using-declarations from Reference.h Change-Id: I297a7ae6044fa329d245ecf08fd5c4cb930f5b19 Reviewed-on: https://gerrit.libreoffice.org/23735 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* use consistent #define checks for the Windows platformNoel Grandin2016-02-176-10/+10
| | | | | | | | | | | | 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>
* [API CHANGE] add operator==/!= to UNO structsNoel Grandin2016-02-111-7/+0
| | | | | | | | | this is useful now that we are storing UNO structs in std::vector Change-Id: Ic558bcd669bd2b3cdf9eb8393269eb906ac52369 Reviewed-on: https://gerrit.libreoffice.org/22257 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Attempted fix for "Resolves: #i126762# Ignore dictionary not in specified...Stephan Bergmann2016-02-021-4/+17
| | | | | | | | | | | | | | | | | | | | | | | ...location." Caused many crashed during "make check", when aLocaleNames is a sequence of 17 ("ar-SA", "ar-DZ", ...), but aLcoations merely a sequence of 2 (".../dict-ar/ar.aff", ".../dict-ar/ar.dic"). From comments further down below, it looks like aLocations will always contain exactly two entries ("also both files have to be in the same directory and the file names must only differ in the extension (.aff/.dic)"), and that all the aLocaleNames members share the same set of aLocations ("Thus here we work-around this by adding the same dictionary several times. Once for each of its supported locales"), so that it would appear to be OK to just check once for the existenceof aLocations[0]. (There is a check for if (aDictIt->aLocaleNames.getLength() > 0 && aDictIt->aLocations.getLength() > 0) below, so it might be that these can be empty, or it might just be "defensive programming." Play it safe, and check here that aLocations is not empty.) Change-Id: I82bea6571983e397a9e164b294a5ba656b511a67
* Resolves: #i126762# Ignore dictionary not in specified location.Kay Schenk2016-02-021-1/+9
| | | | | | | | Damaged "registrymodifications.xcu" causing problems with backups and spell checker (cherry picked from commit 983e71fa37e12fbae29eddc2b855ddaf6280cc0e) Change-Id: I279275e11fb2ee6a223c7905652165d1b7a2e0ac
* OSX build fixeNoel Grandin2016-01-251-0/+1
| | | | | | after my OInterfaceContainer2 change Change-Id: I9ac967ff474fe1a68876d8ffedbdcb94013a8119
* InterfaceContainer2 with vector instead of SequenceNoel Grandin2016-01-256-4/+6
| | | | | | | | | | create an InterfaceContainer2 class to replace InterfaceContainer. It uses a std::vector instead of a Sequence for the mutable listener list, which provides far better performance. Switch all our internal use-sites to the new class. Change-Id: I6b56cfa511ded2395faa22e68fab3b2f16c3cb88
* loplugin: unused return valuesNoel Grandin2016-01-181-1/+0
| | | | | | | Change-Id: I9c61a46c57894bc63a57740206c0bcb4a16553af Reviewed-on: https://gerrit.libreoffice.org/21571 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedmethods unused return value in l10ntools,linguisticNoel Grandin2016-01-152-13/+0
| | | | Change-Id: I11571c25e527787a2054bc8106a27c44b8a96116
* Fix typosAndrea Gelmini2016-01-103-3/+3
| | | | | | | 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 in lingu,lotuswordpro,mysqlcNoel Grandin2015-12-222-8/+5
| | | | Change-Id: I5866c8d95e04714e81a45e73bf00a430859a4327
* cppcheck: noExplicitConstructorCaolán McNamara2015-11-241-1/+1
| | | | Change-Id: I1934441858baeeb41a46f694dbcef2d846b308b7
* Don't assume sal_Unicode is unsigned shortStephan Bergmann2015-11-171-6/+6
| | | | Change-Id: Ia93c541ea21be89578789d58f642109184a18685
* no need to take a mutex when only dealing with stack-local dataNoel Grandin2015-11-165-24/+0
| | | | | | | Change-Id: Ie45e626aad55a8174a53b769a98601bf54dedf65 Reviewed-on: https://gerrit.libreoffice.org/19979 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* use initialiser list for Sequence<OUString>Noel Grandin2015-11-164-8/+4
| | | | | | | Change-Id: Ia5e47261d1fc6fac2d046656c05a1c5eedb07e02 Reviewed-on: https://gerrit.libreoffice.org/19978 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* use initialiser for Sequence<OUString>Noel Grandin2015-11-151-2/+1
| | | | | | | | | | | | | performed using: git grep -lP 'Sequence.*OUString.*\(1\)' | xargs perl -0777 -pi -e "s/Sequence<OUString> (\w+)\(1\); .*\[0\] = (\S+);/Sequence<OUString> \1 { \2 };/g" Change-Id: I4da56c80fa09bfc1e8f868794001e9921431e09f Reviewed-on: https://gerrit.libreoffice.org/19968 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:nullptr (automatic rewrite; Mac-specific code)Stephan Bergmann2015-11-102-14/+14
| | | | Change-Id: I48e80668c76b7fb94dfa2876ae8ca5e895cea3be
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-106-76/+76
| | | | Change-Id: I1aa63ce279fe3914e0f5153b7d8559880a3fccc3
* use uno::Reference::set method instead of assignmentNoel Grandin2015-11-041-1/+1
| | | | Change-Id: I11822c50fa66d038a3d6f38054ab35c2e613f077
* com::sun::star->css in lingucomponentNoel Grandin2015-10-237-39/+36
| | | | Change-Id: Iceef10c51db88b3e85f118a366de45f0f70bbf21
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-126-65/+65
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-125-10/+10
| | | | Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
* Clean up remaining utl::LocalFileHelper::Convert...Stephan Bergmann2015-08-191-2/+2
| | | | | | | | | | | together with 58d68a1bc9146334376206ae7ba8b1a6594a1040 "ConvertURLToSystemPath->getSystemPathFromFileURL" and 28f3464a571a23a2c16bd0980e9021b95d011511 "ConvertSystemPathToURL->getFileURLFromSystemPath," this replaces all those Convert... functionality with direct calls to the corresponding osl::FileBase functions. Change-Id: I2876171cd337a5eb939d25d8cf1e0c1253ff73a7
* lingucomponent: tdf#88206 replace cppu::WeakImplHelper*Takeshi Abe2015-08-196-16/+12
| | | | | | | | | with the variadic variants. Change-Id: Ibe00b26512a4895a603a084f4e816c74b99894df Reviewed-on: https://gerrit.libreoffice.org/17850 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedmethodsNoel Grandin2015-07-301-11/+3
| | | | | | | Change-Id: Ib4d77ee01e7362f5951f81fceeca3c489872d971 Reviewed-on: https://gerrit.libreoffice.org/17378 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Fix typosAndrea Gelmini2015-07-192-2/+2
| | | | | | | Change-Id: I52cbaad71560d73f5e24f3de3cd62b00d678dd6c Reviewed-on: https://gerrit.libreoffice.org/17187 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* Replace boost::scoped_array<T> with std::unique_ptr<T[]>Takeshi Abe2015-06-171-5/+5
| | | | | | | | | | | | | | | | | | | | | | 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>
* I guess we can revert this "temporary check" nowDavid Tardon2015-06-151-5/+0
| | | | | | | | | | | On the other side, it has been in the code for only 2 years, so who knows .-) Revert "libexttextcat: fix the --with-system case too..." This reverts commit a81f44c707c4df56e7b999fa3b656d7fa4543d1b. Change-Id: I4e02ebe971f8a838bf2a1630ed38d4408245bf37
* loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2015-06-084-16/+16
| | | | Change-Id: I617d55415aee923d0734f3f3b74948b26132611b
* loplugin:loopvartoosmallStephan Bergmann2015-06-021-1/+1
| | | | Change-Id: I33f4e74f97ba52b9240e6304d53eb256bd822060
* loplugin:cstylecast: deal with those that are (technically) const_castStephan Bergmann2015-06-022-2/+2
| | | | Change-Id: I0730744b8424bc6dea5e8016199088f86e9570f5
* loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann2015-05-111-2/+2
| | | | Change-Id: Ic494c8e3b072adceca984a3ceec02088ac08e880
* loplugin:staticmethodsNoel Grandin2015-04-232-6/+6
| | | | Change-Id: Id7fc12bc61c6e803f45cc60556d9cf54ac2e529f
* convert CapType to scoped enumNoel Grandin2015-04-082-8/+8
| | | | Change-Id: I88fa3672a1f933ae818368c9bc400c6a845babb6
* loplugin:redundantcastStephan Bergmann2015-04-021-1/+1
| | | | Change-Id: I3ef78881cfed8b66f067e9164efcccf425fb549c
* Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann2015-03-313-3/+3
| | | | Change-Id: I34dc9028a273b6a22395fe662dd071505989dc5c
* Clean up C-style casts from pointers to voidStephan Bergmann2015-03-281-2/+2
| | | | Change-Id: I76204cd86daa7b95f9c70276459d7dd112198135
* const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann2015-03-261-2/+2
| | | | Change-Id: Id916058c4a1483a7a050d93cce45926fbd0df9ed
* V813: Decreased performanceCaolán McNamara2015-03-044-11/+8
| | | | Change-Id: I8a7528366156b288dc422b09cff0d5a32cde3c91
* use initializer list here tooDavid Tardon2015-03-011-3/+3
| | | | Change-Id: I30e4374dc44a58cb1b7315e3d7bfbed99c77030a
* initialize membersDavid Tardon2015-03-011-2/+3
| | | | Change-Id: I907fdbb4ca12a25fbfa99c6fa345c3c07aea0833