summaryrefslogtreecommitdiff
path: root/unoidl
AgeCommit message (Collapse)AuthorFilesLines
2017-08-11convert std::map::insert to std::map::emplace IINoel Grandin2-57/+35
Change-Id: Ief8bd59c903625ba65b75114b7b52c3b7ecbd331 Reviewed-on: https://gerrit.libreoffice.org/41019 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-07-17loplugin:constparams in comphelper,unoidlNoel Grandin1-1/+1
Change-Id: I411d431bd6620c594c5dafd42af6c2a8ac285f3f Reviewed-on: https://gerrit.libreoffice.org/40042 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-06-16Still more tests to suppressStephan Bergmann1-0/+4
...that were missing from 3de594d3347ead24f3211714f0e0010c1434cdf2 "More tests to suppress (all .PHONY test targets should be covered now)" (and one wasn't even properly marked as .PHONY). Thanks to Rene for spotting. Change-Id: Ieed81a3999921bdfe9bca96c6002d8906b2ddfad
2017-06-02Improved loplugin:redundantcast static_cast handling: unoidlStephan Bergmann1-1/+1
Change-Id: Ic959703c5f5ff594d0c430f6756a3fadf09cfd73
2017-04-15Fix typosAndrea Gelmini1-1/+1
Change-Id: I0238202c0dc0380b0e33a9c4567a9513c0afbe91 Reviewed-on: https://gerrit.libreoffice.org/36556 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-04-05loplugin:redundantcast find c-style bool castsNoel Grandin1-2/+2
Change-Id: I3237b93babc67de12c3771aa84766c2141ca93b2 Reviewed-on: https://gerrit.libreoffice.org/36137 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-02-10test coverity sanitizing in-source modellingCaolán McNamara1-0/+7
Change-Id: I31b1f021c414b26372c07a28c235bedb3e5a2b32
2017-02-06loplugin:nullptrJochen Nitschke1-5/+5
seen on tinderbox Linux-F19-x86_64_14-with-check Change-Id: Ica9c2707b43317f7dd1cf99440c6cdb7fe28e0bc Reviewed-on: https://gerrit.libreoffice.org/33938 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-12-21some more fallthrough warning quieteningCaolán McNamara1-0/+2
Change-Id: I61785adbba22a54484b7ffcd6494c75b61b00fc7
2016-12-21quieten the clang fallthrough warningsCaolán McNamara1-10/+10
Change-Id: I73f068971a6908153d887e34425076e6194c657d
2016-12-20bootstrap building with --disable-dynloading on standard linuxCaolán McNamara3-6/+12
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>
2016-10-03Fix typosAndrea Gelmini1-1/+1
Change-Id: Ie75beb4e282a4d1b784a5847262e39cf9c851527 Reviewed-on: https://gerrit.libreoffice.org/29440 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-09-13loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann10-24/+24
The issue of 362d4f0cd4e50111edfae9d30c90602c37ed65a2 "Explicitly mark overriding destructors as 'virtual'" appears to no longer be a problem with MSVC 2013. (The little change in the rewriting code of compilerplugins/clang/override.cxx was necessary to prevent an endless loop when adding "override" to OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager(); in chart2/source/inc/LifeTime.hxx, getting stuck in the leading OOO_DLLPUBLIC_CHARTTOOLS macro. Can't remember what that isAtEndOfImmediateMacroExpansion thing was originally necessary for, anyway.) Change-Id: I534c634504d7216b9bb632c2775c04eaf27e927e
2016-08-30loplugin:stringconstant: adapt to improved OUStringLiteral1 (unoidl)Stephan Bergmann1-1/+1
Change-Id: I8f7fb8bba65f99732bb172b45e309868047f545c
2016-07-27loplugin:countusersofdefaultparams in tools..xmlsecurityNoel Grandin1-1/+1
find methods with default params with only zero or one call site Change-Id: Ie5b30f60e9fe00ba1acf0dfc79b005ded46f05a0 Reviewed-on: https://gerrit.libreoffice.org/27512 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-07loplugin:passstuffbyref also for {css::uno,rtl}::ReferenceStephan Bergmann1-1/+1
Change-Id: Iac3011e99956d1ebb288184fd93555973c76ccdc
2016-05-12Massage code to avoid warningsStephan Bergmann2-8/+6
Change-Id: I112f5e7c845184b9aa5285ecfa40b5bbb34ffde1
2016-05-10Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann1-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
2016-04-26loplugin:nullptrStephan Bergmann1-106/+121
Change-Id: I02e177e07fc5fef225351392c9cd5743a2212967
2016-03-17loplugin:constantparam in registryNoel Grandin1-1/+1
Change-Id: Ic3c60325a722422912d06fe21d072c5f5c4bccb1 Reviewed-on: https://gerrit.libreoffice.org/23322 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-06Fix prev. commitStephan Bergmann1-3/+5
Change-Id: I1dd71d6e35ac2dc5f27a11a3dd269247b1b35d39
2016-02-06Clean up for loops in unoidlStephan Bergmann7-823/+442
Change-Id: I9c2e0a10699580026fbf38f0c857ac148de74f9c
2016-02-04Add --ignore-unpublished to unoidl-checkStephan Bergmann1-17/+35
...to make it a better replacement for SDK's regcompare Change-Id: I7b5eee3d449b8dc7c03601083d06a6d13e4607be
2015-12-14Use emplace_backStephan Bergmann3-184/+126
Change-Id: I006e3c8f411b480917d9cfb9c4f3d082b79c833d
2015-11-10loplugin:nullptr (automatic rewrite; Mac-specific code)Stephan Bergmann1-2/+2
Change-Id: I06921ce068a3ffc9c1502326f03e9b13ee1c6ef6
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann8-37/+37
Change-Id: I7c8f90ae3cb496def3bee9a8a84974dd63478af3
2015-11-06coverity#1338263 Uncaught exceptionCaolán McNamara1-0/+3
Change-Id: Ide3aec7a352786ed441be1f506a5cf488843ea29
2015-11-06coverity#1338262 Uncaught exceptionCaolán McNamara1-0/+3
Change-Id: I95f00b63f21b29bc3c22a4891ebc5668fed723fc
2015-11-06coverity#1338259 Uncaught exceptionCaolán McNamara1-0/+6
Change-Id: I36e2194c304adae5b5c3377519eccc5ef78fd9ea
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann9-23/+23
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-08-21Use sal_uInt16, as required, and use < in for loop conditionMatteo Casalin1-4/+4
Change-Id: Iae44d45d749c6f51be72c0b2d16639c7287c42b2
2015-06-02cppcheck: noExplicitConstructorCaolán McNamara1-1/+1
Change-Id: I8ae8623252546ca94f65fc04b331dd9cafa4fc92
2015-05-28new clang plugin: loopvartoosmallNoel Grandin1-3/+3
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-26cppcheck: noExplicitConstructorCaolán McNamara2-3/+3
Change-Id: If947733a205e8ece1845079be95cbc2d6cbd5029
2015-04-22Various #include <sal/log.hxx> fixupsStephan Bergmann1-0/+1
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-09convert RTReferenceType to scoped enumNoel Grandin1-3/+3
Change-Id: I8320f6f42d5579fbd09450ddca61c4c066de98e4
2015-04-09convert RTMethodMode to scoped enumNoel Grandin1-7/+7
Change-Id: I81599570698eb92abf14fa6386d8545c2031e863
2015-04-09convert RT_ACCESS fields to scoped enumNoel Grandin1-18/+15
Change-Id: Ifef0aaae6208f18c1352fef7c471deb60a97820d
2015-04-09convert RegError to scoped enumNoel Grandin1-22/+22
Change-Id: I55977c38578cb59deb195d91f8948da2e0905b6a
2015-04-09convert RegValueType to scoped enumNoel Grandin1-2/+2
Change-Id: Ic672e75db4d7323760577b19490ffa28d38965b6
2015-04-09convert REG_ constants to scoped enumNoel Grandin1-1/+1
Change-Id: I11d92218c5e0678d497f8964723033e2bd8b3300
2015-03-09V801: Decreased performanceCaolán McNamara1-1/+1
Change-Id: Id8cd45d2844c121f63684734ab3546c24a1aab32
2015-03-02remove executable bit from c / h / cxx / hxx / l filesMichael Stahl1-0/+0
Change-Id: I90d7788208fb86c8aea36c4944ca11d881f11720
2014-12-19unoidl: Use appropriate OUString functions on string constantsStephan Bergmann1-2/+2
Change-Id: I6ff256e091770d67f98b1b6b8d447a22fd720d4e
2014-11-21Silence unhelpful cid#1213376 untrusted loop bound warningsStephan Bergmann1-0/+14
Change-Id: I0a1fcebb268bea942d4bc2d6bf835c3ffb24df3e
2014-11-20This should ideally be consistency-checked in UnoidlProviderStephan Bergmann1-3/+8
...but is not, for performance reasons. Change-Id: I2518eb1f1a2f84cfd836c390a418a2fcb516b82c
2014-11-19Guard against cyclic entity dependenciesStephan Bergmann1-9/+20
...in unoidl-read, although it would be better (but more expensive at start- up) to check consistency in unoidl::Manager. Change-Id: I5f5894fcd51af3aa999fe30621f52bfd11d0e610
2014-11-19Check entity and type name syntaxStephan Bergmann1-50/+181
(For types, only checks their syntax, but not whether they semantically fit in a certain situation, e.g., "boolean" cannot be used as a base interface.) Change-Id: I12f617e74ca13ce2afcec8f611bfdb4912c62960
2014-11-19Check for recursive mapsStephan Bergmann2-38/+83
Change-Id: I50227c41c5b4c5c410939ddfa078b996b5804965
2014-11-19Revert "cosmetic 'en-passant'...."Stephan Bergmann1-746/+696
This reverts commit d4810a814c7c7f3db2df15b62367bd1edc7040f6. Change-Id: Ia062e070c7d7dac5cc4388d1c8e812fb97851d52