loplugin:returnconstant in OPropertyBrowserController

Change-Id: Id5e225dbe5a1c2f27c1032cdaab35a01c78ede81
Reviewed-on: https://gerrit.libreoffice.org/59136
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 8 insertions, 12 deletions

diff --git a/extensions/source/propctrlr/propcontroller.cxx b/extensions/source/propctrlr/propcontroller.cxx
index 0db2e8be590a..8492d115e0d6 100644
--- a/extensions/source/propctrlr/propcontroller.cxx
+++ b/extensions/source/propctrlr/propcontroller.cxx
@@ -361,16 +361,14 @@ namespace pcr
         if (!pParentWin)
             throw RuntimeException("The frame is invalid. Unable to extract the container window.",*this);
 
-        if ( Construct( pParentWin ) )
+        Construct( pParentWin );
+        try
         {
-            try
-            {
-                m_xFrame->setComponent( VCLUnoHelper::GetInterface( m_pView ), this );
-            }
-            catch( const Exception& )
-            {
-                OSL_FAIL( "OPropertyBrowserController::attachFrame: caught an exception!" );
-            }
+            m_xFrame->setComponent( VCLUnoHelper::GetInterface( m_pView ), this );
+        }
+        catch( const Exception& )
+        {
+            OSL_FAIL( "OPropertyBrowserController::attachFrame: caught an exception!" );
         }
 
         startContainerWindowListening();
@@ -669,7 +667,7 @@ namespace pcr
     }
 
 
-    bool OPropertyBrowserController::Construct(vcl::Window* _pParentWin)
+    void OPropertyBrowserController::Construct(vcl::Window* _pParentWin)
     {
         DBG_ASSERT(!haveView(), "OPropertyBrowserController::Construct: already have a view!");
         DBG_ASSERT(_pParentWin, "OPropertyBrowserController::Construct: invalid parent window!");
@@ -690,8 +688,6 @@ namespace pcr
         impl_initializeView_nothrow();
 
         m_pView->Show();
-
-        return true;
     }