summaryrefslogtreecommitdiffstats
path: root/unoidl
Commit message (Collapse)AuthorAgeFilesLines
* Use sal_uInt16, as required, and use < in for loop conditionMatteo Casalin2015-08-211-4/+4
| | | | Change-Id: Iae44d45d749c6f51be72c0b2d16639c7287c42b2
* cppcheck: noExplicitConstructorCaolán McNamara2015-06-021-1/+1
| | | | Change-Id: I8ae8623252546ca94f65fc04b331dd9cafa4fc92
* new clang plugin: loopvartoosmallNoel Grandin2015-05-281-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
* cppcheck: noExplicitConstructorCaolán McNamara2015-05-262-3/+3
| | | | Change-Id: If947733a205e8ece1845079be95cbc2d6cbd5029
* Various #include <sal/log.hxx> fixupsStephan Bergmann2015-04-221-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
* convert RTReferenceType to scoped enumNoel Grandin2015-04-091-3/+3
| | | | Change-Id: I8320f6f42d5579fbd09450ddca61c4c066de98e4
* convert RTMethodMode to scoped enumNoel Grandin2015-04-091-7/+7
| | | | Change-Id: I81599570698eb92abf14fa6386d8545c2031e863
* convert RT_ACCESS fields to scoped enumNoel Grandin2015-04-091-18/+15
| | | | Change-Id: Ifef0aaae6208f18c1352fef7c471deb60a97820d
* convert RegError to scoped enumNoel Grandin2015-04-091-22/+22
| | | | Change-Id: I55977c38578cb59deb195d91f8948da2e0905b6a
* convert RegValueType to scoped enumNoel Grandin2015-04-091-2/+2
| | | | Change-Id: Ic672e75db4d7323760577b19490ffa28d38965b6
* convert REG_ constants to scoped enumNoel Grandin2015-04-091-1/+1
| | | | Change-Id: I11d92218c5e0678d497f8964723033e2bd8b3300
* V801: Decreased performanceCaolán McNamara2015-03-091-1/+1
| | | | Change-Id: Id8cd45d2844c121f63684734ab3546c24a1aab32
* remove executable bit from c / h / cxx / hxx / l filesMichael Stahl2015-03-021-0/+0
| | | | Change-Id: I90d7788208fb86c8aea36c4944ca11d881f11720
* unoidl: Use appropriate OUString functions on string constantsStephan Bergmann2014-12-191-2/+2
| | | | Change-Id: I6ff256e091770d67f98b1b6b8d447a22fd720d4e
* Silence unhelpful cid#1213376 untrusted loop bound warningsStephan Bergmann2014-11-211-0/+14
| | | | Change-Id: I0a1fcebb268bea942d4bc2d6bf835c3ffb24df3e
* This should ideally be consistency-checked in UnoidlProviderStephan Bergmann2014-11-201-3/+8
| | | | | | ...but is not, for performance reasons. Change-Id: I2518eb1f1a2f84cfd836c390a418a2fcb516b82c
* Guard against cyclic entity dependenciesStephan Bergmann2014-11-191-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
* Check entity and type name syntaxStephan Bergmann2014-11-191-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
* Check for recursive mapsStephan Bergmann2014-11-192-38/+83
| | | | Change-Id: I50227c41c5b4c5c410939ddfa078b996b5804965
* Revert "cosmetic 'en-passant'...."Stephan Bergmann2014-11-191-746/+696
| | | | | | This reverts commit d4810a814c7c7f3db2df15b62367bd1edc7040f6. Change-Id: Ia062e070c7d7dac5cc4388d1c8e812fb97851d52
* Avoid overflowStephan Bergmann2014-11-181-3/+9
| | | | Change-Id: Ia80682aeb87225b9bde7398186e121b1d3bdc2ad
* Check that UNOIDL enum has membersStephan Bergmann2014-11-181-0/+4
| | | | Change-Id: I5b362ad374dad5fd4a79b8a7706defb749e25eb3
* Avoid overflowStephan Bergmann2014-11-181-1/+1
| | | | Change-Id: I4afe86dc29788a7b2d2d9c438f182726f80b1cbd
* coverity#1103678 Unchecked dynamic_castCaolán McNamara2014-11-051-2/+2
| | | | Change-Id: Ic87cc3d968447051e52012e59d92e95280d9d35a
* coverity#1242422 Unchecked dynamic_castCaolán McNamara2014-10-241-3/+2
| | | | Change-Id: If5e8b90f32c167bafeb3649f4bc6c31198635947
* cosmetic 'en-passant'....Norbert Thiebaud2014-07-101-696/+746
| | | | Change-Id: I9a787065898004ece4bd55839c563b7395e3a314
* coverity#1224979 Missing break in switchCaolán McNamara2014-07-101-0/+1
| | | | Change-Id: I539ba11497a8c7384b5ca8ee4792548dca139811
* coverity#1224980 Missing break in switchCaolán McNamara2014-07-101-0/+1
| | | | Change-Id: I17379f596fd8b48bd1015538eacc5265cfa83a82
* fix build error 'strlen' is not a member of 'std' locallyDouglas Mencken2014-07-071-0/+1
| | | | | | | Change-Id: I587ef2c38fd9377eef2073a49cd8497b2bd4bf9f Reviewed-on: https://gerrit.libreoffice.org/10076 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* tweak assert so comment appears in abort messageCaolán McNamara2014-06-273-14/+14
| | | | Change-Id: Ibf78e5cd1620f0b61cae030e3870be4a6f87e71d
* Remove circular references between unoidl::Manager and its providersStephan Bergmann2014-05-235-15/+11
| | | | Change-Id: Ic6ad3e2c7f69dd123077a913f35f1971f1e8ca13
* Combine unoidl::loadProvider and unoidl::Manager::addProviderStephan Bergmann2014-05-235-37/+36
| | | | Change-Id: I1240656cc2a4d713c838eb80fa90ce3485aad614
* coverity#1210179 Uninitialized pointer fieldCaolán McNamara2014-04-291-1/+1
| | | | Change-Id: I09534eab67c0ae38cd6965c3b63227d92b72c646
* unoidl-check: Also check for invalid UNOIDL identifiersStephan Bergmann2014-04-111-0/+270
| | | | | | | | ...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
* First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann2014-03-269-23/+23
| | | | | | | ...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
* Find places where OUString and OString are passed by value.Noel Grandin2014-03-181-3/+3
| | | | | | | | | | 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
* Yet another one undefined comparison of 'const' and a 'non-const' iteratorsDouglas Mencken2014-02-241-1/+1
| | | | | | | | 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>
* tests belong to check targetBjoern Michaelsen2014-02-181-1/+3
| | | | | | | | | | | | - 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>
* OK to add optional bases to published old-style servicesStephan Bergmann2014-01-081-27/+36
| | | | Change-Id: Ie6f5f2a634171d9618f3229bcc18ad48fff80d35
* typo fixesAndras Timar2013-12-201-1/+1
| | | | Change-Id: Ia5f104bfd707bcf4e159c78ca2764c861fb0b6d9
* Silence false "may be used uninitialized" warningsStephan Bergmann2013-11-271-2/+5
| | | | Change-Id: Ie708de4a7fbd34e1ab3c417688de79d5bdf0e0b8
* Fix unoidl sourceprovider old-style service duplicate member checksStephan Bergmann2013-11-261-11/+45
| | | | Change-Id: Ifa19a427543e76053e9b249c479e1e13f3599bcc
* Remove outdated TODO (@deprecated annotation are handled fine)Stephan Bergmann2013-11-261-1/+0
| | | | Change-Id: Iab32e6974a181e437630aaae2ea73eb6062ec0ee
* Duplicate base checking code already covers check for opt base != XInterfaceStephan Bergmann2013-11-261-1/+0
| | | | Change-Id: Icd47d87f93641a0b24a96a4be8d3d120f0405f2d
* Fix unoidl sourceprovider interface base and member checksStephan Bergmann2013-11-254-83/+638
| | | | | | ...and enable tests shared with idlc Change-Id: I422b16c9b2636835d276cc2085cb640073894c97
* Fix unoidl sourceprovider polystruct argument checksStephan Bergmann2013-11-221-30/+37
| | | | Change-Id: Ib7ee3188b87b9b4216214b65347e34970e1b92a5
* Fix unoidl sourceprovider typedef checkStephan Bergmann2013-11-221-6/+6
| | | | Change-Id: I7faa689c803f83687cfcc39dc7c4fd145584e849
* Fix unoidl sourceprovider recursive struct/exception member checkStephan Bergmann2013-11-221-1/+8
| | | | Change-Id: Icd156745da10011611590b6b62cd0ec2df2b1266
* Fix unoidl sourceprovider "published" checksStephan Bergmann2013-11-222-59/+348
| | | | Change-Id: I93b9fcc2b20ed7a7c160a9ef3294b6e578678f53
* fixincludeguards.sh: uno*Thomas Arnhold2013-11-054-8/+8
| | | | Change-Id: I8fb3c25cd583b688104a0ee8620696e7b2958ab5