loplugin:returnconstant in ScXMLChartExportWrapper

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

sc/inc/xmlwrap.hxx

@@ -96,7 +96,7 @@ class ScXMLChartExportWrapper
 {
 public:
     ScXMLChartExportWrapper( css::uno::Reference< css::frame::XModel > const & xModel, SfxMedium& rMed );
-    bool Export();
+    void Export();
 
 private:
     css::uno::Reference< css::frame::XModel > mxModel;

sc/source/filter/chart/chart_imp.cxx

@@ -22,12 +22,11 @@ ScXMLChartExportWrapper::ScXMLChartExportWrapper( css::uno::Reference< css::fram
 
 }
 
-bool ScXMLChartExportWrapper::Export()
+void ScXMLChartExportWrapper::Export()
 {
     if ( !mxStorage.is() )
         mxStorage = mrMedium.GetOutputStorage();
 
     uno::Reference< document::XStorageBasedDocument>(mxModel, uno::UNO_QUERY_THROW)->storeToStorage(mxStorage, uno::Sequence< beans::PropertyValue >() );
-    return true;
 }
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */

sc/source/ui/docshell/docsh.cxx

@@ -579,8 +579,6 @@ bool ScDocShell::SaveXML( SfxMedium* pSaveMedium, const css::uno::Reference< css
 
 bool ScDocShell::SaveCurrentChart( SfxMedium& rMedium )
 {
-    bool bRet = false;
-
     try
     {
 
@@ -591,13 +589,14 @@ bool ScDocShell::SaveCurrentChart( SfxMedium& rMedium )
         uno::Reference< frame::XModel > xChartDoc ( xCurrentComponent, uno::UNO_QUERY_THROW );
 
         ScXMLChartExportWrapper aExport( xChartDoc, rMedium );
-        bRet = aExport.Export();
+        aExport.Export();
+        return true;
     }
     catch(...)
     {
         SAL_WARN("sc", "exception thrown while saving chart. Bug!!!");
+        return false;
     }
-    return bRet;
 }
 
 bool ScDocShell::Load( SfxMedium& rMedium )