diff options
| author | Łukasz Hryniuk <lukasz.hryniuk@wp.pl> | 2015-01-04 22:50:05 +0100 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2015-01-07 06:47:04 +0000 |
| commit | 9fd89a5952a48b473cee03a2681eb5d4cd6d1742 (patch) | |
| tree | 1fbf98ba46e86989826adf5d559f51c82d59a5e5 /desktop | |
| parent | a527ad98e58a7fc9af482f9088687d82e5c52cca (diff) | |
fdo#39440 reduce scope of local variables
Beside scope changes, it fixes lack of initialization in a few places.
Change-Id: Ia09fdb9845d8ac17256330a5ec5168401c84f0f2
Reviewed-on: https://gerrit.libreoffice.org/13755
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'desktop')
| -rw-r--r-- | desktop/source/app/app.cxx | 3 | ||||
| -rw-r--r-- | desktop/source/deployment/manager/dp_extensionmanager.cxx | 2 |
2 files changed, 2 insertions, 3 deletions
diff --git a/desktop/source/app/app.cxx b/desktop/source/app/app.cxx index e5f0a84240f4..af0cf51f2fcd 100644 --- a/desktop/source/app/app.cxx +++ b/desktop/source/app/app.cxx @@ -2690,7 +2690,6 @@ void Desktop::HandleAppEvent( const ApplicationEvent& rAppEvent ) void Desktop::OpenSplashScreen() { const CommandLineArgs &rCmdLine = GetCommandLineArgs(); - bool bVisible = false; // Show intro only if this is normal start (e.g. no server, no quickstart, no printing ) if ( !rCmdLine.IsInvisible() && !rCmdLine.IsHeadless() && @@ -2726,7 +2725,7 @@ void Desktop::OpenSplashScreen() if ( rCmdLine.HasSplashPipe() ) aSplashService = "com.sun.star.office.PipeSplashScreen"; - bVisible = true; + bool bVisible = true; Sequence< Any > aSeq( 2 ); aSeq[0] <<= bVisible; aSeq[1] <<= aAppName; diff --git a/desktop/source/deployment/manager/dp_extensionmanager.cxx b/desktop/source/deployment/manager/dp_extensionmanager.cxx index 4643794625e3..345a3e0a574b 100644 --- a/desktop/source/deployment/manager/dp_extensionmanager.cxx +++ b/desktop/source/deployment/manager/dp_extensionmanager.cxx @@ -680,7 +680,6 @@ Reference<css::deployment::XPackage> ExtensionManager::addExtension( Reference<css::deployment::XPackage> xExtensionBackup; uno::Any excOccurred2; - bool bUserDisabled = false; bool bCanInstall = doChecksForAddExtension( xPackageManager, properties, @@ -690,6 +689,7 @@ Reference<css::deployment::XPackage> ExtensionManager::addExtension( xOldExtension ); { + bool bUserDisabled = false; // In this garded section (getMutex) we must not use the argument xCmdEnv // because it may bring up dialogs (XInteractionHandler::handle) this //may potententially deadlock. See issue |