diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2022-12-15 11:44:32 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2022-12-18 18:31:37 +0000 |
| commit | 4844c096a8ab6a9a620c410a0949d4499f12a504 (patch) | |
| tree | 38967ed6ef03f90ae6cef48c313e77b7ba93b139 /canvas/source/cairo/cairo_canvasbitmap.cxx | |
| parent | 9a0b523e0a84d403b9092176ccec4b3e3efe42d0 (diff) | |
loplugin:unocast (cairocanvas::SurfaceProvider)
(See the upcoming commit introducing that loplugin:unocast on why such
dynamic_casts from UNO types are dangerous.)
There are implementation classes whose getSomething already delegates to
RepaintTarget, so they can't also delegate to SurfaceProvider. So introduce the
concept of comphelper::getSomethingImpl additionally delegating to a sequence of
mixin classes before delegating to the base.
Change-Id: I9230f3dc06abbdd1ad92514a11473dae2624f7c1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144404
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'canvas/source/cairo/cairo_canvasbitmap.cxx')
| -rw-r--r-- | canvas/source/cairo/cairo_canvasbitmap.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/canvas/source/cairo/cairo_canvasbitmap.cxx b/canvas/source/cairo/cairo_canvasbitmap.cxx index 3e0bcae618f6..30ccb2cda8e8 100644 --- a/canvas/source/cairo/cairo_canvasbitmap.cxx +++ b/canvas/source/cairo/cairo_canvasbitmap.cxx @@ -145,7 +145,8 @@ namespace cairocanvas sal_Int64 CanvasBitmap::getSomething(css::uno::Sequence<sal_Int8> const & aIdentifier) { return comphelper::getSomethingImpl( - aIdentifier, this, comphelper::FallbackToGetSomethingOf<RepaintTarget>{}); + aIdentifier, this, comphelper::MixinToGetSomethingOf<SurfaceProvider>{}, + comphelper::FallbackToGetSomethingOf<RepaintTarget>{}); } css::uno::Sequence<sal_Int8> const & CanvasBitmap::getUnoTunnelId() { |