diff options
author | Caolán McNamara <caolanm@redhat.com> | 2016-05-12 11:27:11 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2016-05-12 11:32:30 +0100 |
commit | 9226f602bade8a153fe2d5877cc47856cdcffd0c (patch) | |
tree | b734278331aeea7d41dcf091d118a81225a88894 | |
parent | 11ef5ee940b32641ae2697d0426e8a0108f2a858 (diff) |
rejig for legibility, no logic change
Change-Id: I73418e29dd85488613ea5c54ca8d505715ed3cbf
-rw-r--r-- | vcl/source/app/svapp.cxx | 44 |
1 files changed, 22 insertions, 22 deletions
diff --git a/vcl/source/app/svapp.cxx b/vcl/source/app/svapp.cxx index 5a2ae0c8a586..76947cf57bc1 100644 --- a/vcl/source/app/svapp.cxx +++ b/vcl/source/app/svapp.cxx @@ -1441,51 +1441,51 @@ vcl::Window* Application::GetDefDialogParent() // as DefDialogParent // current focus frame - vcl::Window *pWin = nullptr; - if( (pWin = pSVData->maWinData.mpFocusWin) != nullptr ) + vcl::Window *pWin = pSVData->maWinData.mpFocusWin; + if (pWin) { - while( pWin->mpWindowImpl && pWin->mpWindowImpl->mpParent ) + while (pWin->mpWindowImpl && pWin->mpWindowImpl->mpParent) pWin = pWin->mpWindowImpl->mpParent; // check for corrupted window hierarchy, #122232#, may be we now crash somewhere else - if( !pWin->mpWindowImpl ) + if (!pWin->mpWindowImpl) { OSL_FAIL( "Window hierarchy corrupted!" ); pSVData->maWinData.mpFocusWin = nullptr; // avoid further access return nullptr; } - if( (pWin->mpWindowImpl->mnStyle & WB_INTROWIN) == 0 ) + if ((pWin->mpWindowImpl->mnStyle & WB_INTROWIN) == 0) { return pWin->mpWindowImpl->mpFrameWindow->ImplGetWindow(); } } // last active application frame - if( nullptr != (pWin = pSVData->maWinData.mpActiveApplicationFrame) ) + pWin = pSVData->maWinData.mpActiveApplicationFrame; + if (pWin) { return pWin->mpWindowImpl->mpFrameWindow->ImplGetWindow(); } - else + + // first visible top window (may be totally wrong....) + pWin = pSVData->maWinData.mpFirstFrame; + while (pWin) { - // first visible top window (may be totally wrong....) - pWin = pSVData->maWinData.mpFirstFrame; - while( pWin ) + if( pWin->ImplGetWindow()->IsTopWindow() && + pWin->mpWindowImpl->mbReallyVisible && + (pWin->mpWindowImpl->mnStyle & WB_INTROWIN) == 0 + ) { - if( pWin->ImplGetWindow()->IsTopWindow() && - pWin->mpWindowImpl->mbReallyVisible && - (pWin->mpWindowImpl->mnStyle & WB_INTROWIN) == 0 - ) - { - while( pWin->mpWindowImpl->mpParent ) - pWin = pWin->mpWindowImpl->mpParent; - return pWin->mpWindowImpl->mpFrameWindow->ImplGetWindow(); - } - pWin = pWin->mpWindowImpl->mpFrameData->mpNextFrame; + while( pWin->mpWindowImpl->mpParent ) + pWin = pWin->mpWindowImpl->mpParent; + return pWin->mpWindowImpl->mpFrameWindow->ImplGetWindow(); } - // use the desktop - return nullptr; + pWin = pWin->mpWindowImpl->mpFrameData->mpNextFrame; } + + // use the desktop + return nullptr; } Application::DialogCancelMode Application::GetDialogCancelMode() |