summaryrefslogtreecommitdiffstats
path: root/registry
Commit message (Collapse)AuthorAgeFilesLines
* 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
* convert RTMethodMode to scoped enumNoel Grandin2015-04-095-12/+12
| | | | Change-Id: I81599570698eb92abf14fa6386d8545c2031e863
* convert RT_ACCESS fields to scoped enumNoel Grandin2015-04-095-68/+68
| | | | Change-Id: Ifef0aaae6208f18c1352fef7c471deb60a97820d
* convert RegError to scoped enumNoel Grandin2015-04-097-329/+329
| | | | Change-Id: I55977c38578cb59deb195d91f8948da2e0905b6a
* convert RegValueType to scoped enumNoel Grandin2015-04-097-130/+130
| | | | Change-Id: Ic672e75db4d7323760577b19490ffa28d38965b6
* remove reg keytype constants and related codeNoel Grandin2015-04-095-57/+0
| | | | | | since we no longer support the RG_LINKTYPE stuff Change-Id: If388ecfa0c475471b99b26155ad554ec702ca734
* convert REG_ constants to scoped enumNoel Grandin2015-04-098-23/+22
| | | | Change-Id: I11d92218c5e0678d497f8964723033e2bd8b3300
* Clean up C-style casts from pointers to voidStephan Bergmann2015-03-286-96/+96
| | | | Change-Id: I192bc9653d8492a850a5e259e15782bd3bb24922
* loplugin:staticfunctionNoel Grandin2015-03-271-2/+2
| | | | Change-Id: I10c30ef28e7321882a720d1e7869a75a89febfc5
* const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann2015-03-262-5/+5
| | | | Change-Id: I32d7a1d4fed6ef5c56f485cd92464f2d0960e5fd
* loplugin:constantfunction: registryNoel Grandin2015-03-193-36/+0
| | | | Change-Id: I0556481aee672451221aa72a8a90d465f3670313
* remove unnecessary parenthesis in return statementsNoel Grandin2015-02-238-29/+29
| | | | | | | found with $ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;' Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
* loplugin:deletedspecialStephan Bergmann2015-02-071-2/+4
| | | | Change-Id: I52253f6b925a943edc2984ce4d9462bd1edb5f51
* Some more loplugin:cstylecast clean-upStephan Bergmann2015-01-284-19/+19
| | | | Change-Id: I132d22e2af3cf673c17d8964f690d48990834884
* loplugin:unreffunStephan Bergmann2015-01-251-2/+0
| | | | Change-Id: Id7e0ddf60e32a27cb3922ae6c64feeec40f8e079
* coverity#1266504 Useless callCaolán McNamara2015-01-251-2/+0
| | | | Change-Id: I11a2760799937393bfa1a91c1d454ccb3c4027e4
* Some more loplugin:cstylecast: registryStephan Bergmann2015-01-202-2/+2
| | | | Change-Id: I5cfb42729b5d3d97076058edb108965f248f86af
* Some loplugin:revisibility clean-upStephan Bergmann2015-01-053-57/+57
| | | | | | | | | | | | | Stumbled across such redundant visibility re-specifications when looking at the odd case of cppu_unsatisfied_iquery_msg declared CPPU_DLLPUBLIC in cppu/source/cppu/cppu_opt.cxx and used in inline code in include/com/sun/star/uno/Reference.hxx with only a declaration lacking CPPU_DLLPUBLIC visible, and wondering how that actually works on Windows. However, this plugin is probably not worth it being run all the time, so committing it to compilerplugins/clang/store/. Change-Id: Ibc3c4e7499213de1b419ce7eb85455cb832e1510
* boost::unordered_map->std::unordered_mapCaolán McNamara2015-01-021-2/+2
| | | | | | you can get debug stl this way Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19