New Clang 21 trunk -Werror,-Wunnecessary-virtual-specifier

...warning about non-overriding virtual dtors in final classes.  Every single
one of these might or might not be suspicious (e.g., in some cases it might be
better to have a virtual dtor in a derived-from class).  Also, for some of the
virtual dtors, making them non-virtual then caused

> error: unnecessary user-declared destructor [loplugin:unnecessaryoverride]

so those were removed completely.

Change-Id: If0ee71645a347d6096b65988c56f627c42fe54b2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/185223
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>

1 files changed, 2 insertions, 3 deletions

diff --git a/sc/inc/funcdesc.hxx b/sc/inc/funcdesc.hxx
index 21026a401828..8fec801d3b9e 100644
--- a/sc/inc/funcdesc.hxx
+++ b/sc/inc/funcdesc.hxx
@@ -42,7 +42,7 @@ class ScFuncDesc final : public formula::IFunctionDescription
 {
 public:
     ScFuncDesc();
-    virtual ~ScFuncDesc();
+    ~ScFuncDesc();
 
     /**
       Clears the object
@@ -273,7 +273,6 @@ class ScFunctionCategory final : public formula::IFunctionCategory
 public:
     ScFunctionCategory(const ::std::vector<const ScFuncDesc*>& _rCategory,sal_uInt32 _nCategory)
             : m_rCategory(_rCategory),m_nCategory(_nCategory){}
-    virtual ~ScFunctionCategory(){}
 
     /**
       @return count of functions in this category
@@ -315,7 +314,7 @@ public:
       The function lists of the categories are sorted by (case insensitive) function name
     */
             ScFunctionMgr();
-    virtual ~ScFunctionMgr();
+    ~ScFunctionMgr();
 
     /**
       Returns name of category.