diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2015-08-14 11:45:09 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2015-08-14 13:18:56 +0200 |
commit | 9d094b9f0a05d2ea62181201efb8f9e46ecfc8f1 (patch) | |
tree | 36a3352b449ca4c5a622e46cce2dc107dc3d9e21 | |
parent | 431f2ce81893db21f23572926e8a5b045c67e9ae (diff) |
tdf#92446: Revert apparently bogus attempt at fixing a memory leak
6b4c596b01039324cfe78f38c4e3ffb9080bcd34 "Fix memory leak for BASIC sub (as well
as void function)," from just looking at the few lines of code in isolation, and
their revision history, looked like a plausible and effective approach to fix
the leaks reported by CppunitTest_basci_vba when run under lsan.
However, tdf#92446 indicates that that innocent-looking change had rather dire
consequences. So revert it and leave a fix for the memory leak to somebody who
actually understands all this code.
Change-Id: Ic81b1c14a1cfb07c6ec4415cc26b2f826adbc8ca
-rw-r--r-- | basic/source/classes/sbxmod.cxx | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/basic/source/classes/sbxmod.cxx b/basic/source/classes/sbxmod.cxx index 79498cd9aef7..698d55e4ab92 100644 --- a/basic/source/classes/sbxmod.cxx +++ b/basic/source/classes/sbxmod.cxx @@ -2140,13 +2140,8 @@ void SbMethod::Broadcast( sal_uInt32 nHintId ) if( mpPar.Is() ) { // Enregister this as element 0, but don't reset the parent! - switch( GetType() ) { - case SbxEMPTY: - case SbxVOID: - break; - default: + if( GetType() != SbxVOID ) { mpPar->PutDirect( pThisCopy, 0 ); - break; } SetParameters( NULL ); } |