summaryrefslogtreecommitdiffstats
path: root/registry
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:useuniqueptr extend to check local varsNoel Grandin2017-02-012-7/+4
| | | | | | | | | | | just the simple and obvious case for now, of a local var being allocated and deleted inside a single local block, and the delete happening at the end of the block Change-Id: I3a7a094da543debdcd2374737c2ecff91d644625 Reviewed-on: https://gerrit.libreoffice.org/33749 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin: unnecessary destructor oox..saxNoel Grandin2017-01-252-27/+0
| | | | | | | Change-Id: Ie12bfabc92bce04d702f3e77aa5896366e49245e Reviewed-on: https://gerrit.libreoffice.org/33509 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* bootstrap building with --disable-dynloading on standard linuxCaolán McNamara2016-12-202-2/+6
| | | | | | | Change-Id: I5baf70f0053612cba8b74f54aff11ce25cdeb95a Reviewed-on: https://gerrit.libreoffice.org/32202 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* OSL_TRACE->SAL in framework..salNoel Grandin2016-12-132-2/+0
| | | | | | | Change-Id: I9a897af88aa9f6f7ca98ce521c69b5a4ee8462e9 Reviewed-on: https://gerrit.libreoffice.org/31903 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Remove dead registry/workben/ stuffStephan Bergmann2016-11-244-710/+0
| | | | Change-Id: Ie777efded5a5ebe3aad8d2310e174e6f32a21bc3
* tdf#100302: gbuild: always use $(LFS_CFLAGS)Michael Stahl2016-10-071-1/+0
| | | | | | | | | | Currently LFS_CFLAGS are only used in a handful of libraries but there's no obvious reason why it can't just be set everywhere. Also set it in windows.mk, i have no idea if it's needed for MinGW (certainly not for MSVC). Change-Id: I5f62e3011c2089abbf5539fc54c7ff10e07b1599
* convert storeAccessMode to scoped enumNoel Grandin2016-09-192-31/+31
| | | | Change-Id: I67705cdff0440487019a2992571147d648be1bfc
* loplugin:constantparam in package..stocNoel Grandin2016-09-121-1/+1
| | | | | | | Change-Id: I04b6d14d3dd8d55ccc4d4eb9313bc37959690672 Reviewed-on: https://gerrit.libreoffice.org/28824 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:constantparam in sot..svlNoel Grandin2016-09-092-3/+1
| | | | | | | Change-Id: I08db2db3b90725c556e3ba062da5d62d98f6e882 Reviewed-on: https://gerrit.libreoffice.org/28769 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:countusersofdefaultparams in oox..sdNoel Grandin2016-08-052-8/+8
| | | | | | | Change-Id: I4504939a3957606979c6ac36af6611e1fe072d01 Reviewed-on: https://gerrit.libreoffice.org/27902 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* We don't build regmerge when ENABLE_MACOSX_SANDBOXTor Lillqvist2016-08-041-0/+3
| | | | | | | So skip CustomTarget_regcompare_test. But maybe we should build it and just drop it from the app bundle we package? Change-Id: Ibd8556ef686d2e9b7d4004110bf8beb93faf8202
* Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann2016-05-101-1/+1
| | | | | | | | | | | | | | | | ...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
* loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann2016-04-201-1/+1
| | | | Change-Id: I015b172ea88eef9af37303fafa4bdf9749f114b0
* 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