summaryrefslogtreecommitdiff
path: root/UnoControls
AgeCommit message (Collapse)AuthorFilesLines
2017-02-06Add missing #includesStephan Bergmann1-0/+1
...and remove some unncessary using directives/declarations, in preparation of removing now-unnecessary #includes from cppumaker-generated files, post e57ca02849c3d87142ff5ff9099a212e72b8139c "Remove dynamic exception specifications". Change-Id: Iaf1f268871e2ee1d1c76cf90f03557527ebc9067
2017-01-26Remove dynamic exception specificationsStephan Bergmann19-408/+326
...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at <https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html> "Dynamic Exception Specifications" for details. Most changes have been done automatically by the rewriting loplugin:dynexcspec (after enabling the rewriting mode, to be committed shortly). The way it only removes exception specs from declarations if it also sees a definition, it identified some dead declarations-w/o-definitions (that have been removed manually) and some cases where a definition appeared in multiple include files (which have also been cleaned up manually). There's also been cases of macro paramters (that were used to abstract over exception specs) that have become unused now (and been removed). Furthermore, some code needed to be cleaned up manually (avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no configurations available that would actually build that code. Missing @throws documentation has not been applied in such manual clean-up. Change-Id: I3408691256c9b0c12bc5332de976743626e13960 Reviewed-on: https://gerrit.libreoffice.org/33574 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2017-01-25Fix typo in comments: relativ -> relativeTakeshi Abe1-2/+2
Change-Id: I58833e503e93b8057f92710c5828bba6cecd4e09 Reviewed-on: https://gerrit.libreoffice.org/33488 Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp> Tested-by: Takeshi Abe <tabe@fixedpoint.jp>
2017-01-19New loplugin:dynexcspec: Add @throws documentation, UnoControlsStephan Bergmann1-0/+4
Change-Id: Iedbef454a22b34df777e6c32fad95fed96f2fdce
2017-01-17use rtl::Reference in BaseControlNoel Grandin2-14/+11
no need to hold both raw pointer and a uno::Reference to the same object Change-Id: I6aa2b2e259aed77ab038fc72a3ece7165e477f50 Reviewed-on: https://gerrit.libreoffice.org/33163 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-12-09Catch some misuses of VclPtr constructionStephan Bergmann2-2/+2
...that go unnoticed due to the non-explicit VclPtr::oeprator reference_type * Change-Id: Ia63edf8425d3ecb7c7f98eb56a710ac0cceccb67
2016-10-25loplugin:expandablemethods in UnoControls..vbahelperNoel Grandin2-31/+19
Change-Id: I7e607df2a1f6d4eb207ee0d3eb30f41fac44a0c6 Reviewed-on: https://gerrit.libreoffice.org/30262 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-10-19loplugin:expandablemethods in UnoControls..vbahelperNoel Grandin2-24/+2
Change-Id: Iec2c5e13119d97c9ca87e083c0f7cb00227e5231 Reviewed-on: https://gerrit.libreoffice.org/29997 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-10-03Fix typosAndrea Gelmini4-8/+8
Change-Id: Ie44267e7c9f92eb1920f05235826abff761251f1 Reviewed-on: https://gerrit.libreoffice.org/29430 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-09-13loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann9-9/+9
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-07-27improve passstuffbyref return analysisNoel Grandin2-2/+2
Change-Id: I4258bcc97273d8bb7a8c4879fac02a427f76e18c Reviewed-on: https://gerrit.libreoffice.org/27317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-15tdf#100726: Improve readability of OUString concatenation a bitnadith1-8/+4
Change-Id: I6c51b9004089721f199bc35b1e5c13df36847f02 Reviewed-on: https://gerrit.libreoffice.org/27227 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-31teach refcounting clang plugin about uno::ReferenceNoel Grandin1-1/+2
uno::Reference is only allowed to used with classes that have a ::static_type member. So convert all those places to rtl::Reference. Maybe we need some LIBO_INTERNAL_ONLY constructors on rtl::Reference and uno::Reference to make this a little smoother? Change-Id: Icdcb35d71ca40a87b1dc474096776412adbfc7e3 Reviewed-on: https://gerrit.libreoffice.org/25516 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-22Fix typosAndrea Gelmini1-1/+1
Change-Id: Iea11112a1bf1ff934de0e355b9e81979aa1966a3 Reviewed-on: https://gerrit.libreoffice.org/24286 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-04-21clang-tidy modernize-loop-convert in a*Noel Grandin1-12/+6
Change-Id: If8eada464c6dc7350e50fc90eb27aa47bd90e198
2016-04-14loplugin:passstuffbyref in variousNoel Grandin1-4/+4
Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
2016-03-18tools: split out color macros into own headerChris Sherlock1-5/+1
Change-Id: Ib983601d60a3dae1e738bb385dfbb36094bdb064
2016-02-23new loplugin: commaoperatorNoel Grandin1-4/+8
Change-Id: I03f24e61f696b7619855e3c7010aa0d874e5a4ff
2016-01-13loplugin:unusedmethods unused return value in UnoControlNoel Grandin1-2/+0
Change-Id: I6099667c1a057e36cd10a9d79792fadbb84fbf26
2015-12-21loplugin:unusedfields UnoControls,ucbNoel Grandin2-2/+0
Change-Id: I2ff2db21d79d80785f3b6849c879537d5111a960
2015-11-16no need to take a mutex when only dealing with stack-local dataNoel Grandin1-1/+0
Change-Id: Ie45e626aad55a8174a53b769a98601bf54dedf65 Reviewed-on: https://gerrit.libreoffice.org/19979 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser for Sequence<OUString>Noel Grandin1-2/+1
replaced using: git grep -lP 'Sequence.*OUString.*\(\s*1\s*\)' | xargs perl -0777 -pi -e "s/Sequence<\s*OUString\s*> (\w+)\(\s*1\s*\); .*\[0\] = (\S+);/Sequence<OUString> \1 { \2 };/g" Change-Id: I20ad0489da887a9712982531c3b127339bb8b3b9 Reviewed-on: https://gerrit.libreoffice.org/19969 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-11com::sun::star->css in UnoControls/Noel Grandin12-1438/+411
and remove some noise comments Change-Id: I290ec365b58fa1b21838a6faf84006434c3e7bbd Reviewed-on: https://gerrit.libreoffice.org/19904 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann8-30/+30
Change-Id: I8be8b75317f0a8cf9005feefea05c053b76c7b0f
2015-10-30use uno::Reference::set method instead of assignmentNoel Grandin7-13/+13
Change-Id: I58410209f32f988f258a588364e0b037c2790211
2015-10-19cleanup some local var declarationsNoel Grandin1-2/+2
found with git grep -nP '(\w+)\s+=\s+\g1\(' | lots-of-hand-filtering Change-Id: I598b0cfa6607823eaef09d95e610e05145c727f7
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann9-192/+192
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-09-30Fix typosAndrea Gelmini2-2/+2
Change-Id: I3926eca56ac9f54f9ddbf88610888ce97cf00b34 Reviewed-on: https://gerrit.libreoffice.org/18990 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-08-03new loplugin: refcountingNoel Grandin2-12/+12
This was a feature requested by mmeeks, as a result of tdf#92611. It validates that things that extend XInterface are not directly heap/stack-allocated, but have their lifecycle managed via css::uno::Reference or rtl::Reference. Change-Id: I28e3b8b236f6a4a56d0a6d6f26ad54e44b36e692 Reviewed-on: https://gerrit.libreoffice.org/16924 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-13Make content of OSL_ASSERT, DBG_ASSERT, etc. visiblie in non-debug buildsStephan Bergmann2-7/+0
...to avoid lots of loplugin:staticmethods warnings. Also enables DBG_ASSERT etc. also for --enable-debug builds in addition to --enable-dbgutil builds. Change-Id: Ib89ecd9ab8ce7abb2c64790ace248b31f9d2b64d
2015-07-04Fix typosAndrea Gelmini1-3/+3
Change-Id: Id9296115f30858e7fd470a199e59343a96d7deec Reviewed-on: https://gerrit.libreoffice.org/16712 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2015-06-08loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann5-10/+10
Change-Id: I2828f5fe78efffaa5dee19a3d56592d12878d956
2015-05-28new clang plugin: loopvartoosmallNoel Grandin1-2/+2
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-05remove unused #include awt/InvalidateStyle.hppMichael Stahl1-1/+0
Change-Id: I7ea52365157fc642401db64c3b4a40d4643d16ae
2015-04-16UnoControls, toolkit, uuoi, xmlsecurity: convert new to ::Create.Michael Meeks3-2/+4
Change-Id: I7b8c7ece656589c50fb066e9fa1565fd59f930da
2015-04-01Replace remaining getCppuType et al with cppu::UnoTypeStephan Bergmann2-2/+2
Change-Id: I61a85caf1587291eaa6b999050a52c92d9e416e3
2015-03-27loplugin:staticfunctionNoel Grandin1-2/+2
Change-Id: I982ba552579019e4902ae59fddf14a6b34ba5954
2015-03-17Fix various XServiceInfo implementationsStephan Bergmann2-0/+18
...to match what is recorded in the .component files Change-Id: Ie548cd37872d3b8540222201afaac73040e65c8f
2015-03-11V668 no sense in testing the result of new against nullCaolán McNamara2-50/+44
Change-Id: I4a33bd92fc8448638a4bfe1eab7e5041a4c5cc39
2015-03-08TyposJulien Nabet1-1/+1
Change-Id: Ibc378fa5f515de61bb768b4ef082638b40c94e00
2015-02-28TyposJulien Nabet1-2/+2
Change-Id: I151957e415eff793e3d054050526b7d6892d28d4
2015-02-23remove unnecessary parenthesis in return statementsNoel Grandin1-1/+1
found with $ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;' Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
2015-02-09Fix of a few typosAndrea Gelmini1-2/+2
Change-Id: Ib2a311f0341d165a8f9d3f7a11ec36378fd69519 Reviewed-on: https://gerrit.libreoffice.org/14373 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-01-08brute-force find-and-remove of unused #define constants.Noel Grandin2-2/+0
Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
2014-12-16Daft += "\0" triggers assertStephan Bergmann1-4/+0
...try GetProcessServiceManager().createInstance("stardiv.UnoControls.ProgressMonitor").addText("foo", "bar", false) in Basic... Change-Id: I30318c3e8e671a97b6a3fe2dd9ec03add21794ab
2014-12-16Daft "\0" triggers assert in OUString literal ctorStephan Bergmann2-2/+1
...try GetProcessServiceManager().createInstance("stardiv.UnoControls.StatusIndicator") in Basic... Change-Id: Iafc22188feb8a1d3f1b19ac4f6e209be62a44d17
2014-12-16Missing UnoControls/MakefileStephan Bergmann1-0/+24
Change-Id: Iebf1c42c384909f6226c25eb151985f8bc244c93
2014-12-04Kill the libreofficekit static library / shim.c.Andrzej Hunt1-14/+0
It looks like the cleanest method of getting lok_init into a LibreOfficeKitInit.h header (in a c89 compatible way) is to have it as a static function. (inline is only available in C99 or later -- this is actually available on Linux which is the only place that we can actually use lok_init anyways currently, however given we have to keep c89 for the C code (for MSVC) compatibility, selectively enabling c99 would likely be more messy.) Conflicts: libreofficekit/Module_libreofficekit.mk Change-Id: I0493e7a68ed5397479220bb6ba8c3db870b6dd32
2014-11-18cppuhelper: clean up public headers with include-what-you-useMichael Stahl11-0/+13
Change-Id: I41ba46831f24b2960a1fe982b74a2b623e682e0b
2014-10-29remove unnecessary 'using namespace rtl' declarationsNoel Grandin8-8/+0
It turns out that almost none of them were necessary. Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83 Reviewed-on: https://gerrit.libreoffice.org/12133 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-30 10:38:30 +0000