loplugin:unusedfields

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

include/svx/sidebar/ValueSetWithTextControl.hxx

@@ -49,8 +49,6 @@ public:
 private:
     struct ValueSetWithTextItem
     {
-        Image maItemImage;
-        Image maSelectedItemImage;
         OUString maItemText;
         OUString maItemText2;
     };

libreofficekit/source/gtk/lokdocview.cxx

@@ -969,8 +969,7 @@ static gboolean postDocumentLoad(gpointer pData)
     // Total number of columns in this document.
     guint nColumns = ceil((double)nDocumentWidthPixels / nTileSizePixels);
 
-    priv->m_pTileBuffer = std::unique_ptr<TileBuffer>(new TileBuffer(priv->m_pDocument,
+    priv->m_pTileBuffer = std::unique_ptr<TileBuffer>(new TileBuffer(nColumns));
-                                                                     nColumns));
     gtk_widget_set_size_request(GTK_WIDGET(pLOKDocView),
                                 nDocumentWidthPixels,
                                 nDocumentHeightPixels);
@@ -3289,8 +3288,7 @@ lok_doc_view_set_zoom (LOKDocView* pDocView, float fZoom)
     // Total number of columns in this document.
     guint nColumns = ceil((double)nDocumentWidthPixels / nTileSizePixels);
 
-    priv->m_pTileBuffer = std::unique_ptr<TileBuffer>(new TileBuffer(priv->m_pDocument,
+    priv->m_pTileBuffer = std::unique_ptr<TileBuffer>(new TileBuffer(nColumns));
-                                                                     nColumns));
     gtk_widget_set_size_request(GTK_WIDGET(pDocView),
                                 nDocumentWidthPixels,
                                 nDocumentHeightPixels);

libreofficekit/source/gtk/tilebuffer.hxx

@@ -90,10 +90,8 @@ private:
 class TileBuffer
 {
  public:
- TileBuffer(LibreOfficeKitDocument *document = nullptr,
+ TileBuffer(int columns = 0)
-            int columns = 0)
+     : m_nWidth(columns)
-     : m_pLOKDocument(document)
-     , m_nWidth(columns)
     {
         cairo_surface_t *pSurface = cairo_image_surface_create(CAIRO_FORMAT_ARGB32, nTileSizePixels, nTileSizePixels);
         m_DummyTile.setSurface(pSurface);
@@ -135,8 +133,6 @@ class TileBuffer
      */
     void setInvalid(int x, int y, float zoom, GTask* task, GThreadPool*);
 
-    /// Contains the reference to the LOK Document that this tile buffer is for.
-    LibreOfficeKitDocument *m_pLOKDocument;
     /// Stores all the tiles cached by this tile buffer.
     std::map<int, Tile> m_mTiles;
     /// Width of the current tile buffer (number of columns)

oox/source/core/xmlfilterbase.cxx

@@ -174,19 +174,17 @@ struct XmlFilterBaseImpl
 {
     typedef RefMap< OUString, Relations > RelationsMap;
 
-    Reference<XComponentContext>   mxContext;
     FastParser                     maFastParser;
     const OUString                 maBinSuffix;
     RelationsMap                   maRelationsMap;
     TextFieldStack                 maTextFieldStack;
-    const NamespaceMap&             mrNamespaceMap;
+    const NamespaceMap&            mrNamespaceMap;
 
     /// @throws RuntimeException
-    explicit            XmlFilterBaseImpl( const Reference< XComponentContext >& rxContext );
+    explicit            XmlFilterBaseImpl();
 };
 
-XmlFilterBaseImpl::XmlFilterBaseImpl( const Reference< XComponentContext >& rxContext ) :
+XmlFilterBaseImpl::XmlFilterBaseImpl() :
-    mxContext(rxContext),
     maBinSuffix( ".bin" ),
     mrNamespaceMap(StaticNamespaceMap::get())
 {
@@ -196,7 +194,7 @@ XmlFilterBaseImpl::XmlFilterBaseImpl( const Reference< XComponentContext >& rxCo
 
 XmlFilterBase::XmlFilterBase( const Reference< XComponentContext >& rxContext ) :
     FilterBase( rxContext ),
-    mxImpl( new XmlFilterBaseImpl( rxContext ) ),
+    mxImpl( new XmlFilterBaseImpl ),
     mnRelId( 1 ),
     mnMaxDocId( 0 ),
     mbMSO2007(false),

sd/inc/Outliner.hxx

@@ -246,12 +246,6 @@ private:
     */
     bool mbFoundObject;
 
-    /** When set to <TRUE/> this flag indicates that an error has occurred
-        that should terminate the iteration over the objects to search/spell
-        check.
-    */
-    bool mbError;
-
     /** This flag indicates whether to search forward or backwards.
     */
     bool mbDirectionIsForward;

sd/source/ui/view/Outliner.cxx

@@ -155,7 +155,6 @@ SdOutliner::SdOutliner( SdDrawDocument* pDoc, OutlinerMode nMode )
       mnPageCount(0),
       mbEndOfSearch(false),
       mbFoundObject(false),
-      mbError(false),
       mbDirectionIsForward(true),
       mbRestrictSearchToSelection(false),
       maMarkListCopy(),

sw/inc/unocrsr.hxx

@@ -25,17 +25,9 @@
 
 namespace sw
 {
-    enum class UnoCursorHintType
-    {
-        DOC_DISPOSING,
-        LEAVES_SECTION
-    };
-
     struct SW_DLLPUBLIC UnoCursorHint final : public SfxHint
     {
-        UnoCursorHintType m_eType;
+        UnoCursorHint() {}
-        UnoCursorHint(UnoCursorHintType eType)
-                : m_eType(eType) {};
         virtual ~UnoCursorHint() override;
     };
 }

sw/source/core/doc/doccorr.cxx

@@ -159,7 +159,7 @@ void PaMCorrAbs( const SwPaM& rRange,
         if (bChange && bLeaveSection)
         {
             // the UNO cursor has left its section. We need to notify it!
-            sw::UnoCursorHint aHint(sw::UnoCursorHintType::LEAVES_SECTION);
+            sw::UnoCursorHint aHint;
             pUnoCursor->m_aNotifier.Broadcast(aHint);
         }
     }

sw/source/core/doc/docnew.cxx

@@ -414,7 +414,7 @@ SwDoc::~SwDoc()
     getIDocumentRedlineAccess().GetRedlineTable().DeleteAndDestroyAll();
     getIDocumentRedlineAccess().GetExtraRedlineTable().DeleteAndDestroyAll();
 
-    const sw::UnoCursorHint aHint(sw::UnoCursorHintType::DOC_DISPOSING);
+    const sw::UnoCursorHint aHint;
     cleanupUnoCursorTable();
     for(const auto& pWeakCursor : mvUnoCursorTable)
     {

xmlscript/source/xml_helper/xml_impctx.cxx

@@ -320,15 +320,12 @@ class ExtendedAttributes :
     OUString * m_pQNames;
     OUString * m_pValues;
 
-    rtl::Reference<DocumentHandlerImpl> m_xHandler;
-
 public:
     inline ExtendedAttributes(
         sal_Int32 nAttributes,
         sal_Int32 * pUids,
         OUString * pLocalNames, OUString * pQNames,
-        Reference< xml::sax::XAttributeList > const & xAttributeList,
+        Reference< xml::sax::XAttributeList > const & xAttributeList );
-        DocumentHandlerImpl * pHandler );
     virtual ~ExtendedAttributes() throw () override;
 
     // XAttributes
@@ -355,14 +352,12 @@ inline ExtendedAttributes::ExtendedAttributes(
     sal_Int32 nAttributes,
     sal_Int32 * pUids,
     OUString * pLocalNames, OUString * pQNames,
-    Reference< xml::sax::XAttributeList > const & xAttributeList,
+    Reference< xml::sax::XAttributeList > const & xAttributeList )
-    DocumentHandlerImpl * pHandler )
     : m_nAttributes( nAttributes )
     , m_pUids( pUids )
     , m_pLocalNames( pLocalNames )
     , m_pQNames( pQNames )
     , m_pValues( new OUString[ nAttributes ] )
-    , m_xHandler( pHandler )
 {
     for ( sal_Int32 nPos = 0; nPos < nAttributes; ++nPos )
     {
@@ -542,7 +537,7 @@ void DocumentHandlerImpl::startElement(
     xAttributes = static_cast< xml::input::XAttributes * >(
         new ExtendedAttributes(
             nAttribs, pUids, pLocalNames, pQNames,
-            xAttribs, this ) );
+            xAttribs ) );
 
     getElementName( rQElementName, &nUid, &aLocalName );