summaryrefslogtreecommitdiff
path: root/basic
AgeCommit message (Collapse)Author
2014-07-18Drop an unused functionTakeshi Abe
Change-Id: I59eb46f9d57e9fd72ed86c972685be93d154e3c7
2014-07-17fdo#75757: remove inheritance to std::vectorTakeshi Abe
... 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>
2014-07-17Renamed brdcst.[hc]xx to SfxBroadcaster.[hc]xxTobias Lippert
- 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>
2014-07-16Avoid possible memory leaks in case of exceptionsTakeshi Abe
Change-Id: Icecc2cce52d7f27c030270639e6b85877e3aa620
2014-07-11new loplugin: externalandnotdefinedNoel Grandin
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
2014-07-06DBG_ASSERT->assert when followed by dereferenceCaolán McNamara
Change-Id: I9390228721df1b713d8cf2e719854b74d6b639c8
2014-07-03coverity#706241 Uncaught exceptionCaolán McNamara
we've already checked to see if the element exists, so we know that its not there. Change-Id: I538c0d48a883d478e4c748d222b21958d083c049
2014-07-01coverity#706240 Uncaught exceptionCaolán McNamara
Change-Id: Ie31f5e1ff74ff8b9f35d7ce08e1f02a780375890
2014-07-01coverity#735397 dead codeNorbert Thiebaud
Change-Id: If8d82a39af7a506f42b6c13d8408ffc479b564e7
2014-07-01clang scan-build: various warningsCaolán McNamara
Change-Id: I4bdfb074b3cf6fcb49765322308dfa4b9ed67713
2014-06-27loplugin:unreffun: also warn about redundant redeclarationsStephan Bergmann
Change-Id: I9a812220b58cf6da00d854e65794f7c673ab239d
2014-06-26clang: Dead initializationCaolán McNamara
Change-Id: Ieae5303e55f21044ef8e91e63c4896696cfad51c
2014-06-26clang: Returning null referenceCaolán McNamara
Change-Id: I2cfc14e73696a027283dfa5eef98f784505dc5a3
2014-06-26clang: Uninitialized argument valueCaolán McNamara
Change-Id: Id65c5815185cf022171e3523c6ad92545830104c
2014-06-25Remove this weird inheritance from smart-pointer-wrapped class.Kohei Yoshida
SbxVarEntry is now its own class. Change-Id: I5c5ce1990fa83930acced1d507f5b0de60bf221e
2014-06-25Use boost::optional to store alias name.Kohei Yoshida
Change-Id: I809b21ea156061a265c0d83d58534df10bc273bc
2014-06-25pData -> mpVarEntries.Kohei Yoshida
pData is a nightmare with grepping as it's too generic of a name. Change-Id: I01b3f1b503f21ee13c97733fa66804874a2cddc4
2014-06-25Remove this class that only derives from std::vector and not much else.Kohei Yoshida
Change-Id: Ibc584f4148cec49a9ac34a240cc2fa3e87daf443
2014-06-25fixes for up-casting to Reference<XInterface>Noel Grandin
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>
2014-06-19basic: silence expected SAL_WARN outputMiklos Vajna
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
2014-06-17coverity#1210096 Uncaught exceptionCaolán McNamara
Change-Id: I1d06c7a6a61556c2b501a16c224ccc6fe52307d0
2014-06-17coverity#1210098 Uncaught exceptionCaolán McNamara
Change-Id: I531891fa1cce10a331dd651ee9ce52d1f8d0c169
2014-06-17improve the inlinesimplememberfunctions clang pluginNoel Grandin
Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
2014-06-13loplugin:staticcallStephan Bergmann
Change-Id: I6cd46964c523c9393e9d249fdb43aaa38679fa06
2014-06-13coverity#1213486 Uncaught exceptionCaolán McNamara
Change-Id: Ia4d919d88bf22168aa770ad35fd8360b0b155efe
2014-06-10iprevent violation of exception specMarkus Mohrhard
Change-Id: I1f638ae3a2e88e0cadaab0046ef087fe77e16b06
2014-06-09loplugin: inlinesimplememberfunctionsNoel Grandin
Change-Id: I42119f656ca528286fb25d2d36c0af54b7d04a6b
2014-06-04Do not both _use_library_objects and _use_libraries sbStephan Bergmann
Change-Id: I11f667104a0ca7279542fab53ad754d4de8c807d
2014-06-04Avoid static BasicDLL (that would be destroyed after DeInitVCL now)Stephan Bergmann
Change-Id: Id8968a7746815ecce79b6eb1ce971b74e770efa4
2014-06-04DeInitVCL at end of testsStephan Bergmann
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
2014-06-04WaE: warning C4101: 'ex' : unreferenced local variableThomas Arnhold
Change-Id: Ib8b71b50fe52a7cc4e261626549ac3e067851797
2014-06-02basic: typoThomas Arnhold
Change-Id: I10c9e480a94a966dac4f6c17fc69fa8892c56955
2014-06-02Fix memory leak for BASIC sub (as well as void function)Stephan Bergmann
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
2014-05-31Avoid possible memory leaks in case of exceptionsTakeshi Abe
Change-Id: Iad3de981a1c9660b1322315640e531c9891db0bf
2014-05-29Detach all DocBasicItem objects upon process termination.Kohei Yoshida
To prevent rogue DocBasicItem objects (which are global objects) from attempting to stop listening after the main app is dead. Change-Id: I68a667137ca4e0eff18278f4d2d5dd67466eaf34
2014-05-29Avoid static local of a singleton. This crashed writer unit tests.Kohei Yoshida
Change-Id: I7c5f2d372676cdb317a8dad636bf1aab7dd5db37
2014-05-29Avoid possible memory leaks in case of exceptionsTakeshi Abe
Change-Id: Iac63a5d60478e5cd8e2b77c889c7b312d3d15f67
2014-05-28coverity#706235 Uncaught exceptionCaolán McNamara
Change-Id: I06321ca2b686207a1d9bd698461aedf84daf31e1
2014-05-28coverity#706234 Uncaught exceptionCaolán McNamara
Change-Id: I7542008369ed3433cd8bfce7702f036148195516
2014-05-28coverity#706232 Uncaught exceptionCaolán McNamara
Change-Id: I07e13af4f4578d5d6823b6d10aa9c37d04ecc395
2014-05-28coverity#706231 Uncaught exceptionCaolán McNamara
Change-Id: I1f1acb91a97ff7c9fb50ad36eca7b91d361a1276
2014-05-28Avoid undefined signed integer overflowStephan Bergmann
Change-Id: I3f32ea88dbb34a05baccba49c15b6691d923753e
2014-05-28Fix memory leakStephan Bergmann
...by using css::uno::Type instead of a naked typelib_TypeDescription. Change-Id: I387692265e9e032cb5ed6519739ebb3307db6f28
2014-05-28Fix memory leak (missing typelib_TypeDescription_release)Stephan Bergmann
...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
2014-05-27OSL_TRACE -> SAL_INFOThomas Arnhold
Change-Id: Ib9e1ed6119b5286871ac10136f7fcbefffe7aebe
2014-05-27remove more unnecessary use of OUString constructorNoel Grandin
Change-Id: Iae14cb3df65295b6894fd9e05411c5698e9c8aba
2014-05-23remove boilerplate in UNO Exception constructor callsNoel Grandin
Now that we have default values for Exception constructor params, remove lots of boilerplate code. Change-Id: I620bd641eecfed38e6123873b3b94aaf47922e74
2014-05-23add default value for Context param in uno::Exception constructorsNoel Grandin
and all it's subtypes, which is almost never used, so this allows us to simplify lots of call sites. Change-Id: I0b05793ea2bdd1027679f63252d42ce4af89433b
2014-05-22Prefer cppu::UnoType<T>::get() to ::getCppuType((T*)0) part19Julien Nabet
Change-Id: Iab50c52a132c90389992ef68c2d31df95a193ab9
2014-05-22fdo#68983: basic: if the library is not loaded fully, copy source storageMichael Stahl
Also fixes fdo#42899 and fdo#67685 in a different way; the previous fix for fdo#42899 caused the problem with password-protected libraries for which the password is not known: only the binary representation of the BAISC module was stored, not the source code; by simply copying from the source storage the problem can be avoided. It would be possible to ask for the password when storing, but that would not work when non-interactive (called via API). An alternative fix would be to pass in the SfxObjectShell::IsSetModifyEnabled() flag and actually reset the BASIC library's modify flag correctly, but that requires adding a parameter to XStorageBasedLibraryContainer::storeLibrariesToStorage(). (regression from af34774d260a68fc02cd78ba90dd8d4afaf1a2a4 ) Change-Id: I4701401f35171139fc2fe8d225d13d4e533091a0