remove some unused local vars

found by a more aggressive variant of loplugin:unusedvariables.

This is my first pass, committing the simplest and most obviously
unnecessary vars

Change-Id: I9676a6e39a101937097788548764506c93811c57
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87414
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

4 files changed, 1 insertions, 9 deletions

diff --git a/sw/source/core/access/accmap.cxx b/sw/source/core/access/accmap.cxx
index 47b66565c93a..814fd99c91ef 100644
--- a/sw/source/core/access/accmap.cxx
+++ b/sw/source/core/access/accmap.cxx
@@ -1188,7 +1188,6 @@ void SwAccessibleMap::InvalidateShapeInParaSelection()
         //Checked for shapes.
         SwAccessibleShapeMap_Impl::const_iterator aIter = mpShapeMap->cbegin();
         SwAccessibleShapeMap_Impl::const_iterator aEndIter = mpShapeMap->cend();
-        ::rtl::Reference< SwAccessibleContext > xParentAccImpl;
 
         if( bIsSelAll)
         {
@@ -1205,8 +1204,6 @@ void SwAccessibleMap::InvalidateShapeInParaSelection()
         {
             while( aIter != aEndIter )
             {
-                SwAccessibleChild aFrame( (*aIter).first );
-
                 const SwFrameFormat *pFrameFormat = (*aIter).first ? ::FindFrameFormat( (*aIter).first ) : nullptr;
                 if( !pFrameFormat )
                 {
@@ -2209,7 +2206,7 @@ void SwAccessibleMap::RemoveContext( const SdrObject *pObj )
         SwAccessibleShapeMap_Impl::iterator aIter = mpShapeMap->find( pObj );
         if( aIter != mpShapeMap->end() )
         {
-            uno::Reference < XAccessible > xAcc( (*aIter).second );
+            uno::Reference < XAccessible > xTempHold( (*aIter).second );
             mpShapeMap->erase( aIter );
             RemoveGroupContext(pObj);
             // The shape selection flag is not cleared, but one might do
diff --git a/sw/source/core/undo/unins.cxx b/sw/source/core/undo/unins.cxx
index 1bbd14bbe9d5..46e0f55a4725 100644
--- a/sw/source/core/undo/unins.cxx
+++ b/sw/source/core/undo/unins.cxx
@@ -421,7 +421,6 @@ void SwUndoInsert::RepeatImpl(::sw::RepeatContext & rContext)
     case SwNodeType::Ole:
         {
             // StarView does not yet provide an option to copy a StarOBJ
-            tools::SvRef<SotStorage> aRef = new SotStorage( OUString() );
             SwOLEObj& rSwOLE = static_cast<SwOLENode*>(pCNd)->GetOLEObj();
 
             // temporary storage until object is inserted
@@ -796,7 +795,6 @@ void SwUndoReRead::SetAndSave(::sw::UndoRedoContext & rContext)
     // cache the old values
     std::unique_ptr<Graphic> pOldGrf( mpGraphic ? new Graphic(*mpGraphic) : nullptr);
     o3tl::optional<OUString> aOldNm = maNm;
-    o3tl::optional<OUString> aOldFltr = maFltr;
     MirrorGraph nOldMirr = mnMirror;
     // since all of them are cleared/modified by SaveGraphicData:
     SaveGraphicData( *pGrfNd );
diff --git a/sw/source/filter/ww8/WW8TableInfo.cxx b/sw/source/filter/ww8/WW8TableInfo.cxx
index 81fa51e5d5e4..4d97fd7f6bab 100644
--- a/sw/source/filter/ww8/WW8TableInfo.cxx
+++ b/sw/source/filter/ww8/WW8TableInfo.cxx
@@ -658,8 +658,6 @@ WW8TableInfo::processTableLine(const SwTable * pTable,
 
     const SwTableBoxes & rBoxes = pTableLine->GetTabBoxes();
 
-    WW8TableNodeInfo::Pointer_t pTextNodeInfo;
-
     for (size_t n = 0; n < rBoxes.size(); ++n)
     {
         const SwTableBox * pBox = rBoxes[n];
diff --git a/sw/source/filter/xml/wrtxml.cxx b/sw/source/filter/xml/wrtxml.cxx
index d0d5b1e48094..9a9aad263c1f 100644
--- a/sw/source/filter/xml/wrtxml.cxx
+++ b/sw/source/filter/xml/wrtxml.cxx
@@ -82,7 +82,6 @@ ErrCode SwXMLWriter::Write_( const uno::Reference < task::XStatusIndicator >& xS
             comphelper::getProcessComponentContext();
 
     // Get data sink ...
-    tools::SvRef<SotStorageStream> xDocStream;
     uno::Reference<document::XGraphicStorageHandler> xGraphicStorageHandler;
     rtl::Reference<SvXMLGraphicHelper> xGraphicHelper ;
     uno::Reference< document::XEmbeddedObjectResolver > xObjectResolver;