diff options
author | Noel Power <noel.power@suse.com> | 2013-09-12 21:37:42 +0100 |
---|---|---|
committer | Lionel Elie Mamane <lionel@mamane.lu> | 2013-09-15 17:01:23 +0200 |
commit | 2c0cc6fac2437c864a90270eadfe005dfc707726 (patch) | |
tree | dd34cbc8d8abcea2087cd2bf385074a5fc404f2b /basctl | |
parent | 952c5fe20d5975c8a2c9f5093d05472ca0fc78fd (diff) |
fix for fdo#69173 crasher
the layout changes for the basic IDE ( for the object browser and object
catalog ) seem flacky, I have seen since those changes have been introduced
some strange ( but random ) behaviour ( like the odd unrepeatable core ( maybe
related to this ) and also sometimes Modules appearing in the tree under the
wrong nodes etc. I'm no expert in the basic IDE code but this patch seems to
fix the problem.
However there is one drawback, in the core inducing scenario the tree view
( object catalog ) dissappears, this is because the patch suppresses the
problematic layout in this case ( as the layout seem not to be currently able
to deal with 'no-existent' (recently) deleted current window )
Probably in this scenario a fallback currentwin (instead of nil) could be set
this would behave better but ideally. Ultimately the layout class should
probably be modified ( possibly redesigned )
Change-Id: I9d1e23bd6fc4aae32aa78da8278c318f7051136a
Diffstat (limited to 'basctl')
-rw-r--r-- | basctl/source/basicide/basides1.cxx | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/basctl/source/basicide/basides1.cxx b/basctl/source/basicide/basides1.cxx index 474e792eb5c5..67798897be90 100644 --- a/basctl/source/basicide/basides1.cxx +++ b/basctl/source/basicide/basides1.cxx @@ -1010,7 +1010,10 @@ void Shell::SetCurWindow( BaseWindow* pNewWin, bool bUpdateTabBar, bool bRemembe } } else + { + SetWindow(pLayout); pLayout = 0; + } if ( bUpdateTabBar ) { sal_uLong nKey = GetWindowId( pCurWin ); |