summaryrefslogtreecommitdiff
path: root/registry
AgeCommit message (Collapse)Author
2016-05-10Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann
...which (in LIBO_INTERNAL_ONLY) for Clang expands to [[clang::fallthrough]] in preparation of enabling -Wimplicit-fallthrough. (This is only relevant for C++11, as neither C nor old C++ has a way to annotate intended fallthroughs.) Could use BOOST_FALLTHROUGH instead of introducing our own SAL_FALLTHROUGH, but that would require adding back in dependencies on boost_headers to many libraries where we carefully removed any remaining Boost dependencies only recently. (At least make SAL_FALLTHROUGH strictly LIBO_INTERNAL_ONLY, so its future evolution will not have any impact on the stable URE interface.) C++17 will have a proper [[fallthroug]], eventually removing the need for a macro altogether. Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
2016-04-20loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann
Change-Id: I015b172ea88eef9af37303fafa4bdf9749f114b0
2016-03-17loplugin:constantparam in registryNoel Grandin
Change-Id: Ic3c60325a722422912d06fe21d072c5f5c4bccb1 Reviewed-on: https://gerrit.libreoffice.org/23322 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-04Skip some executables in the OS X sandboxed (Mac App Store) caseTor Lillqvist
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
2016-02-23new loplugin: commaoperatorNoel Grandin
Change-Id: I03f24e61f696b7619855e3c7010aa0d874e5a4ff
2016-02-16use consistent #define checks for the Windows platformNoel Grandin
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>
2016-02-12loplugin:unusedmethodsNoel Grandin
Change-Id: I0fb06079ff8a9d0b2f5a2929f31846f91391f5c4 Reviewed-on: https://gerrit.libreoffice.org/22229 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-09Remove excess newlinesChris Sherlock
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>
2016-02-04Split long command line into multiple shorter onesStephan Bergmann
...maybe that makes Windows more happy Change-Id: I4fe1d73f5638e1efe07162735c94856e0541ae78
2016-02-04[ABI CHANGE] In SDK, replace regcompare with unoidl-checkStephan Bergmann
...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
2016-02-04Resurrect registry/test/regcompare/Stephan Bergmann
Change-Id: Icf4d3b4137a6b6b70dd543bf4758a79f90e9f57f
2016-02-04loplugin:fpcomparison in registry/Noel Grandin
Change-Id: Ifdc2df9ba51ce8ef2c8793bb51c28a547f22214f Reviewed-on: https://gerrit.libreoffice.org/21866 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-01-18loplugin: unused return valuesNoel Grandin
Change-Id: I9c61a46c57894bc63a57740206c0bcb4a16553af Reviewed-on: https://gerrit.libreoffice.org/21571 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-01-10Fix typosAndrea Gelmini
Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2015-12-21loplugin:unusedmethodsNoel Grandin
Change-Id: Ifafdaf6da0225f244853a0042a6458643b570623
2015-11-25loplugin:unusedfields in include/package,include/registryNoel Grandin
Change-Id: Iadba30885ba3bab7aaee58e298a9dc6b22c8b977
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann
Change-Id: Ie5a37a49f1b1c08a20f89e167ca415fc645ed40d
2015-11-06coverity#1338272 Uncaught exceptionCaolán McNamara
Change-Id: I5b2b7044c77de899ce8dc1e44f3673835bd3ce4c
2015-11-06coverity#1338270 Uncaught exceptionCaolán McNamara
Change-Id: I5c6fae963c4aabe15d82a51f5801e62656188051
2015-11-06coverity#1338252 Uncaught exceptionCaolán McNamara
Change-Id: Ia058e42700c7724efd74fd6c5560ccca919fb2e8
2015-10-26remove extra spaces around ::Noel Grandin
just makes it harder to grep for methods Change-Id: I0925db2f9284972fcf7e1e3adf90cbe21f69eb15
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann
Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
2015-10-07Abstract out repeated code into a lambdaStephan Bergmann
...to silence a clang-analyzer-deadcode.DeadStores on the last update of pBuffer Change-Id: I144b64c330c178f78109fefadacf62fca3765694
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: I42c0ab2012553e11f8c1372103488ba1d291d5a9 Reviewed-on: https://gerrit.libreoffice.org/18965 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2015-09-29loplugin:removeunusedmethods, remove unused stuffNoel Grandin
Change-Id: I35456b2a3ad2a84a1d045f09cdfb29e4c19b8350
2015-08-23tdf#39440 cppcheck: redundantAssignmentMichael Weghorn
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>
2015-08-21Force #defined constants to sal_uInt32 to reduce conversion warningsMatteo Casalin
Change-Id: I3968bcd55ab4d0ff6215f76be242acc17cf5ccab
2015-08-20new loplugin automemNoel Grandin
find places where we should be using std::unique_ptr Change-Id: I5b9defe778fdc4738ecea381215396874db59e66
2015-07-23inline a handful of use-once #definesNoel Grandin
Change-Id: Id2654555c4042f8c0bdbd6bab6507e705f08326b
2015-07-16loplugin:simplifyboolStephan Bergmann
Change-Id: Ic0115e76076e84cc132a9fc9b0ce6a90cd9f639c
2015-07-08Fix typosAndrea Gelmini
Change-Id: I020c7ce4d9aa93cb0292378a410cdb9f7788e920 Signed-off-by: Michael Stahl <mstahl@redhat.com>
2015-07-03Make RegistryTypeWriter non-copyableStephan Bergmann
(and note the operator == vs. = typo) Change-Id: I9c25149967ed9b80f5e8f014de0dfee2e8f485d8
2015-07-03Remove RegistryTypeWriter_Api indirectionStephan Bergmann
Change-Id: I313adfb72272a631299efb3d2464873853a126be
2015-07-03Remove unused functionsStephan Bergmann
Change-Id: Ic4058c133a676646bc0c5d1e52207471bb78a1cb
2015-07-03Make RegistryTypeReader non-copyableStephan Bergmann
(and note the operator == vs. = typo) Change-Id: I4567927f3f702f8e74b168becce4002e84d5c7a0
2015-07-03Remove RegistryTypeReader_Api indirectionStephan Bergmann
Change-Id: I43710d057043a258ec9ab94bca0703434098e6c9
2015-07-03Remove unused functionsStephan Bergmann
Change-Id: Ib615ff7eb12232b0b83664af134744d96754ffd8
2015-07-03No need for a C API hereStephan Bergmann
Change-Id: Ie453a7c97a762932884c5c41b13607a170368107
2015-07-03Move module-private includes from include/registry/ to registry/Stephan Bergmann
Change-Id: I72bdd6627a191a65d5c3b479c100e30442a8bb9e
2015-07-02loplugin:unusedmethods registry,storeNoel Grandin
Change-Id: Ie78eb881a7fc47d0cd7b9862bd0cd200153ce77d Reviewed-on: https://gerrit.libreoffice.org/16679 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.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-11cppcheck: noExplicitConstructorCaolán McNamara
Change-Id: Id6291335945a8f10cef3afd1a2b3bae65fdf4562
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-11loplugin:cstylecast: nop between pointer types of exactly same spellingStephan Bergmann
Change-Id: I07d9567da23094836134a54afce660500bf1b3f5
2015-04-22Various #include <sal/log.hxx> fixupsStephan Bergmann
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
2015-04-15remove unnecessary use of void in function declarationsNoel Grandin
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
2015-04-09loplugin:salboolNoel Grandin
Change-Id: Id4194f4d5bb6fcf064985fddc6f7344a4d34ca04
2015-04-09fix windows and android buildsNoel Grandin
more 'extern C' fallout from my conversion of enum to scoped enum Change-Id: I4c9aabbfbd255775a8f3edc2b7c8c62647f539eb
generated by cgit (git 2.34.1) at 2024-06-25 17:55:16 +0000