summaryrefslogtreecommitdiff
path: root/toolkit
AgeCommit message (Collapse)AuthorFilesLines
2015-10-27don't allocate uno::Reference on the heapNoel Grandin1-10/+5
There is no point, since it's the size of a pointer anyway (found by temporarily making the new operator in uno::Reference deleted). Change-Id: I62a8b957fef9184f65d705600acfdab4116dcb34 Reviewed-on: https://gerrit.libreoffice.org/19603 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-26remove extra spaces around ::Noel Grandin1-1/+1
just makes it harder to grep for methods Change-Id: I0925db2f9284972fcf7e1e3adf90cbe21f69eb15
2015-10-23ucb: replace boost::ptr_vector with std::vector<std::unique_ptr>Michael Stahl1-2/+0
Change-Id: I05c64aaed4aaea813b369dccc721f9d10167f161
2015-10-21remove untyped Link<>Noel Grandin1-2/+2
Change-Id: I809f9e10309ceadda0a82c3818277323b34ec61b Reviewed-on: https://gerrit.libreoffice.org/19491 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-20loplugin:defaultparamsStephan Bergmann1-1/+1
Change-Id: I1d7e9bae07ff7bb13005d408522c6bb922ae00c6
2015-10-19loplugin:defaultparamsStephan Bergmann1-1/+1
Change-Id: If4234ce8e13781bcdba84d689e512a24fef007d0
2015-10-19convert DBG_ASSERTWARNING to SAL_WARN_IFNoel Grandin1-1/+1
I know the comment in debug.hxx said to use SAL_INFO, but SAL_WARN makes much more sense. Change-Id: I3d81f9db89ef8cebe1e4f2983941d69b4c7e6b7d
2015-10-16Allow injection of keyboard and mouse events through UNOMatthew J. Francis2-0/+119
Change-Id: I3d139c6378f5274be1e7bfd88f72d1576c13243d Reviewed-on: https://gerrit.libreoffice.org/19321 Reviewed-by: Matthew Francis <mjay.francis@gmail.com> Tested-by: Matthew Francis <mjay.francis@gmail.com>
2015-10-16convert PASSED to COMPLETED in 'make check' test messagesNoel Grandin1-1/+1
Change-Id: I5501f4f40024d3a47bb7007896e07c6252151cf6
2015-10-15cppcheck:variableScopeNoel Grandin1-2/+2
Change-Id: I9b671637fbe0f32e695d22b74bfb40a39a9fe884 Reviewed-on: https://gerrit.libreoffice.org/19364 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann20-450/+450
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12loplugin:mergeclassesNoel Grandin1-2/+2
Change-Id: I555e3db911a97e798ce9f10c76d7fe187ea550a7
2015-10-09Related tdf#94557: Combo box entries are case-sensitive (sometimes)Julien Nabet1-2/+2
Revert partly http://cgit.freedesktop.org/libreoffice/core/commit/?id=76f33f10309b0ee384a75a7a854858b068d60495 author Julien Nabet <serval2412@yahoo.fr> 2015-03-16 21:31:23 (GMT) committer Caolán McNamara <caolanm@redhat.com> 2015-03-26 14:14:44 (GMT) commit 76f33f10309b0ee384a75a7a854858b068d60495 (patch) tree e12e3b6f2ca46cb9998b93b5c1bb3406408ad3cb parent 6c4100cf81e1678ff386549c0cdaa2f73a154ddc (diff) tdf#67990: Management of case in combobox See comments of tdf#94557 for more information Change-Id: I0432db0c3f76c20e55d8fecb69734952dc0cf53b Reviewed-on: https://gerrit.libreoffice.org/19260 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2015-10-01remove old standalone Sun bug numbersNoel Grandin1-1/+1
Sun bug numbers without any accompanying text are completely useless. Fixed with git grep -lP '//\s*#\d+#\s*$' | xargs perl -i -ne'/\/\/\s*#\d+#\s*$/d or print' And then hand-checking the result to restore places where it deleted code. And then some more grepping and hand-editing to kill the others. Change-Id: Ia96ce4466db8bb8da363ebf41f0ae7f45f28bf29 Reviewed-on: https://gerrit.libreoffice.org/19023 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-09-29Fix typosAndrea Gelmini3-3/+3
Change-Id: Ifddd2533b767924791338555031b848c59980c52 Reviewed-on: https://gerrit.libreoffice.org/18989 Reviewed-by: Joren De Cuyper <jorendc@libreoffice.org> Tested-by: Joren De Cuyper <jorendc@libreoffice.org>
2015-09-29toolkit: convert alloca() to std::unique_ptrMichael Stahl2-6/+4
Change-Id: Ie50b4a1a189fffdaa6a0b26040c2e7c4cc42cd69
2015-09-21convert Link<> to typedNoel Grandin2-21/+18
Change-Id: If3e2b00092440ebd42ae5b73ae2b0e44c3702683
2015-09-20convert Link<> to typedNoel Grandin3-25/+17
Change-Id: I10b050dc4aae45e646761a82520caa96969bc511 Reviewed-on: https://gerrit.libreoffice.org/18700 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-09-18boost->stdCaolán McNamara4-8/+8
Change-Id: Ie490bf2c6921f393bdeed96b1a8815996b701bf0 Reviewed-on: https://gerrit.libreoffice.org/18670 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-09-16boost->stdCaolán McNamara1-2/+2
Change-Id: Ic18be8b86727dd1179a7e39c70493d088ed00557
2015-09-14convert Link<> to typedNoel Grandin1-2/+1
Change-Id: I1c501671d72edd5b998e80c7fa1e91dbeb507af8
2015-09-12Use better/auto int types, remove a couple of OUStrings copiesMatteo Casalin1-20/+20
Change-Id: I911742bddba1dac2641d7d2ac1dad0ed195474dd
2015-09-11convert Link<> to typedNoel Grandin1-11/+11
Change-Id: I70d7c7ae4fc68d2f36a00d7c1b6eba71f63872d7
2015-09-11convert Link<> to typedNoel Grandin1-69/+63
and remove unused maChildEventListeners Change-Id: I845a9af608c3429cf9ccb0e8041f24f423839513
2015-09-09Get rid of OSL_UNUSEDStephan Bergmann1-3/+1
In many cases it was originally used to silence warnings about variables used only in OSL_ENSURE etc., but that's no longer necessary since OSL_ENSURE is based on SAL_WARN etc. In some other cases it was apparently used as a speculative debugging aid, to have a value assigned to a variable to be easily able to view the value in a debugger. And in some cases it was used with (otherwise ignored) return values of (typically SQL-related) function calls, but it is probably better to eventually find and fix all the problematic ones among such calls via function declarations annotated SAL_WARN_UNUSED_RESULT. Change-Id: Ib925b38da70073b546fdcd00fa8524dc302accbf
2015-09-08toolkit: tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe14-26/+23
with the variadic variants. Change-Id: If62a0c3da7f9732a60316dfd49323f6ab838fb6d Reviewed-on: https://gerrit.libreoffice.org/18396 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-09-04convert Link<> to typedNoel Grandin1-5/+4
Change-Id: I2136c3db2742afcb4722f69297276bea1e0119f4 Reviewed-on: https://gerrit.libreoffice.org/18306 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-08-31loplugin:stringconstant: OUStringBuffer: appendAscii -> appendStephan Bergmann3-7/+7
Change-Id: I03529fd3a769d0ebf1f17e3885cd6ecb691a8678
2015-08-28make PostUserEvent Link<> typedNoel Grandin2-8/+6
Change-Id: I13f10bda985d55d419a5bff481130a456ae2db8a
2015-08-21Remove demonstrated-useless comphelper::compareStephan Bergmann1-2/+1
Change-Id: Ifd1b38afb963255b76f0d821eb46e4c6a972128a
2015-08-17Put Polygon from tools under tools:: namespaceNorbert Thiebaud1-2/+3
Polygon is one of these names that Clash with some system objects A similar work has been done earlier with PolyPolygon. Change-Id: Icf2217cb2906292b7275760f1a16be0e150312f5 Reviewed-on: https://gerrit.libreoffice.org/17789 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
2015-08-11loplugin: defaultparamsNoel Grandin6-9/+9
Change-Id: I50ba6a836473961d952ed88e56532501469c5368
2015-08-06toolkit: don't let ToolkitWorkerFunction join its own threadMichael Stahl1-1/+2
Change-Id: I6694bf83e1e5ba6563d399e454442ce3574e8d26
2015-08-03vcl:: replace boost::function with std::functionMichael Stahl1-1/+0
Change-Id: Ibcc536558f26ed15c59263c25bfeb690950dd3d0
2015-08-03inline some use-once typedefsNoel Grandin1-3/+1
Change-Id: I02cbbba56a2ad83e0ac3d806265a7e0d6a29594d Reviewed-on: https://gerrit.libreoffice.org/17495 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-27inline a bunch of use-once macrosNoel Grandin2-2/+54
no point in having a macro unless it's actually going to reduce the number of lines of code Change-Id: Ic8760d6506cf272d7bd088f7b3b4dcbf288099fc
2015-07-24Resolves: tdf#90609 SolarMutex assert on threaded loadCaolán McNamara1-0/+2
Change-Id: I81a784ab506d0d3ae266d7341b8ff7a163dcd8d2
2015-07-24convert ImageAlign to enum classNoel Grandin2-41/+41
Change-Id: I22e2e66a1f0b230403ad1d9c1f5b7f34a6ee6ea5
2015-07-24convert WALLPAPER constants to scoped enumNoel Grandin1-3/+3
Change-Id: I3b4b4e8a94904e22823a263bd5446a2e1aa47661
2015-07-19Fix typosAndrea Gelmini2-2/+2
Change-Id: I52cbaad71560d73f5e24f3de3cd62b00d678dd6c Reviewed-on: https://gerrit.libreoffice.org/17187 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2015-07-16loplugin:simplifyboolStephan Bergmann4-10/+10
Change-Id: I9875ceec4276382d4bf96bc7e150d594df646602
2015-07-14loplugin:unusedmethods toolkit,packageNoel Grandin1-16/+0
Change-Id: I83618cd2fd12fb0c1691dc7255fc7192e68649cd Reviewed-on: https://gerrit.libreoffice.org/17031 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-13Make content of OSL_ASSERT, DBG_ASSERT, etc. visiblie in non-debug buildsStephan Bergmann1-5/+1
...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 Gelmini2-2/+2
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-29Revert "return and use std::vector from OInterfaceContainerHelper"Noel Grandin2-16/+18
This reverts commit e57314f61f67b093510c5a8a8f34a62126ba8734.
2015-06-29return and use std::vector from OInterfaceContainerHelperNoel Grandin2-18/+16
since most of the time we don’t need a heavyweight uno::Sequence. Adds a new method getElementsAsVector(). Change-Id: I9e72bef0c0c723ffd0dd7d4152db5baec6784a7a Reviewed-on: https://gerrit.libreoffice.org/15747 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-28coverity#1308539 Uncaught exceptionCaolán McNamara2-2/+2
Change-Id: Id9cad4eaccd71c769fea06d71bbfdd03e8c112ae
2015-06-20gb_OBJCXXFLAGS requires -lobjcStephan Bergmann1-0/+3
...as it causes at least recent Clang trunk (towards 3.7) to emit .cfi_personality 155, ___objc_personality_v0 where __objc_personality_v0 is exported from /usr/lib/libobjc.A.dylib Change-Id: If4ccf15b2c3167185ba50d78ecf4379e02814933
2015-06-17Some missing SolarMutexGuard around VclPtr acquire/releaseStephan Bergmann4-2/+14
At least OutputDevice::acquire/release use a plain unguarded int and ++, --, so apparently rely on the SolarMutex being locked whenever they are called. Fixed those places that caused "make check" to fail for me when temporarily adding DBG_TESTSOLARMUTEX() to OutputDevice::acquire/release. (A recurring pattern is that a class fails to ensure the SolarMutex is locked around the destruction of non-null VclPtr members.) Change-Id: I77cba6f3908f2de1b516ce28f1c3c43b3f57a9c5
2015-06-16Fix typosAndrea Gelmini1-1/+1
Change-Id: Ieaf67cf597e05dbae5affa153af907fda4462a41 Reviewed-on: https://gerrit.libreoffice.org/16295 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>