mark UNO structs as SAL_WARN_UNUSED, where possible

Change-Id: Ie3de518f60c9f1313c68df54dbdc1fb2804f1f0d

27 files changed, 71 insertions, 56 deletions

diff --git a/chart2/source/view/main/ChartView.cxx b/chart2/source/view/main/ChartView.cxx
index 9814f3cbc71d..30c19c03626b 100644
--- a/chart2/source/view/main/ChartView.cxx
+++ b/chart2/source/view/main/ChartView.cxx
@@ -1735,7 +1735,6 @@ awt::Rectangle ChartView::impl_createDiagramAndContent( const CreateShapeParam2D
         if(2==nDimensionCount)
             pSeriesPlotter->setTransformationSceneToScreen( pVCooSys->getTransformationSceneToScreen() );
         //better performance for big data
-        awt::Size aCoordinateRegionResolution(1000,1000);
         {
             //calculate resolution for coordinate system
             Sequence<sal_Int32> aCoordinateSystemResolution = pVCooSys->getCoordinateSystemResolution( rPageSize, m_aPageResolution );
diff --git a/codemaker/source/cppumaker/cpputype.cxx b/codemaker/source/cppumaker/cpputype.cxx
index a27058d81152..e4f6fa8b5a8e 100644
--- a/codemaker/source/cppumaker/cpputype.cxx
+++ b/codemaker/source/cppumaker/cpputype.cxx
@@ -201,6 +201,9 @@ protected:
 
     bool passByReference(OUString const & name) const;
 
+    bool canBeWarnUnused(OUString const & name) const;
+    bool canBeWarnUnused(OUString const & name, int depth) const;
+
     OUString resolveOuterTypedefs(OUString const & name) const;
 
     OUString resolveAllTypedefs(OUString const & name) const;
@@ -906,6 +909,61 @@ bool CppuType::passByReference(OUString const & name) const {
     }
 }
 
+bool CppuType::canBeWarnUnused(OUString const & name) const {
+    return canBeWarnUnused(name, 0);
+}
+bool CppuType::canBeWarnUnused(OUString const & name, int depth) const {
+    // prevent infinite recursion and blowing the stack
+    if (depth > 10)
+        return false;
+    OUString aResolvedName = resolveOuterTypedefs(name);
+    switch (m_typeMgr->getSort(aResolvedName)) {
+    case codemaker::UnoType::SORT_BOOLEAN:
+    case codemaker::UnoType::SORT_BYTE:
+    case codemaker::UnoType::SORT_SHORT:
+    case codemaker::UnoType::SORT_UNSIGNED_SHORT:
+    case codemaker::UnoType::SORT_LONG:
+    case codemaker::UnoType::SORT_UNSIGNED_LONG:
+    case codemaker::UnoType::SORT_HYPER:
+    case codemaker::UnoType::SORT_UNSIGNED_HYPER:
+    case codemaker::UnoType::SORT_FLOAT:
+    case codemaker::UnoType::SORT_DOUBLE:
+    case codemaker::UnoType::SORT_CHAR:
+    case codemaker::UnoType::SORT_ENUM_TYPE:
+    case codemaker::UnoType::SORT_STRING:
+    case codemaker::UnoType::SORT_TYPE:
+        return true;
+    case codemaker::UnoType::SORT_PLAIN_STRUCT_TYPE:
+    {
+        rtl::Reference< unoidl::Entity > ent;
+        m_typeMgr->getSort(aResolvedName, &ent);
+        rtl::Reference< unoidl::PlainStructTypeEntity > ent2(
+            dynamic_cast< unoidl::PlainStructTypeEntity * >(ent.get()));
+        if (!ent2->getDirectBase().isEmpty() && !canBeWarnUnused(ent2->getDirectBase(), depth+1))
+            return false;
+        for ( const unoidl::PlainStructTypeEntity::Member& rMember : ent2->getDirectMembers())
+        {
+            if (!canBeWarnUnused(rMember.type, depth+1))
+                return false;
+        }
+        return true;
+    }
+    case codemaker::UnoType::SORT_SEQUENCE_TYPE:
+    {
+        OUString aInnerType = aResolvedName.copy(2);
+        return canBeWarnUnused(aInnerType, depth+1);
+    }
+    case codemaker::UnoType::SORT_ANY:
+    case codemaker::UnoType::SORT_INSTANTIATED_POLYMORPHIC_STRUCT_TYPE:
+    case codemaker::UnoType::SORT_INTERFACE_TYPE:
+        return false;
+    default:
+        throw CannotDumpException(
+            "unexpected entity \"" + name
+            + "\" in call to CppuType::canBeWarnUnused");
+    }
+}
+
 OUString CppuType::resolveOuterTypedefs(OUString const & name) const {
     for (OUString n(name);;) {
         rtl::Reference< unoidl::Entity > ent;
@@ -1811,8 +1869,11 @@ private:
 };
 
 void PlainStructType::dumpDeclaration(FileStream & out) {
-    out << "\n#ifdef SAL_W32\n#   pragma pack(push, 8)\n#endif\n\n" << indent()
-        << "struct SAL_DLLPUBLIC_RTTI " << id_;
+    out << "\n#ifdef SAL_W32\n#   pragma pack(push, 8)\n#endif\n\n" << indent();
+    out << "struct SAL_DLLPUBLIC_RTTI ";
+    if (canBeWarnUnused(name_))
+        out << "SAL_WARN_UNUSED ";
+    out << id_;
     OUString base(entity_->getDirectBase());
     if (!base.isEmpty()) {
         out << ": public " << codemaker::cpp::scopedCppName(u2b(base));
diff --git a/comphelper/source/misc/configuration.cxx b/comphelper/source/misc/configuration.cxx
index 8b93ff97f1fb..69b4aea5f1a9 100644
--- a/comphelper/source/misc/configuration.cxx
+++ b/comphelper/source/misc/configuration.cxx
@@ -128,11 +128,8 @@ comphelper::detail::ConfigurationWrapper::~ConfigurationWrapper() {}
 bool comphelper::detail::ConfigurationWrapper::isReadOnly(OUString const & path)
     const
 {
-css::beans::Property SB(access_->getPropertyByHierarchicalName(path));
-    return
-        (access_->getPropertyByHierarchicalName(path).Attributes
-         & css::beans::PropertyAttribute::READONLY)
-        != 0;
+    css::beans::Property SB(access_->getPropertyByHierarchicalName(path));
+    return (SB.Attributes & css::beans::PropertyAttribute::READONLY) != 0;
 }
 
 css::uno::Any comphelper::detail::ConfigurationWrapper::getPropertyValue(
diff --git a/cui/source/dialogs/pastedlg.cxx b/cui/source/dialogs/pastedlg.cxx
index ed8c705e2029..eccd83cc0d46 100644
--- a/cui/source/dialogs/pastedlg.cxx
+++ b/cui/source/dialogs/pastedlg.cxx
@@ -117,7 +117,6 @@ SotClipboardFormatId SvPasteObjectDialog::GetFormat( const TransferableDataHelpe
                                  aEnd( ((DataFlavorExVector&)*pFormats).end() );
     while( aIter != aEnd )
     {
-        css::datatransfer::DataFlavor aFlavor( *aIter );
         SotClipboardFormatId nFormat = (*aIter++).mnSotId;
 
         ::std::map< SotClipboardFormatId, OUString >::iterator itName =
diff --git a/embeddedobj/source/commonembedding/specialobject.cxx b/embeddedobj/source/commonembedding/specialobject.cxx
index fe665bf7237f..b9ec0abf9d78 100644
--- a/embeddedobj/source/commonembedding/specialobject.cxx
+++ b/embeddedobj/source/commonembedding/specialobject.cxx
@@ -138,7 +138,6 @@ awt::Size SAL_CALL OSpecialEmbeddedObject::getVisualAreaSize( sal_Int64 nAspect
         throw embed::WrongStateException( "The own object has no model!",
                                     static_cast< ::cppu::OWeakObject* >(this) );
 
-    awt::Size aResult;
     return maSize;
 }
 
diff --git a/extensions/source/propctrlr/cellbindinghelper.cxx b/extensions/source/propctrlr/cellbindinghelper.cxx
index 860af6fb571b..e1d8f1a70d9c 100644
--- a/extensions/source/propctrlr/cellbindinghelper.cxx
+++ b/extensions/source/propctrlr/cellbindinghelper.cxx
@@ -306,7 +306,6 @@ namespace pcr
             OSL_ENSURE( xBindingProps.is() || !_rxBinding.is(), "CellBindingHelper::getAddressFromCellBinding: no property set for the binding!" );
             if ( xBindingProps.is() )
             {
-                CellAddress aAddress;
                 bReturn = (bool)( xBindingProps->getPropertyValue( PROPERTY_BOUND_CELL ) >>= _rAddress );
             }
         }
diff --git a/extensions/source/propctrlr/formcomponenthandler.cxx b/extensions/source/propctrlr/formcomponenthandler.cxx
index c853bcdfeba0..f0602f9fc02b 100644
--- a/extensions/source/propctrlr/formcomponenthandler.cxx
+++ b/extensions/source/propctrlr/formcomponenthandler.cxx
@@ -631,7 +631,7 @@ namespace pcr
         sal_Int32 nPropId = m_pInfoService->getPropertyId( _rPropertyName );
         DBG_ASSERT( nPropId != -1, "FormComponentPropertyHandler::convertToPropertyValue: not one of my properties!!" );
 
-        Property aProperty( impl_getPropertyFromId_throw( nPropId ) );
+        impl_getPropertyFromId_throw( nPropId );
 
         Any aControlValue( _rPropertyValue );
         if ( !aControlValue.hasValue() )
diff --git a/framework/source/layoutmanager/toolbarlayoutmanager.cxx b/framework/source/layoutmanager/toolbarlayoutmanager.cxx
index 54dae84be9e0..31b1912c53d1 100644
--- a/framework/source/layoutmanager/toolbarlayoutmanager.cxx
+++ b/framework/source/layoutmanager/toolbarlayoutmanager.cxx
@@ -1753,7 +1753,6 @@ awt::Point ToolbarLayoutManager::implts_findNextCascadeFloatingPos()
 
     awt::Point aStartPos( nCascadeIndentX, nCascadeIndentY );
     awt::Point aCurrPos( aStartPos );
-    awt::Rectangle aRect;
 
     if ( xContainerWindow.is() )
     {
@@ -3274,8 +3273,6 @@ throw (uno::RuntimeException, std::exception)
 
     if ( aUIElement.m_xUIElement.is() && xWindow.is() )
     {
-        awt::Rectangle aRect;
-
         bWinFound = true;
         uno::Reference< awt::XDockableWindow > xDockWindow( xWindow, uno::UNO_QUERY );
         if ( xDockWindow->isFloating() )
@@ -3590,7 +3587,6 @@ throw (uno::RuntimeException, std::exception)
     {
         if ( !bDockingInProgress )
         {
-            awt::Rectangle aRect;
             uno::Reference< awt::XDockableWindow > xDockWindow( xWindow, uno::UNO_QUERY );
             if ( xDockWindow->isFloating() )
             {
diff --git a/reportdesign/source/ui/misc/UITools.cxx b/reportdesign/source/ui/misc/UITools.cxx
index 083d42a8cc50..fbef47b96b5c 100644
--- a/reportdesign/source/ui/misc/UITools.cxx
+++ b/reportdesign/source/ui/misc/UITools.cxx
@@ -295,7 +295,6 @@ namespace
         {
             if ( SfxItemState::SET == _rItemSet.GetItemState(aIt->nWID) && xInfo->hasPropertyByName(aIt->sName) )
             {
-                const beans::Property aProp = xInfo->getPropertyByName( aIt->sName );
                 if ( ( aIt->nFlags & beans::PropertyAttribute::READONLY ) != beans::PropertyAttribute::READONLY )
                 {
                     const SfxPoolItem* pItem = _rItemSet.GetItem(aIt->nWID);
diff --git a/sc/source/filter/excel/xichart.cxx b/sc/source/filter/excel/xichart.cxx
index 75af048ad80d..be226cb5cf19 100644
--- a/sc/source/filter/excel/xichart.cxx
+++ b/sc/source/filter/excel/xichart.cxx
@@ -1196,10 +1196,6 @@ void XclImpChText::ConvertTitlePosition( const XclChTextKey& rTitleKey ) const
         sal_Int32 nScRot = XclTools::GetScRotation( GetRotation(), 0 );
         double fRad = nScRot * F_PI18000;
         double fSin = fabs( sin( fRad ) );
-        double fCos = fabs( cos( fRad ) );
-        css::awt::Size aBoundSize(
-            static_cast< sal_Int32 >( fCos * aTitleSize.Width + fSin * aTitleSize.Height + 0.5 ),
-            static_cast< sal_Int32 >( fSin * aTitleSize.Width + fCos * aTitleSize.Height + 0.5 ) );
         // calculate the title position from the values in the CHTEXT record
         css::awt::Point aTitlePos(
             CalcHmmFromChartX( maData.maRect.mnX ),
diff --git a/sc/source/filter/xml/xmlexprt.cxx b/sc/source/filter/xml/xmlexprt.cxx
index 25eb6032f75b..c000b1dd0056 100644
--- a/sc/source/filter/xml/xmlexprt.cxx
+++ b/sc/source/filter/xml/xmlexprt.cxx
@@ -1309,7 +1309,6 @@ void ScXMLExport::WriteRowContent()
     }
     if (!bIsFirst)
     {
-        table::CellAddress aCellAddress;
         if (nIndex != -1)
             AddAttribute(sAttrStyleName, *pCellStyles->GetStyleNameByIndex(nIndex, bIsAutoStyle));
         if (nPrevValidationIndex > -1)
diff --git a/scripting/source/stringresource/stringresource.cxx b/scripting/source/stringresource/stringresource.cxx
index 07b7aaf102df..c539ce7f0a73 100644
--- a/scripting/source/stringresource/stringresource.cxx
+++ b/scripting/source/stringresource/stringresource.cxx
@@ -1750,9 +1750,6 @@ void StringResourcePersistenceImpl::implScanLocaleNames( const Sequence< OUStrin
         }
         else if( !bDefaultFound && aExtension == "default" )
         {
-            //OUString aName = aInetObj.getBase();
-            Locale aLocale;
-
             if( checkNamingSceme( aPureName, m_aNameBase, aDefaultLocale ) )
                 bDefaultFound = true;
         }
diff --git a/sd/source/ui/accessibility/AccessiblePageShape.cxx b/sd/source/ui/accessibility/AccessiblePageShape.cxx
index 5e490e068d69..bc838104c208 100644
--- a/sd/source/ui/accessibility/AccessiblePageShape.cxx
+++ b/sd/source/ui/accessibility/AccessiblePageShape.cxx
@@ -95,8 +95,6 @@ awt::Rectangle SAL_CALL AccessiblePageShape::getBounds()
         if (xSet.is())
         {
             uno::Any aValue;
-            awt::Point aPosition;
-            awt::Size aSize;
 
             aValue = xSet->getPropertyValue ("BorderLeft");
             aValue >>= aBoundingBox.X;
diff --git a/sd/source/ui/func/fuinsert.cxx b/sd/source/ui/func/fuinsert.cxx
index 86634c8cf0b2..2b510f140d83 100644
--- a/sd/source/ui/func/fuinsert.cxx
+++ b/sd/source/ui/func/fuinsert.cxx
@@ -201,8 +201,6 @@ void FuInsertClipboard::DoExecute( SfxRequest&  )
     std::unique_ptr<SfxAbstractPasteDialog> pDlg(pFact->CreatePasteDialog( mpViewShell->GetActiveWindow() ));
     if ( pDlg )
     {
-        css::datatransfer::DataFlavor  aFlavor;
-
         pDlg->Insert( SotClipboardFormatId::EMBED_SOURCE, OUString() );
         pDlg->Insert( SotClipboardFormatId::LINK_SOURCE, OUString() );
         pDlg->Insert( SotClipboardFormatId::DRAWING, OUString() );
diff --git a/sd/source/ui/view/sdview.cxx b/sd/source/ui/view/sdview.cxx
index 71c6ce268503..8accd2af7d53 100644
--- a/sd/source/ui/view/sdview.cxx
+++ b/sd/source/ui/view/sdview.cxx
@@ -936,7 +936,6 @@ void View::DoConnect(SdrOle2Obj* pObj)
                 {
                     // TODO/LEAN: working with visual area can switch object to running state
                     Size aDrawSize = aRect.GetSize();
-                    awt::Size aSz;
 
                     MapMode aMapMode( mrDoc.GetScaleUnit() );
                     Size aObjAreaSize = pObj->GetOrigObjSize( &aMapMode );
diff --git a/sdext/source/presenter/PresenterAccessibility.cxx b/sdext/source/presenter/PresenterAccessibility.cxx
index 999047328f5c..3ecbbf4425a1 100644
--- a/sdext/source/presenter/PresenterAccessibility.cxx
+++ b/sdext/source/presenter/PresenterAccessibility.cxx
@@ -1019,8 +1019,6 @@ awt::Rectangle SAL_CALL PresenterAccessible::AccessibleObject::getBounds()
 {
     ThrowIfDisposed();
 
-    awt::Rectangle aBox;
-
     const awt::Point aLocation (GetRelativeLocation());
     const awt::Size aSize (GetSize());
 
diff --git a/sdext/source/presenter/PresenterSlideSorter.cxx b/sdext/source/presenter/PresenterSlideSorter.cxx
index 58e5ebef7814..3ce9138dc472 100644
--- a/sdext/source/presenter/PresenterSlideSorter.cxx
+++ b/sdext/source/presenter/PresenterSlideSorter.cxx
@@ -1001,7 +1001,6 @@ void PresenterSlideSorter::PaintPreview (
     // Paint a border around the preview.
     if (mxPreviewFrame.is())
     {
-        const geometry::RealRectangle2D aBox (0, 0, aSize.Width, aSize.Height);
         const util::Color aFrameColor (0x00000000);
         PresenterCanvasHelper::SetDeviceColor(aRenderState, aFrameColor);
         rxCanvas->drawPolyPolygon(mxPreviewFrame, aViewState, aRenderState);
diff --git a/sfx2/source/dialog/filtergrouping.cxx b/sfx2/source/dialog/filtergrouping.cxx
index 61af42268231..69e9dababfa9 100644
--- a/sfx2/source/dialog/filtergrouping.cxx
+++ b/sfx2/source/dialog/filtergrouping.cxx
@@ -546,9 +546,6 @@ namespace sfx2
     {
         void operator() ( const MapGroupEntry2GroupEntry::value_type& _rMapEntry )
         {
-#ifdef DBG_UTIL
-            FilterDescriptor aHaveALook = *_rMapEntry.first;
-#endif
             *_rMapEntry.second = *_rMapEntry.first;
         }
     };
diff --git a/sot/source/base/formats.cxx b/sot/source/base/formats.cxx
index a22d3c70258f..50d347d0b0e0 100644
--- a/sot/source/base/formats.cxx
+++ b/sot/source/base/formats.cxx
@@ -1390,7 +1390,6 @@ static sal_uInt16 GetTransferableAction_Impl(
     {
         if( rDataFlavorExVector.size() )
         {
-            DataFlavor              aFlavor;
             const SotAction_Impl*   pArrayStart = pArray;
             SotClipboardFormatId    nId = pArray->nFormatId;
 
diff --git a/svtools/source/control/vclxaccessibleheaderbaritem.cxx b/svtools/source/control/vclxaccessibleheaderbaritem.cxx
index 4979614f2f8e..77097b7f1d53 100644
--- a/svtools/source/control/vclxaccessibleheaderbaritem.cxx
+++ b/svtools/source/control/vclxaccessibleheaderbaritem.cxx
@@ -83,8 +83,6 @@ awt::Rectangle VCLXAccessibleHeaderBarItem::implGetBounds() throw (RuntimeExcept
     awt::Rectangle aBounds;
     OExternalLockGuard aGuard( this );
 
-    css::awt::Size aSize;
-
     if ( m_pHeadBar )
         aBounds = AWTRectangle( m_pHeadBar->GetItemRect( sal_uInt16( m_nIndexInParent ) ) );
 
diff --git a/svtools/source/misc/transfer.cxx b/svtools/source/misc/transfer.cxx
index f9f84be461ac..e8a900d8b4fc 100644
--- a/svtools/source/misc/transfer.cxx
+++ b/svtools/source/misc/transfer.cxx
@@ -603,7 +603,6 @@ void TransferableHelper::AddFormat( const DataFlavor& rFlavor )
     if( bAdd )
     {
         DataFlavorEx   aFlavorEx;
-        DataFlavor     aObjDescFlavor;
 
         aFlavorEx.MimeType = rFlavor.MimeType;
         aFlavorEx.HumanPresentableName = rFlavor.HumanPresentableName;
diff --git a/sw/source/core/frmedt/fefly1.cxx b/sw/source/core/frmedt/fefly1.cxx
index e41ae588bd58..d06b788c13df 100644
--- a/sw/source/core/frmedt/fefly1.cxx
+++ b/sw/source/core/frmedt/fefly1.cxx
@@ -1644,7 +1644,6 @@ ObjCntType SwFEShell::GetObjCntType( const SdrObject& rObj ) const
             uno::Reference< beans::XPropertySetInfo >  xInfo = xSet->getPropertySetInfo();
             if(xInfo->hasPropertyByName( sName ))
             {
-                beans::Property xProperty = xInfo->getPropertyByName( sName );
                 aVal = xSet->getPropertyValue( sName );
                 if( aVal.getValue() && form::FormButtonType_URL == *static_cast<form::FormButtonType const *>(aVal.getValue()) )
                     eType = OBJCNT_URLBUTTON;
diff --git a/test/source/sheet/xsheetoutline.cxx b/test/source/sheet/xsheetoutline.cxx
index 018be8982b37..406f0cb30087 100644
--- a/test/source/sheet/xsheetoutline.cxx
+++ b/test/source/sheet/xsheetoutline.cxx
@@ -120,8 +120,6 @@ void XSheetOutline::testShowLevel()
     uno::Reference< sheet::XSpreadsheet > aSheet(init(), UNO_QUERY_THROW);
     uno::Reference< sheet::XSheetOutline > aSheetOutline(aSheet, UNO_QUERY_THROW);
 
-    table::CellRangeAddress aLevelRangeAddress;
-
     // test columns
 
     table::CellRangeAddress aLevel1 = getAddressFromRangeString(aSheet, colLevel1);
diff --git a/ucb/source/cacher/cachedcontentresultset.cxx b/ucb/source/cacher/cachedcontentresultset.cxx
index b2f04092bb37..488fecdcb4a3 100644
--- a/ucb/source/cacher/cachedcontentresultset.cxx
+++ b/ucb/source/cacher/cachedcontentresultset.cxx
@@ -1127,30 +1127,26 @@ Any SAL_CALL CachedContentResultSet
         throw UnknownPropertyException();
     }
 
-    Property aProp = m_pMyPropSetInfo->getPropertyByName( rPropertyName );
+    m_pMyPropSetInfo->getPropertyByName( rPropertyName );
         //throws UnknownPropertyException, if so
 
     Any aValue;
-    if( rPropertyName == CCRS_PropertySetInfo
-                        ::m_aPropertyNameForCount )
+    if( rPropertyName == CCRS_PropertySetInfo::m_aPropertyNameForCount )
     {
         osl::Guard< osl::Mutex > aGuard( m_aMutex );
         aValue <<= m_nKnownCount;
     }
-    else if( rPropertyName == CCRS_PropertySetInfo
-                            ::m_aPropertyNameForFinalCount )
+    else if( rPropertyName == CCRS_PropertySetInfo::m_aPropertyNameForFinalCount )
     {
         osl::Guard< osl::Mutex > aGuard( m_aMutex );
         aValue <<= m_bFinalCount;
     }
-    else if( rPropertyName == CCRS_PropertySetInfo
-                            ::m_aPropertyNameForFetchSize )
+    else if( rPropertyName == CCRS_PropertySetInfo::m_aPropertyNameForFetchSize )
     {
         osl::Guard< osl::Mutex > aGuard( m_aMutex );
         aValue <<= m_nFetchSize;
     }
-    else if( rPropertyName == CCRS_PropertySetInfo
-                            ::m_aPropertyNameForFetchDirection )
+    else if( rPropertyName == CCRS_PropertySetInfo::m_aPropertyNameForFetchDirection )
     {
         osl::Guard< osl::Mutex > aGuard( m_aMutex );
         aValue <<= m_nFetchDirection;
diff --git a/vcl/source/filter/graphicfilter.cxx b/vcl/source/filter/graphicfilter.cxx
index c86d070072a8..fd9ffdad447e 100644
--- a/vcl/source/filter/graphicfilter.cxx
+++ b/vcl/source/filter/graphicfilter.cxx
@@ -889,7 +889,6 @@ static Graphic ImpGetScaledGraphic( const Graphic& rGraphic, FilterConfigItem& r
             if( ( nMode == 1 ) || ( nMode == 2 ) )
             {
                 GDIMetaFile aMtf( rGraphic.GetGDIMetaFile() );
-                css::awt::Size aDefaultSize( 10000, 10000 );
                 Size aNewSize( OutputDevice::LogicToLogic( Size( nLogicalWidth, nLogicalHeight ), MAP_100TH_MM, aMtf.GetPrefMapMode() ) );
 
                 if( aNewSize.Width() && aNewSize.Height() )
diff --git a/xmloff/source/chart/SchXMLExport.cxx b/xmloff/source/chart/SchXMLExport.cxx
index ac70a8e968b9..0c7ca7d4c636 100644
--- a/xmloff/source/chart/SchXMLExport.cxx
+++ b/xmloff/source/chart/SchXMLExport.cxx
@@ -1833,7 +1833,6 @@ void SchXMLExportHelper_Impl::exportPlotArea(
     std::vector< XMLPropertyState > aPropertyStates;
 
     bool bIs3DChart = false;
-    drawing::HomogenMatrix aTransMatrix;
 
     msStringBuffer.setLength( 0 );
 
diff --git a/xmloff/source/draw/shapeexport.cxx b/xmloff/source/draw/shapeexport.cxx
index 150b7b11c3a1..231b8bcdef75 100644
--- a/xmloff/source/draw/shapeexport.cxx
+++ b/xmloff/source/draw/shapeexport.cxx
@@ -2158,7 +2158,6 @@ void XMLShapeExport::ImpExportPolygonShape(
         ImpExportNewTrans_FeaturesAndWrite(aTRScale, fTRShear, fTRRotate, aTRTranslate, nFeatures);
 
         // create and export ViewBox
-        awt::Point aPoint(0, 0);
         awt::Size aSize(FRound(aTRScale.getX()), FRound(aTRScale.getY()));
         SdXMLImExViewBox aViewBox(0, 0, aSize.Width, aSize.Height);
         mrExport.AddAttribute(XML_NAMESPACE_SVG, XML_VIEWBOX, aViewBox.GetExportString());
@@ -2620,7 +2619,6 @@ void XMLShapeExport::ImpExportConnectorShape(
             fTRRotate, aTRTranslate, pRefPoint);
 
     // fdo#49678: create and export ViewBox
-    awt::Point aPoint(0, 0);
     awt::Size aSize(FRound(aTRScale.getX()), FRound(aTRScale.getY()));
     SdXMLImExViewBox aViewBox(0, 0, aSize.Width, aSize.Height);
     mrExport.AddAttribute(XML_NAMESPACE_SVG, XML_VIEWBOX, aViewBox.GetExportString());