diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-12-25 19:21:12 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-12-26 07:16:07 +0100 |
commit | c54d34f70819c5928fe30585e86d744eda4a254a (patch) | |
tree | 81672bb2fd4c95cb09a6ed14197c8a0a8655dcb5 /canvas/source/cairo/cairo_spritecanvas.cxx | |
parent | 09895ae4963850c05d7a315599195dd177f13b06 (diff) |
loplugin:passstuffbyref improved return in canvas and svtools
and for now, ignore methods with params so we don't fall into the trap
of thinking that calls to methods like:
Bar& foo(Bar &p) { return p; }
can be converted from
Bar f() { return foo(Bar()); }
to
Bar const & f() { return foo(Bar()); }
Change-Id: Ia3795eb2baf353cb6bec4ebf40451f2789d66ad7
Reviewed-on: https://gerrit.libreoffice.org/47034
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'canvas/source/cairo/cairo_spritecanvas.cxx')
-rw-r--r-- | canvas/source/cairo/cairo_spritecanvas.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/canvas/source/cairo/cairo_spritecanvas.cxx b/canvas/source/cairo/cairo_spritecanvas.cxx index 12750d868644..578fa1d12dbc 100644 --- a/canvas/source/cairo/cairo_spritecanvas.cxx +++ b/canvas/source/cairo/cairo_spritecanvas.cxx @@ -173,12 +173,12 @@ namespace cairocanvas return maDeviceHelper.getOutputDevice(); } - SurfaceSharedPtr SpriteCanvas::getBufferSurface() + SurfaceSharedPtr const & SpriteCanvas::getBufferSurface() { return maDeviceHelper.getBufferSurface(); } - SurfaceSharedPtr SpriteCanvas::getWindowSurface() + SurfaceSharedPtr const & SpriteCanvas::getWindowSurface() { return maDeviceHelper.getWindowSurface(); } |