summaryrefslogtreecommitdiff
path: root/unoidl
AgeCommit message (Collapse)Author
2015-06-02cppcheck: noExplicitConstructorCaolán McNamara
Change-Id: I8ae8623252546ca94f65fc04b331dd9cafa4fc92
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-26cppcheck: noExplicitConstructorCaolán McNamara
Change-Id: If947733a205e8ece1845079be95cbc2d6cbd5029
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-09convert RTReferenceType to scoped enumNoel Grandin
Change-Id: I8320f6f42d5579fbd09450ddca61c4c066de98e4
2015-04-09convert RTMethodMode to scoped enumNoel Grandin
Change-Id: I81599570698eb92abf14fa6386d8545c2031e863
2015-04-09convert RT_ACCESS fields to scoped enumNoel Grandin
Change-Id: Ifef0aaae6208f18c1352fef7c471deb60a97820d
2015-04-09convert RegError to scoped enumNoel Grandin
Change-Id: I55977c38578cb59deb195d91f8948da2e0905b6a
2015-04-09convert RegValueType to scoped enumNoel Grandin
Change-Id: Ic672e75db4d7323760577b19490ffa28d38965b6
2015-04-09convert REG_ constants to scoped enumNoel Grandin
Change-Id: I11d92218c5e0678d497f8964723033e2bd8b3300
2015-03-09V801: Decreased performanceCaolán McNamara
Change-Id: Id8cd45d2844c121f63684734ab3546c24a1aab32
2015-03-02remove executable bit from c / h / cxx / hxx / l filesMichael Stahl
Change-Id: I90d7788208fb86c8aea36c4944ca11d881f11720
2014-12-19unoidl: Use appropriate OUString functions on string constantsStephan Bergmann
Change-Id: I6ff256e091770d67f98b1b6b8d447a22fd720d4e
2014-11-21Silence unhelpful cid#1213376 untrusted loop bound warningsStephan Bergmann
Change-Id: I0a1fcebb268bea942d4bc2d6bf835c3ffb24df3e
2014-11-20This should ideally be consistency-checked in UnoidlProviderStephan Bergmann
...but is not, for performance reasons. Change-Id: I2518eb1f1a2f84cfd836c390a418a2fcb516b82c
2014-11-19Guard against cyclic entity dependenciesStephan Bergmann
...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 Bergmann
(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 Bergmann
Change-Id: I50227c41c5b4c5c410939ddfa078b996b5804965
2014-11-19Revert "cosmetic 'en-passant'...."Stephan Bergmann
This reverts commit d4810a814c7c7f3db2df15b62367bd1edc7040f6. Change-Id: Ia062e070c7d7dac5cc4388d1c8e812fb97851d52
2014-11-18Avoid overflowStephan Bergmann
Change-Id: Ia80682aeb87225b9bde7398186e121b1d3bdc2ad
2014-11-18Check that UNOIDL enum has membersStephan Bergmann
Change-Id: I5b362ad374dad5fd4a79b8a7706defb749e25eb3
2014-11-18Avoid overflowStephan Bergmann
Change-Id: I4afe86dc29788a7b2d2d9c438f182726f80b1cbd
2014-11-05coverity#1103678 Unchecked dynamic_castCaolán McNamara
Change-Id: Ic87cc3d968447051e52012e59d92e95280d9d35a
2014-10-24coverity#1242422 Unchecked dynamic_castCaolán McNamara
Change-Id: If5e8b90f32c167bafeb3649f4bc6c31198635947
2014-07-10cosmetic 'en-passant'....Norbert Thiebaud
Change-Id: I9a787065898004ece4bd55839c563b7395e3a314
2014-07-10coverity#1224979 Missing break in switchCaolán McNamara
Change-Id: I539ba11497a8c7384b5ca8ee4792548dca139811
2014-07-10coverity#1224980 Missing break in switchCaolán McNamara
Change-Id: I17379f596fd8b48bd1015538eacc5265cfa83a82
2014-07-07fix build error 'strlen' is not a member of 'std' locallyDouglas Mencken
Change-Id: I587ef2c38fd9377eef2073a49cd8497b2bd4bf9f Reviewed-on: https://gerrit.libreoffice.org/10076 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2014-06-27tweak assert so comment appears in abort messageCaolán McNamara
Change-Id: Ibf78e5cd1620f0b61cae030e3870be4a6f87e71d
2014-05-23Remove circular references between unoidl::Manager and its providersStephan Bergmann
Change-Id: Ic6ad3e2c7f69dd123077a913f35f1971f1e8ca13
2014-05-23Combine unoidl::loadProvider and unoidl::Manager::addProviderStephan Bergmann
Change-Id: I1240656cc2a4d713c838eb80fa90ce3485aad614
2014-04-29coverity#1210179 Uninitialized pointer fieldCaolán McNamara
Change-Id: I09534eab67c0ae38cd6965c3b63227d92b72c646
2014-04-11unoidl-check: Also check for invalid UNOIDL identifiersStephan Bergmann
...but only in those parts of registry B that are not also in registry A. That way, we can detect newly introduced violations while ignoring the old (published) violations for backwards compatibility. Change-Id: Ifb8ea98fffca29647aa6677a5ade86e5b194ddee
2014-03-26First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann
...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
2014-03-18Find places where OUString and OString are passed by value.Noel Grandin
It's not very efficient, because we generally end up copying it twice - once into the parameter and again into the destination OUString. So I create a clang plugin that finds such places and generates a warning so that we can convert them to pass-by-reference. Change-Id: I5341a6ea9e3190f4b4c05c42c85595e3dcd83361
2014-02-24Yet another one undefined comparison of 'const' and a 'non-const' iteratorsDouglas Mencken
Fixes: no match for ‘operator!=’ in ‘i != std::vector<_Tp, _Alloc>::rend() [with _Tp = rtl::OUString, _Alloc = std::allocator<rtl::OUString>]()’ Change-Id: I7b4ba07ebe51c73893a3d6b77dcf5681b7638efb Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-02-18tests belong to check targetBjoern Michaelsen
- the CustomTarget seems to be a test, not a build target - worse, it is .PHONY - ... and eats 30% of the top-level build-nocheck time alone - so hopefully, we can we at least make it a check target Change-Id: I113777cbd849aab9e8d0a77184e31194bc07232e Reviewed-on: https://gerrit.libreoffice.org/8111 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2014-01-08OK to add optional bases to published old-style servicesStephan Bergmann
Change-Id: Ie6f5f2a634171d9618f3229bcc18ad48fff80d35
2013-12-20typo fixesAndras Timar
Change-Id: Ia5f104bfd707bcf4e159c78ca2764c861fb0b6d9
2013-11-27Silence false "may be used uninitialized" warningsStephan Bergmann
Change-Id: Ie708de4a7fbd34e1ab3c417688de79d5bdf0e0b8
2013-11-26Fix unoidl sourceprovider old-style service duplicate member checksStephan Bergmann
Change-Id: Ifa19a427543e76053e9b249c479e1e13f3599bcc
2013-11-26Remove outdated TODO (@deprecated annotation are handled fine)Stephan Bergmann
Change-Id: Iab32e6974a181e437630aaae2ea73eb6062ec0ee
2013-11-26Duplicate base checking code already covers check for opt base != XInterfaceStephan Bergmann
Change-Id: Icd47d87f93641a0b24a96a4be8d3d120f0405f2d
2013-11-25Fix unoidl sourceprovider interface base and member checksStephan Bergmann
...and enable tests shared with idlc Change-Id: I422b16c9b2636835d276cc2085cb640073894c97
2013-11-22Fix unoidl sourceprovider polystruct argument checksStephan Bergmann
Change-Id: Ib7ee3188b87b9b4216214b65347e34970e1b92a5
2013-11-22Fix unoidl sourceprovider typedef checkStephan Bergmann
Change-Id: I7faa689c803f83687cfcc39dc7c4fd145584e849
2013-11-22Fix unoidl sourceprovider recursive struct/exception member checkStephan Bergmann
Change-Id: Icd156745da10011611590b6b62cd0ec2df2b1266
2013-11-22Fix unoidl sourceprovider "published" checksStephan Bergmann
Change-Id: I93b9fcc2b20ed7a7c160a9ef3294b6e578678f53
2013-11-05fixincludeguards.sh: uno*Thomas Arnhold
Change-Id: I8fb3c25cd583b688104a0ee8620696e7b2958ab5
2013-10-24fdo#67725 unoidl::AggregatingCursor must wrap modules for aggregation, tooStephan Bergmann
Otherwise cppuhelper::TypeManager::createTypeDescriptionEnumeration, sitting on top such an AggregatingCursor, will miss any entities from provider P' in module M if any previous provider P contains the same module M. That happened when climaker generates cli_oootypes.dll, where the enumeration missed everything from offapi in top-level module "com" because it had already seen udkapi's "com", and only reported the handful of entities under offapi's other top-level module "org" (which does not appear in udkapi). Change-Id: If538391bde22bcc346417b5988cf12023f0d4172