compilerplugins/clang/unnecessaryvirtual-dead.results |    2 -
 compilerplugins/clang/unnecessaryvirtual.results      |   24 +++++++++---------
 include/drawinglayer/primitive2d/baseprimitive2d.hxx  |    4 +--
 include/svtools/acceleratorexecute.hxx                |    2 -
 4 files changed, 17 insertions(+), 15 deletions(-)

New commits:
commit e327bccf14130eb469f06c96721b9d8aea5dd506
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Sun Jan 2 13:37:04 2022 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Sun Jan 2 19:36:19 2022 +0100

    loplugin:unnecessaryvirtual
    
    Change-Id: Ia7d68c07d44aaa9ce44fe4e7ed72987e96433269
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127866
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/compilerplugins/clang/unnecessaryvirtual-dead.results 
b/compilerplugins/clang/unnecessaryvirtual-dead.results
index 5d14653d6cdb..b09f822608b3 100644
--- a/compilerplugins/clang/unnecessaryvirtual-dead.results
+++ b/compilerplugins/clang/unnecessaryvirtual-dead.results
@@ -6,7 +6,7 @@ canvas/inc/base/graphicdevicebase.hxx:305
     void canvas::GraphicDeviceBase::removePropertyChangeListener(const class 
rtl::OUString &,const class com::sun::star::uno::Reference<class 
com::sun::star::beans::XPropertyChangeListener> &,)
 canvas/inc/base/graphicdevicebase.hxx:318
     void canvas::GraphicDeviceBase::removeVetoableChangeListener(const class 
rtl::OUString &,const class com::sun::star::uno::Reference<class 
com::sun::star::beans::XVetoableChangeListener> &,)
-include/basegfx/utils/unopolypolygon.hxx:94
+include/basegfx/utils/unopolypolygon.hxx:92
     void basegfx::unotools::UnoPolyPolygon::modifying()const
 sc/source/core/opencl/formulagroupcl.cxx:1070
     void sc::opencl::(anonymous 
namespace)::DynamicKernelSlidingArgument::GenSlidingWindowFunction(class 
std::basic_stringstream<char> &,)
diff --git a/compilerplugins/clang/unnecessaryvirtual.results 
b/compilerplugins/clang/unnecessaryvirtual.results
index 8287b882fbf5..3d939a4764c4 100644
--- a/compilerplugins/clang/unnecessaryvirtual.results
+++ b/compilerplugins/clang/unnecessaryvirtual.results
@@ -248,7 +248,7 @@ forms/source/xforms/namedcollection.hxx:100
     class com::sun::star::uno::Sequence<class rtl::OUString> 
NamedCollection::getElementNames()
 forms/source/xforms/namedcollection.hxx:105
     unsigned char NamedCollection::hasByName(const class rtl::OUString &,)
-include/basegfx/utils/unopolypolygon.hxx:94
+include/basegfx/utils/unopolypolygon.hxx:92
     void basegfx::unotools::UnoPolyPolygon::modifying()const
 include/comphelper/interaction.hxx:53
     void comphelper::OInteraction::select()
@@ -260,6 +260,10 @@ include/comphelper/unique_disposing_ptr.hxx:169
     void 
comphelper::unique_disposing_solar_mutex_reset_ptr::reset(type-parameter-0-0 *,)
 include/comphelper/weakeventlistener.hxx:120
     void comphelper::OWeakListenerAdapter::disposing(const struct 
com::sun::star::lang::EventObject &,)
+include/drawinglayer/primitive2d/baseprimitive2d.hxx:156
+    class drawinglayer::primitive2d::Primitive2DContainer 
drawinglayer::primitive2d::BasePrimitive2D::getDecomposition(const class 
com::sun::star::uno::Sequence<struct com::sun::star::beans::PropertyValue> &,)
+include/drawinglayer/primitive2d/baseprimitive2d.hxx:162
+    struct com::sun::star::geometry::RealRectangle2D 
drawinglayer::primitive2d::BasePrimitive2D::getRange(const class 
com::sun::star::uno::Sequence<struct com::sun::star::beans::PropertyValue> &,)
 include/svl/svdde.hxx:236
     class DdeData * DdeTopic::Get(enum SotClipboardFormatId,)
 include/svl/svdde.hxx:237
@@ -272,8 +276,8 @@ include/svl/svdde.hxx:243
     _Bool DdeTopic::StartAdviseLoop()
 include/svl/svdde.hxx:300
     void DdeService::~DdeService()
-include/svx/svdundo.hxx:753
-    class std::unique_ptr<class SdrUndoAction> 
SdrUndoFactory::CreateUndoSort(class SdrPage &,const class std::vector<int> &,)
+include/svtools/acceleratorexecute.hxx:115
+    void svt::AcceleratorExecute::~AcceleratorExecute()
 include/vbahelper/vbacollectionimpl.hxx:289
     int ScVbaCollectionBase::getCount()
 include/vbahelper/vbacollectionimpl.hxx:294
@@ -300,8 +304,6 @@ include/vcl/customweld.hxx:45
     class rtl::OUString weld::CustomWidgetController::GetHelpText()const
 include/vcl/weld.hxx:104
     void weld::Widget::set_visible(_Bool,)
-jvmfwk/inc/vendorbase.hxx:157
-    _Bool jfw_plugin::VendorBase::initialize(const class std::vector<struct 
std::pair<class rtl::OUString, class rtl::OUString> > &,)
 sc/source/core/opencl/formulagroupcl.cxx:1062
     _Bool sc::opencl::(anonymous 
namespace)::DynamicKernelSlidingArgument::NeedParallelReduction()const
 sc/source/core/opencl/formulagroupcl.cxx:1070
@@ -402,8 +404,6 @@ sw/inc/flypos.hxx:37
     void SwPosFlyFrame::~SwPosFlyFrame()
 toolkit/inc/controls/geometrycontrolmodel.hxx:185
     void OGeometryControlModel::fillProperties(class 
com::sun::star::uno::Sequence<struct com::sun::star::beans::Property> &,class 
com::sun::star::uno::Sequence<struct com::sun::star::beans::Property> &,)const
-ucb/source/ucp/webdav-curl/DAVTypes.hxx:96
-    void http_dav_ucp::DAVOptions::~DAVOptions()
 vcl/inc/jsdialog/jsdialogbuilder.hxx:151
     void JSDialogSender::sendFullUpdate(_Bool,)
 vcl/inc/jsdialog/jsdialogbuilder.hxx:154
@@ -424,6 +424,8 @@ vcl/inc/jsdialog/jsdialogbuilder.hxx:382
     void JSWidget::freeze()
 vcl/inc/jsdialog/jsdialogbuilder.hxx:388
     void JSWidget::thaw()
+vcl/inc/jsdialog/jsdialogbuilder.hxx:493
+    void JSComboBox::set_entry_text_without_notify(const class rtl::OUString 
&,)
 vcl/inc/salframe.hxx:147
     void SalFrame::SetRepresentedURL(const class rtl::OUString &,)
 vcl/inc/salframe.hxx:196
@@ -456,13 +458,13 @@ vcl/inc/unx/salframe.h:208
     void X11SalFrame::updateGraphics(_Bool,)
 vcl/inc/unx/salinst.h:41
     class SalX11Display * X11SalInstance::CreateDisplay()const
-workdir/../vcl/inc/qt5/QtFrame.hxx:158
-    void QtFrame::registerDragSource(class QtDragSource *,)
 workdir/../vcl/inc/qt5/QtFrame.hxx:159
-    void QtFrame::deregisterDragSource(const class QtDragSource *,)
+    void QtFrame::registerDragSource(class QtDragSource *,)
 workdir/../vcl/inc/qt5/QtFrame.hxx:160
-    void QtFrame::registerDropTarget(class QtDropTarget *,)
+    void QtFrame::deregisterDragSource(const class QtDragSource *,)
 workdir/../vcl/inc/qt5/QtFrame.hxx:161
+    void QtFrame::registerDropTarget(class QtDropTarget *,)
+workdir/../vcl/inc/qt5/QtFrame.hxx:162
     void QtFrame::deregisterDropTarget(const class QtDropTarget *,)
 writerfilter/source/dmapper/TableManager.hxx:492
     void writerfilter::dmapper::TableManager::tableExceptionProps(const class 
tools::SvRef<class writerfilter::dmapper::TablePropertyMap> &,)
diff --git a/include/drawinglayer/primitive2d/baseprimitive2d.hxx 
b/include/drawinglayer/primitive2d/baseprimitive2d.hxx
index c264912941aa..aef821557c3d 100644
--- a/include/drawinglayer/primitive2d/baseprimitive2d.hxx
+++ b/include/drawinglayer/primitive2d/baseprimitive2d.hxx
@@ -152,13 +152,13 @@ public:
     /** The getDecomposition implementation for UNO API will use 
getDecomposition from this implementation. It
         will construct a ViewInformation2D from the ViewParameters for that 
purpose
      */
-    virtual Primitive2DContainer
+    Primitive2DContainer
     getDecomposition(const css::uno::Sequence<css::beans::PropertyValue>& 
rViewParameters);
 
     /** The getRange implementation for UNO API will use getRange from this 
implementation. It
         will construct a ViewInformation2D from the ViewParameters for that 
purpose
      */
-    virtual css::geometry::RealRectangle2D
+    css::geometry::RealRectangle2D
     getRange(const css::uno::Sequence<css::beans::PropertyValue>& 
rViewParameters);
 
     // XAccounting
diff --git a/include/svtools/acceleratorexecute.hxx 
b/include/svtools/acceleratorexecute.hxx
index c5c9f4e0b13a..5acf9bcd2560 100644
--- a/include/svtools/acceleratorexecute.hxx
+++ b/include/svtools/acceleratorexecute.hxx
@@ -112,7 +112,7 @@ class SVT_DLLPUBLIC AcceleratorExecute final
 
 
         /** @short  fight against inlining ... */
-        virtual ~AcceleratorExecute();
+        ~AcceleratorExecute();
 
 
         /** @short  init this instance.

Reply via email to