summaryrefslogtreecommitdiffstats
path: root/basic
Commit message (Collapse)AuthorAgeFilesLines
* typo: more then -> more thanTakeshi Abe2014-08-011-1/+1
| | | | Change-Id: I2e477d66f25bde7256938ccb1f95ab26add24922
* basic: -Werror=sign-compareMichael Stahl2014-07-291-1/+3
| | | | Change-Id: I7c776faa74a61b0d263c3b446972da9461b3d4f6
* fdo#46037: remove unused comphelper/configurationhelper.hxxAlexandre Vicenzi2014-07-291-1/+0
| | | | | | | Change-Id: I66f9d2912202ba1393d0c65189f8a945bca4fcaa Reviewed-on: https://gerrit.libreoffice.org/10603 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* convert linked list to std::vector in SbxDimArrayNoel Grandin2014-07-291-61/+40
| | | | | | | Change-Id: I2640cfe1f9606d992f622b99ff3c607555771e23 Reviewed-on: https://gerrit.libreoffice.org/10298 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
* fdo#52076 remove empty library instead of writing it out emptyLionel Elie Mamane2014-07-241-1/+5
| | | | | | | Change-Id: Id4fd8c4f34e8ed07044eb428049731c4b9b694ca Reviewed-on: https://gerrit.libreoffice.org/10507 Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu> Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
* Drop unused #includesTakeshi Abe2014-07-242-7/+0
| | | | Change-Id: I77ca957204bacee95e32a7f2a54c9311002aeef9
* Drop an unused functionTakeshi Abe2014-07-182-21/+0
| | | | Change-Id: I59eb46f9d57e9fd72ed86c972685be93d154e3c7
* fdo#75757: remove inheritance to std::vectorTakeshi Abe2014-07-172-33/+21
| | | | | | | | | ... which was introduced at 2110397670695991b3a5cd28a15ba0ffd2a3a611. Change-Id: If0f634b29e1891574267edf8cc07b24d07a9406c Reviewed-on: https://gerrit.libreoffice.org/10363 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* Renamed brdcst.[hc]xx to SfxBroadcaster.[hc]xxTobias Lippert2014-07-174-4/+4
| | | | | | | | | | - Remove includes from files where they are not needed. - Update pch files Change-Id: I0188e3934ef429008c1ef495ab1d5b27f38664d5 Reviewed-on: https://gerrit.libreoffice.org/10342 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
* Avoid possible memory leaks in case of exceptionsTakeshi Abe2014-07-163-15/+15
| | | | Change-Id: Icecc2cce52d7f27c030270639e6b85877e3aa620
* new loplugin: externalandnotdefinedNoel Grandin2014-07-117-27/+51
| | | | | | | | Find "missing headers," where a function is declared directly in the .cxx (as extern) and not defined, and should arguably instead be declared in an include file. Change-Id: I6d83ee432b2ab0cd050aec2b27c3658d32ac02a2
* DBG_ASSERT->assert when followed by dereferenceCaolán McNamara2014-07-061-2/+2
| | | | Change-Id: I9390228721df1b713d8cf2e719854b74d6b639c8
* coverity#706241 Uncaught exceptionCaolán McNamara2014-07-032-9/+31
| | | | | | | we've already checked to see if the element exists, so we know that its not there. Change-Id: I538c0d48a883d478e4c748d222b21958d083c049
* coverity#706240 Uncaught exceptionCaolán McNamara2014-07-011-11/+17
| | | | Change-Id: Ie31f5e1ff74ff8b9f35d7ce08e1f02a780375890
* coverity#735397 dead codeNorbert Thiebaud2014-07-011-5/+1
| | | | Change-Id: If8d82a39af7a506f42b6c13d8408ffc479b564e7
* clang scan-build: various warningsCaolán McNamara2014-07-011-2/+2
| | | | Change-Id: I4bdfb074b3cf6fcb49765322308dfa4b9ed67713
* loplugin:unreffun: also warn about redundant redeclarationsStephan Bergmann2014-06-272-3/+0
| | | | Change-Id: I9a812220b58cf6da00d854e65794f7c673ab239d
* clang: Dead initializationCaolán McNamara2014-06-261-2/+1
| | | | Change-Id: Ieae5303e55f21044ef8e91e63c4896696cfad51c
* clang: Returning null referenceCaolán McNamara2014-06-261-3/+5
| | | | Change-Id: I2cfc14e73696a027283dfa5eef98f784505dc5a3
* clang: Uninitialized argument valueCaolán McNamara2014-06-261-6/+4
| | | | Change-Id: Id65c5815185cf022171e3523c6ad92545830104c
* Remove this weird inheritance from smart-pointer-wrapped class.Kohei Yoshida2014-06-251-114/+122
| | | | | | SbxVarEntry is now its own class. Change-Id: I5c5ce1990fa83930acced1d507f5b0de60bf221e
* Use boost::optional to store alias name.Kohei Yoshida2014-06-252-32/+23
| | | | Change-Id: I809b21ea156061a265c0d83d58534df10bc273bc
* pData -> mpVarEntries.Kohei Yoshida2014-06-251-43/+43
| | | | | | pData is a nightmare with grepping as it's too generic of a name. Change-Id: I01b3f1b503f21ee13c97733fa66804874a2cddc4
* Remove this class that only derives from std::vector and not much else.Kohei Yoshida2014-06-251-24/+12
| | | | Change-Id: Ibc584f4148cec49a9ac34a240cc2fa3e87daf443
* fixes for up-casting to Reference<XInterface>Noel Grandin2014-06-253-7/+8
| | | | | | | | | Fix regressions introduced with 6a043e9c0acff20e1618ca8ec15c21d5d0fd0d37 "Use the new type-checking Reference constructor to reduce code noise" Change-Id: I85662856f21c810a7db497fe3b0e116f075b1687 Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
* basic: silence expected SAL_WARN outputMiklos Vajna2014-06-192-3/+3
| | | | | | | | | These were HACK() and DbgOut() messages previously, none of these ended up on stderr. It seems that they are more or less harmless, so as long as nobody fixes them, silence them, so more useful warnings are not hidden when one does e.g. xray on a drawinglayer shape. Change-Id: I7343d93d2916d9978852418aca725d027eb09128
* coverity#1210096 Uncaught exceptionCaolán McNamara2014-06-171-5/+11
| | | | Change-Id: I1d06c7a6a61556c2b501a16c224ccc6fe52307d0
* coverity#1210098 Uncaught exceptionCaolán McNamara2014-06-171-5/+12
| | | | Change-Id: I531891fa1cce10a331dd651ee9ce52d1f8d0c169
* improve the inlinesimplememberfunctions clang pluginNoel Grandin2014-06-175-21/+1
| | | | Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
* loplugin:staticcallStephan Bergmann2014-06-1310-31/+31
| | | | Change-Id: I6cd46964c523c9393e9d249fdb43aaa38679fa06
* coverity#1213486 Uncaught exceptionCaolán McNamara2014-06-131-10/+18
| | | | Change-Id: Ia4d919d88bf22168aa770ad35fd8360b0b155efe
* iprevent violation of exception specMarkus Mohrhard2014-06-101-1/+6
| | | | Change-Id: I1f638ae3a2e88e0cadaab0046ef087fe77e16b06
* loplugin: inlinesimplememberfunctionsNoel Grandin2014-06-097-25/+2
| | | | Change-Id: I42119f656ca528286fb25d2d36c0af54b7d04a6b
* Do not both _use_library_objects and _use_libraries sbStephan Bergmann2014-06-041-1/+0
| | | | Change-Id: I11f667104a0ca7279542fab53ad754d4de8c807d
* Avoid static BasicDLL (that would be destroyed after DeInitVCL now)Stephan Bergmann2014-06-041-7/+2
| | | | Change-Id: Id8968a7746815ecce79b6eb1ce971b74e770efa4
* DeInitVCL at end of testsStephan Bergmann2014-06-044-0/+4
| | | | | | | | | | | | | | | | | | | | | This required some changes to the framework: * Init-/DeInitVCL is no longer done per individual test in BootstrapFixture, but once per CppunitTest invocation in a new vclbootstrapprotector (similarly to the exisiting unobootstrapprotector). CppunitTests that need VCL now need to declare gb_CppunitTest_use_vcl. * For things to work properly, the UNO component context needs to be disposed from within DeInitVCL (cf. Desktop's Application::DeInit called from DeInitVCL). The easiest solution was to introduce an Application::setDeInitHook (where the hook is called from DeInitVCL) specifically for vclbootstrapprotector to call. * PythonTests don't (yet) call DeInitVCL; they still hook into BootstrapFixture's original test_init functionality (to call InitVCL), and do not make use of the vclbootstrapprotector. Change-Id: I4f3a3c75db30b58c1cd49d81c51db14902ed68b2
* WaE: warning C4101: 'ex' : unreferenced local variableThomas Arnhold2014-06-041-1/+1
| | | | Change-Id: Ib8b71b50fe52a7cc4e261626549ac3e067851797
* basic: typoThomas Arnhold2014-06-021-1/+1
| | | | Change-Id: I10c9e480a94a966dac4f6c17fc69fa8892c56955
* Fix memory leak for BASIC sub (as well as void function)Stephan Bergmann2014-06-021-2/+8
| | | | | | | | | | | | | d88593af59d9126cdbcd6c0a5b06fb9c673dc6f9 "INTEGRATION: CWS ab34: #i73457# Prevent sub from beeing set as param 0 for return type void" to fix i#73457 "Memory Leak in all Basic type void Method calls" had made the PutDirect call dependent on != SbxVOID only (which would cover void functions) but not also on != SbxEMPTY (which would apparently be required to also cover subs, as seen with CppunitTest_basic_vba run under lsan). Either this was an oversight with the original fix, or subs have meanwhile changed from GetType() == SbxVOID to SbxEMPTY? Change-Id: I3e5dbf79bfd5eea3cfec8ed3fa984d13167aa501
* Avoid possible memory leaks in case of exceptionsTakeshi Abe2014-05-314-28/+16
| | | | Change-Id: Iad3de981a1c9660b1322315640e531c9891db0bf
* Detach all DocBasicItem objects upon process termination.Kohei Yoshida2014-05-291-1/+15
| | | | | | | To prevent rogue DocBasicItem objects (which are global objects) from attempting to stop listening after the main app is dead. Change-Id: I68a667137ca4e0eff18278f4d2d5dd67466eaf34
* Avoid static local of a singleton. This crashed writer unit tests.Kohei Yoshida2014-05-292-8/+22
| | | | Change-Id: I7c5f2d372676cdb317a8dad636bf1aab7dd5db37
* Avoid possible memory leaks in case of exceptionsTakeshi Abe2014-05-298-45/+42
| | | | Change-Id: Iac63a5d60478e5cd8e2b77c889c7b312d3d15f67
* coverity#706235 Uncaught exceptionCaolán McNamara2014-05-281-2/+3
| | | | Change-Id: I06321ca2b686207a1d9bd698461aedf84daf31e1
* coverity#706234 Uncaught exceptionCaolán McNamara2014-05-281-2/+3
| | | | Change-Id: I7542008369ed3433cd8bfce7702f036148195516
* coverity#706232 Uncaught exceptionCaolán McNamara2014-05-281-2/+4
| | | | Change-Id: I07e13af4f4578d5d6823b6d10aa9c37d04ecc395
* coverity#706231 Uncaught exceptionCaolán McNamara2014-05-281-1/+1
| | | | Change-Id: I1f1acb91a97ff7c9fb50ad36eca7b91d361a1276
* Avoid undefined signed integer overflowStephan Bergmann2014-05-281-4/+5
| | | | Change-Id: I3f32ea88dbb34a05baccba49c15b6691d923753e
* Fix memory leakStephan Bergmann2014-05-282-32/+19
| | | | | | ...by using css::uno::Type instead of a naked typelib_TypeDescription. Change-Id: I387692265e9e032cb5ed6519739ebb3307db6f28
* Fix memory leak (missing typelib_TypeDescription_release)Stephan Bergmann2014-05-281-12/+1
| | | | | | | | | | | ...by radically simplifying TypeToIdlClass(). It is unclear to me why this was so complicated. The only mildly plausible reaons would be that the old code was careful to return null instead of throwing an exception for an unknown type, but the TypeToIdlClass-call-sites either strongly expect a non-null return value (by dereferencing it without any further checking) or use OSL_ASSERT or similar to verify it. So lets hope this is good. Change-Id: I3e24eb6117e84c9d33f4c5f0e2fa88da4c4f2c30