summaryrefslogtreecommitdiffstats
path: root/registry
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:constantparam in registryNoel Grandin2016-03-175-26/+11
| | | | | | | Change-Id: Ic3c60325a722422912d06fe21d072c5f5c4bccb1 Reviewed-on: https://gerrit.libreoffice.org/23322 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Skip some executables in the OS X sandboxed (Mac App Store) caseTor Lillqvist2016-03-041-2/+4
| | | | | | | | I doubt end-users will miss gengal.bin, regview, or regmerge. We need the gengal program at build time. But don't bother shipping it on OS X, at least not in the sandboxed (App Store) case. Change-Id: Id73bef1ba71d126c2d2962fe846e9c31963d6c24
* new loplugin: commaoperatorNoel Grandin2016-02-231-3/+6
| | | | Change-Id: I03f24e61f696b7619855e3c7010aa0d874e5a4ff
* use consistent #define checks for the Windows platformNoel Grandin2016-02-161-1/+1
| | | | | | | | | | stage 1 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro Change-Id: Iece73abdee530937e0737190b1aa97a46cd3075f Reviewed-on: https://gerrit.libreoffice.org/22390 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* loplugin:unusedmethodsNoel Grandin2016-02-121-1/+0
| | | | | | | Change-Id: I0fb06079ff8a9d0b2f5a2929f31846f91391f5c4 Reviewed-on: https://gerrit.libreoffice.org/22229 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Remove excess newlinesChris Sherlock2016-02-097-42/+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>
* Split long command line into multiple shorter onesStephan Bergmann2016-02-041-62/+124
| | | | | | ...maybe that makes Windows more happy Change-Id: I4fe1d73f5638e1efe07162735c94856e0541ae78
* [ABI CHANGE] In SDK, replace regcompare with unoidl-checkStephan Bergmann2016-02-044-2211/+112
| | | | | | | | | | | | | ...as the latter was the more flexible replacement for the former for LO- internal use already. The only gotcha to watch out for is that unoidl-check cannot be used to check "naked" .urd files, but only ones where the content has been moved to /UCR via regmerge. Leave registry/CustomTarget_regcompare_test.mk around to verify that unoidl-check behaves the same as did regcompare on those old-format .urd/.rdb files. Change-Id: Ic13ede48535bf942126c810d88bac7e4081d984e
* Resurrect registry/test/regcompare/Stephan Bergmann2016-02-043-98/+145
| | | | Change-Id: Icf4d3b4137a6b6b70dd543bf4758a79f90e9f57f
* loplugin:fpcomparison in registry/Noel Grandin2016-02-041-2/+3
| | | | | | | Change-Id: Ifdc2df9ba51ce8ef2c8793bb51c28a547f22214f Reviewed-on: https://gerrit.libreoffice.org/21866 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin: unused return valuesNoel Grandin2016-01-181-2/+2
| | | | | | | Change-Id: I9c61a46c57894bc63a57740206c0bcb4a16553af Reviewed-on: https://gerrit.libreoffice.org/21571 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Fix typosAndrea Gelmini2016-01-102-2/+2
| | | | | | | Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* loplugin:unusedmethodsNoel Grandin2015-12-212-53/+0
| | | | Change-Id: Ifafdaf6da0225f244853a0042a6458643b570623
* loplugin:unusedfields in include/package,include/registryNoel Grandin2015-11-251-94/+0
| | | | Change-Id: Iadba30885ba3bab7aaee58e298a9dc6b22c8b977
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-107-152/+152
| | | | Change-Id: Ie5a37a49f1b1c08a20f89e167ca415fc645ed40d
* coverity#1338272 Uncaught exceptionCaolán McNamara2015-11-061-41/+49
| | | | Change-Id: I5b2b7044c77de899ce8dc1e44f3673835bd3ce4c
* coverity#1338270 Uncaught exceptionCaolán McNamara2015-11-061-83/+91
| | | | Change-Id: I5c6fae963c4aabe15d82a51f5801e62656188051
* coverity#1338252 Uncaught exceptionCaolán McNamara2015-11-061-51/+60
| | | | Change-Id: Ia058e42700c7724efd74fd6c5560ccca919fb2e8
* remove extra spaces around ::Noel Grandin2015-10-261-2/+2
| | | | | | just makes it harder to grep for methods Change-Id: I0925db2f9284972fcf7e1e3adf90cbe21f69eb15
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-122-4/+4
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-121-2/+2
| | | | Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
* Abstract out repeated code into a lambdaStephan Bergmann2015-10-071-18/+13
| | | | | | ...to silence a clang-analyzer-deadcode.DeadStores on the last update of pBuffer Change-Id: I144b64c330c178f78109fefadacf62fca3765694
* loplugin:unusedmethodsNoel Grandin2015-10-014-657/+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-301-1/+1
| | | | | | | Change-Id: I42c0ab2012553e11f8c1372103488ba1d291d5a9 Reviewed-on: https://gerrit.libreoffice.org/18965 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* loplugin:removeunusedmethods, remove unused stuffNoel Grandin2015-09-292-17/+0
| | | | Change-Id: I35456b2a3ad2a84a1d045f09cdfb29e4c19b8350
* tdf#39440 cppcheck: redundantAssignmentMichael Weghorn2015-08-231-4/+2
| | | | | | | | | | | This addresses some of cppcheck's "redundantAssignment" warnings ("Variable '<varName>' is reassigned a value before the old one has been used"). Change-Id: I8fd2950d5aa1a3f4bd9e1c4c336abe465d1f657e Reviewed-on: https://gerrit.libreoffice.org/17926 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Force #defined constants to sal_uInt32 to reduce conversion warningsMatteo Casalin2015-08-211-38/+38
| | | | Change-Id: I3968bcd55ab4d0ff6215f76be242acc17cf5ccab
* new loplugin automemNoel Grandin2015-08-202-49/+20
| | | | | | find places where we should be using std::unique_ptr Change-Id: I5b9defe778fdc4738ecea381215396874db59e66
* inline a handful of use-once #definesNoel Grandin2015-07-233-43/+30
| | | | Change-Id: Id2654555c4042f8c0bdbd6bab6507e705f08326b
* loplugin:simplifyboolStephan Bergmann2015-07-161-1/+1
| | | | Change-Id: Ic0115e76076e84cc132a9fc9b0ce6a90cd9f639c
* Fix typosAndrea Gelmini2015-07-081-1/+1
| | | | | Change-Id: I020c7ce4d9aa93cb0292378a410cdb9f7788e920 Signed-off-by: Michael Stahl <mstahl@redhat.com>
* Make RegistryTypeWriter non-copyableStephan Bergmann2015-07-032-33/+3
| | | | | | (and note the operator == vs. = typo) Change-Id: I9c25149967ed9b80f5e8f014de0dfee2e8f485d8
* Remove RegistryTypeWriter_Api indirectionStephan Bergmann2015-07-032-123/+62
| | | | Change-Id: I313adfb72272a631299efb3d2464873853a126be
* Remove unused functionsStephan Bergmann2015-07-032-87/+1
| | | | Change-Id: Ic4058c133a676646bc0c5d1e52207471bb78a1cb
* Make RegistryTypeReader non-copyableStephan Bergmann2015-07-032-25/+3
| | | | | | (and note the operator == vs. = typo) Change-Id: I4567927f3f702f8e74b168becce4002e84d5c7a0
* Remove RegistryTypeReader_Api indirectionStephan Bergmann2015-07-032-176/+96
| | | | Change-Id: I43710d057043a258ec9ab94bca0703434098e6c9
* Remove unused functionsStephan Bergmann2015-07-032-128/+1
| | | | Change-Id: Ib615ff7eb12232b0b83664af134744d96754ffd8
* No need for a C API hereStephan Bergmann2015-07-036-22/+14
| | | | Change-Id: Ie453a7c97a762932884c5c41b13607a170368107
* Move module-private includes from include/registry/ to registry/Stephan Bergmann2015-07-0312-8/+963
| | | | Change-Id: I72bdd6627a191a65d5c3b479c100e30442a8bb9e
* loplugin:unusedmethods registry,storeNoel Grandin2015-07-021-3/+0
| | | | | | | Change-Id: Ie78eb881a7fc47d0cd7b9862bd0cd200153ce77d Reviewed-on: https://gerrit.libreoffice.org/16679 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* remove unnecessary check for null when calling deleteNoel Grandin2015-06-151-5/+2
| | | | | | | | 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
* cppcheck: noExplicitConstructorCaolán McNamara2015-06-111-1/+1
| | | | Change-Id: Id6291335945a8f10cef3afd1a2b3bae65fdf4562
* new clang plugin: loopvartoosmallNoel Grandin2015-05-281-2/+2
| | | | | | | | | | | | | 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
* loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann2015-05-112-8/+8
| | | | Change-Id: I07d9567da23094836134a54afce660500bf1b3f5
* Various #include <sal/log.hxx> fixupsStephan Bergmann2015-04-222-0/+2
| | | | | | | | | | | | | | rtl/string.hxx and rtl/ustring.hxx both unnecessarily #include <sal/log.hxx> (and don't make use of it themselves), but many other files happen to depend on it. Cleaned up some, but something like grep -FwL sal/log.hxx $(git grep -Elw \ 'SAL_INFO|SAL_INFO_IF|SAL_WARN|SAL_WARN_IF') -- \*.cxx) shows lots more files that potentially need fixing before the include can be removed from rtl/string.hxx and rtl/ustring.hxx. Change-Id: Ibf033363e83d37851776f392dc0b077381cd8b90
* remove unnecessary use of void in function declarationsNoel Grandin2015-04-153-3/+3
| | | | | | | | | | | | | | | | ie. void f(void); becomes void f(); I used the following command to make the changes: git grep -lP '\(\s*void\s*\)' -- *.cxx \ | xargs perl -pi -w -e 's/(\w+)\s*\(\s*void\s*\)/$1\(\)/g;' and ran it for both .cxx and .hxx files. Change-Id: I314a1b56e9c14d10726e32841736b0ad5eef8ddd
* loplugin:salboolNoel Grandin2015-04-091-5/+5
| | | | Change-Id: Id4194f4d5bb6fcf064985fddc6f7344a4d34ca04
* fix windows and android buildsNoel Grandin2015-04-091-39/+11
| | | | | | more 'extern C' fallout from my conversion of enum to scoped enum Change-Id: I4c9aabbfbd255775a8f3edc2b7c8c62647f539eb
* fix android buildNoel Grandin2015-04-091-1/+1
| | | | | | | | since after my converting stuff in include/registry/types.h to use 'enum class', it can obviously no longer be "extern C", so drop the "extern C" and rename the file to reflect that. Change-Id: Ia30f5731316525e48531c4785ab7471a428bcf6f
* convert RTReferenceType to scoped enumNoel Grandin2015-04-095-19/+19
| | | | Change-Id: I8320f6f42d5579fbd09450ddca61c4c066de98e4