diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2012-03-14 13:27:56 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2012-03-14 13:32:02 +0100 |
commit | 6e67c03dc0225fc66343546b14e902b9d238b1a3 (patch) | |
tree | 6e078783d65e280a721b4e46f0ae0ca6b950f121 /canvas | |
parent | fdo#47246: dtrans: fix multiply linked ImplHelper (diff) | |
download | core-6e67c03dc0225fc66343546b14e902b9d238b1a3.tar.gz core-6e67c03dc0225fc66343546b14e902b9d238b1a3.zip |
Enable -Wnon-virtual-dtor for GCC 4.6
...which has the necessary features to support it.
Change a lot of classes to either contain a protected non-virtual dtor
(which is backwards compatible, so even works for cppumaker-generated
UNO headers) or a public virtual one.
cppuhelper/propertysetmixin.hxx still needs to disable the warning, as
the relevant class has a non-virtual dtor but friends, which would still
cause GCC to warn.
Includes a patch for libcmis, intended to be upstreamed.
Diffstat (limited to 'canvas')
-rw-r--r-- | canvas/inc/canvas/base/sprite.hxx | 3 | ||||
-rw-r--r-- | canvas/inc/canvas/base/spritesurface.hxx | 3 | ||||
-rw-r--r-- | canvas/source/cairo/cairo_sprite.hxx | 3 | ||||
-rw-r--r-- | canvas/source/null/sprite.hxx | 3 | ||||
-rw-r--r-- | canvas/source/vcl/sprite.hxx | 3 |
5 files changed, 15 insertions, 0 deletions
diff --git a/canvas/inc/canvas/base/sprite.hxx b/canvas/inc/canvas/base/sprite.hxx index 38a2270b8e5c..161551b64ed2 100644 --- a/canvas/inc/canvas/base/sprite.hxx +++ b/canvas/inc/canvas/base/sprite.hxx @@ -99,6 +99,9 @@ namespace canvas /** Query sprite priority */ virtual double getPriority() const = 0; + + protected: + ~Sprite() {} }; /** Functor providing a StrictWeakOrdering for sprite references diff --git a/canvas/inc/canvas/base/spritesurface.hxx b/canvas/inc/canvas/base/spritesurface.hxx index c60099f1563b..6ff1c3c8c954 100644 --- a/canvas/inc/canvas/base/spritesurface.hxx +++ b/canvas/inc/canvas/base/spritesurface.hxx @@ -68,6 +68,9 @@ namespace canvas virtual void updateSprite( const Sprite::Reference& rSprite, const ::basegfx::B2DPoint& rPos, const ::basegfx::B2DRange& rUpdateArea ) = 0; + + protected: + ~SpriteSurface() {} }; } diff --git a/canvas/source/cairo/cairo_sprite.hxx b/canvas/source/cairo/cairo_sprite.hxx index b5ecb85b6b97..ca96f3786ac4 100644 --- a/canvas/source/cairo/cairo_sprite.hxx +++ b/canvas/source/cairo/cairo_sprite.hxx @@ -66,6 +66,9 @@ namespace cairocanvas virtual void redraw( const ::cairo::CairoSharedPtr& pCairo, const ::basegfx::B2DPoint& rOrigOutputPos, bool bBufferedUpdate ) const = 0; + + protected: + ~Sprite() {} }; } diff --git a/canvas/source/null/sprite.hxx b/canvas/source/null/sprite.hxx index 56a38be96914..dc25d23ce6e4 100644 --- a/canvas/source/null/sprite.hxx +++ b/canvas/source/null/sprite.hxx @@ -42,6 +42,9 @@ namespace nullcanvas /** Redraw sprite */ virtual void redraw() const = 0; + + protected: + ~Sprite() {} }; } diff --git a/canvas/source/vcl/sprite.hxx b/canvas/source/vcl/sprite.hxx index b2713eb4486d..b86137d4f383 100644 --- a/canvas/source/vcl/sprite.hxx +++ b/canvas/source/vcl/sprite.hxx @@ -66,6 +66,9 @@ namespace vclcanvas virtual void redraw( OutputDevice& rOutDev, const ::basegfx::B2DPoint& rPos, bool bBufferedUpdate ) const = 0; + + protected: + ~Sprite() {} }; } |