summaryrefslogtreecommitdiff
path: root/UnoControls
AgeCommit message (Collapse)AuthorFilesLines
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>
2014-09-29loplugin: cstylecastNoel Grandin2-3/+3
Change-Id: I28443b688f8ab752162846e5cea661f26d269cad
2014-09-15UnoControls: sal_False/True -> false/trueStephan Bergmann7-19/+19
Change-Id: If30d43a2693c6df2d483ec135efa54ccb643fdb0
2014-08-01typo: more then -> more thanTakeshi Abe1-1/+1
Change-Id: I2e477d66f25bde7256938ccb1f95ab26add24922
2014-06-17improve the inlinesimplememberfunctions clang pluginNoel Grandin6-60/+10
Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
2014-06-10-Werror,-Wtautological-undefined-compareStephan Bergmann1-6/+0
Change-Id: If546680f4c16ccd733188a65d82129ec2358017b
2014-06-09loplugin: inlinesimplememberfunctionsNoel Grandin2-16/+2
Change-Id: I42119f656ca528286fb25d2d36c0af54b7d04a6b
2014-05-23Prefer cppu::UnoType<T>::get() to ::getCppuType((T*)0) part20Julien Nabet7-52/+52
Change-Id: If87cdfb2c605254f6d69baa4ca5aec09091caa68
2014-04-30Many spelling fixes: some more files.Pedro Giffuni1-1/+1
Attempt to clean up most but certainly not all the spelling mistakes that found home in OpenOffice through decades. (cherry picked from commit e62c0f54ef18a5a79b76e934834b148523c69847) Conflicts: LICENSE NOTICE_category_b UnoControls/source/base/basecontainercontrol.cxx UnoControls/source/base/registercontrols.cxx UnoControls/source/controls/OConnectionPointContainerHelper.cxx UnoControls/source/controls/progressbar.cxx UnoControls/source/controls/progressmonitor.cxx UnoControls/source/controls/statusindicator.cxx UnoControls/source/inc/framecontrol.hxx Change-Id: I882a1d640d931b4e89b2d19f3585fd35fdd320ca
2014-04-27automaticly -> automatically.Jan Holesovsky5-9/+9
Change-Id: Iae55083160eee86ac8301f272634dd3ae65fd847
2014-04-19fixincludeguards.sh: some smaller dirsThomas Arnhold9-27/+27
Change-Id: Ic25bd678dc299627299b22145efd7bebcf2b39d0
2014-04-06Bin empty @seealso, @param, @return and @onerror metacommentsTor Lillqvist9-1685/+0
Change-Id: I6517028670a953954b31599fa3e23f4c8ee8cfc9
2014-04-06Kill superfluous spacing before semicolonsTor Lillqvist18-691/+691
Change-Id: I2c5c2c2e8c57796d147141748fb57a4c5645a96a