diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-08-13 10:19:56 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-08-14 14:22:45 +0200 |
commit | 013c278d564d4e50c81bec7915c33f4edd75c00e (patch) | |
tree | 4cd39d07ba8b4fdb247ebb001a5e190ff7708a19 /sc/source/ui/docshell/docsh.cxx | |
parent | c5155f59e9fb7f5841de8a6a6fae2219400ac508 (diff) |
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>
Diffstat (limited to 'sc/source/ui/docshell/docsh.cxx')
-rw-r--r-- | sc/source/ui/docshell/docsh.cxx | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/sc/source/ui/docshell/docsh.cxx b/sc/source/ui/docshell/docsh.cxx index fba755a6e823..076d52c20afa 100644 --- a/sc/source/ui/docshell/docsh.cxx +++ b/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 ) |