loplugin:passstuffbyref in svx

Change-Id: I7d405830785cb12aaceb65d6ed03dda04aa90fd8

34 files changed, 52 insertions, 52 deletions

diff --git a/include/svx/AccessibleControlShape.hxx b/include/svx/AccessibleControlShape.hxx
index 6d33e35ae969..c419d22c1282 100644
--- a/include/svx/AccessibleControlShape.hxx
+++ b/include/svx/AccessibleControlShape.hxx
@@ -59,7 +59,7 @@ public:
         const AccessibleShapeTreeInfo& rShapeTreeInfo);
     virtual ~AccessibleControlShape( );
 
-    css::uno::Reference< css::beans::XPropertySet > SAL_CALL  GetControlModel( ) { return m_xControlModel;} ;
+    const css::uno::Reference< css::beans::XPropertySet >& SAL_CALL  GetControlModel( ) { return m_xControlModel;} ;
     AccessibleControlShape* SAL_CALL GetLabeledByControlShape();
 protected:
     //---  XAccessible  ----------------------------------------
diff --git a/include/svx/AccessibleShape.hxx b/include/svx/AccessibleShape.hxx
index a3abffbd37cc..af842a0dad75 100644
--- a/include/svx/AccessibleShape.hxx
+++ b/include/svx/AccessibleShape.hxx
@@ -376,7 +376,7 @@ public:
 
     //===== Misc ========================================================
 
-    css::uno::Reference< css::drawing::XShape >
+    const css::uno::Reference< css::drawing::XShape >&
         GetXShape() { return mxShape; }
 
     /** set the index _nIndex at the accessible shape
diff --git a/include/svx/AccessibleShapeTreeInfo.hxx b/include/svx/AccessibleShapeTreeInfo.hxx
index 52a121001572..68430f647fc6 100644
--- a/include/svx/AccessibleShapeTreeInfo.hxx
+++ b/include/svx/AccessibleShapeTreeInfo.hxx
@@ -73,8 +73,8 @@ public:
 
     /** Deprecated.  Don't use this method.
     */
-    css::uno::Reference<
-        css::accessibility::XAccessibleComponent>
+    const css::uno::Reference<
+        css::accessibility::XAccessibleComponent>&
         GetDocumentWindow() const { return mxDocumentWindow;}
 
     /** Set a new broadcaster that sends events indicating shape changes.
@@ -91,8 +91,8 @@ public:
             The returned reference may be empty if the broadcaster has not
             been set or has been set to an empty reference.
     */
-    css::uno::Reference<
-        css::document::XEventBroadcaster>
+    const css::uno::Reference<
+        css::document::XEventBroadcaster>&
         GetModelBroadcaster() const { return mxModelBroadcaster;}
 
     /** Set the view that will be used to construct SvxTextEditSources which
@@ -122,8 +122,8 @@ public:
         @return
             The reference to the currently set controller may be empty.
     */
-    css::uno::Reference<
-        css::frame::XController>
+    const css::uno::Reference<
+        css::frame::XController>&
         GetController() const { return mxController;}
 
     /** Set the window that is used to construct SvxTextEditSources which in
diff --git a/include/svx/EnhancedCustomShape2d.hxx b/include/svx/EnhancedCustomShape2d.hxx
index c85f88856552..c1f5ea99eb63 100644
--- a/include/svx/EnhancedCustomShape2d.hxx
+++ b/include/svx/EnhancedCustomShape2d.hxx
@@ -185,7 +185,7 @@ class SVX_DLLPUBLIC EnhancedCustomShape2d : public SfxItemSet
         SdrObject*              CreateObject( bool bLineGeometryNeededOnly );
         void                    ApplyGluePoints( SdrObject* pObj );
         Rectangle               GetTextRect() const;
-        Rectangle               GetLogicRect() const { return aLogicRect; }
+        const Rectangle&        GetLogicRect() const { return aLogicRect; }
 
         sal_uInt32              GetHdlCount() const;
         bool                    GetHandlePosition( const sal_uInt32 nIndex, Point& rReturnPosition ) const;
diff --git a/include/svx/SmartTagItem.hxx b/include/svx/SmartTagItem.hxx
index 0cc025db9aca..bcfc8a9cd002 100644
--- a/include/svx/SmartTagItem.hxx
+++ b/include/svx/SmartTagItem.hxx
@@ -77,8 +77,8 @@ public:
     const css::uno::Reference<css::text::XTextRange>& GetTextRange() const { return mxRange; }
     const css::uno::Reference<css::frame::XController>& GetController() const { return mxController; }
     const css::lang::Locale& GetLocale() const { return maLocale; }
-    const OUString GetApplicationName() const { return maApplicationName; }
-    const OUString GetRangeText() const { return maRangeText; }
+    const OUString& GetApplicationName() const { return maApplicationName; }
+    const OUString& GetRangeText() const { return maRangeText; }
 };
 
 #endif // INCLUDED_SVX_SMARTTAGITEM_HXX
diff --git a/include/svx/SmartTagMgr.hxx b/include/svx/SmartTagMgr.hxx
index 3167da114962..ae741ac48f6c 100644
--- a/include/svx/SmartTagMgr.hxx
+++ b/include/svx/SmartTagMgr.hxx
@@ -215,7 +215,7 @@ public:
 
     /** Returns the name of the application this instance has been created by.
     */
-    const OUString GetApplicationName() const { return maApplicationName; }
+    const OUString& GetApplicationName() const { return maApplicationName; }
 
     // css::lang::XEventListener
     virtual void SAL_CALL disposing( const css::lang::EventObject& Source ) throw(css::uno::RuntimeException, std::exception) override;
diff --git a/include/svx/dlgctrl.hxx b/include/svx/dlgctrl.hxx
index cbbcb8f44d42..23d108d587bb 100644
--- a/include/svx/dlgctrl.hxx
+++ b/include/svx/dlgctrl.hxx
@@ -98,7 +98,7 @@ protected:
     void MarkToResetSettings(bool bUpdateForeground, bool bUpdateBackground);
 
     RECT_POINT          GetRPFromPoint( Point, bool bRTL = false ) const;
-    Point               GetPointFromRP( RECT_POINT ) const;
+    const Point&        GetPointFromRP( RECT_POINT ) const;
     void                SetFocusRect();
     Point               SetActualRPWithoutInvalidate( RECT_POINT eNewRP );  // returns the last point
 
diff --git a/include/svx/drawitem.hxx b/include/svx/drawitem.hxx
index 3cbcc729654b..64e94e9b7faf 100644
--- a/include/svx/drawitem.hxx
+++ b/include/svx/drawitem.hxx
@@ -46,7 +46,7 @@ public:
     virtual bool            QueryValue( css::uno::Any& rVal, sal_uInt8 nMemberId = 0 ) const override;
     virtual bool            PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId ) override;
 
-    XColorListRef           GetColorList() const { return pColorList; }
+    const XColorListRef&     GetColorList() const { return pColorList; }
 };
 
 class SAL_WARN_UNUSED SVX_DLLPUBLIC SvxGradientListItem : public SfxPoolItem
@@ -71,7 +71,7 @@ public:
     virtual bool            QueryValue( css::uno::Any& rVal, sal_uInt8 nMemberId = 0 ) const override;
     virtual bool            PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId ) override;
 
-    XGradientListRef        GetGradientList() const { return pGradientList; }
+    const XGradientListRef& GetGradientList() const { return pGradientList; }
 };
 
 class SAL_WARN_UNUSED SVX_DLLPUBLIC SvxHatchListItem : public SfxPoolItem
@@ -95,7 +95,7 @@ public:
     virtual bool            QueryValue( css::uno::Any& rVal, sal_uInt8 nMemberId = 0 ) const override;
     virtual bool            PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId ) override;
 
-    XHatchListRef           GetHatchList() const { return pHatchList; }
+    const XHatchListRef&    GetHatchList() const { return pHatchList; }
 };
 
 class SAL_WARN_UNUSED SVX_DLLPUBLIC SvxBitmapListItem : public SfxPoolItem
@@ -120,7 +120,7 @@ public:
     virtual bool            QueryValue( css::uno::Any& rVal, sal_uInt8 nMemberId = 0 ) const override;
     virtual bool            PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId ) override;
 
-    XBitmapListRef          GetBitmapList() const { return pBitmapList; }
+    const XBitmapListRef&   GetBitmapList() const { return pBitmapList; }
 };
 
 class SAL_WARN_UNUSED SVX_DLLPUBLIC SvxDashListItem : public SfxPoolItem
@@ -145,7 +145,7 @@ public:
     virtual bool            QueryValue( css::uno::Any& rVal, sal_uInt8 nMemberId = 0 ) const override;
     virtual bool            PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId ) override;
 
-    XDashListRef            GetDashList() const { return pDashList; }
+    const XDashListRef&     GetDashList() const { return pDashList; }
 };
 
 class SAL_WARN_UNUSED SVX_DLLPUBLIC SvxLineEndListItem : public SfxPoolItem
@@ -170,7 +170,7 @@ public:
     virtual bool            QueryValue( css::uno::Any& rVal, sal_uInt8 nMemberId = 0 ) const override;
     virtual bool            PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId ) override;
 
-    XLineEndListRef         GetLineEndList() const { return pLineEndList; }
+    const XLineEndListRef&  GetLineEndList() const { return pLineEndList; }
 };
 
 #endif
diff --git a/include/svx/fmsrcimp.hxx b/include/svx/fmsrcimp.hxx
index 6e5ed194d0ff..2f1f9ebdc454 100644
--- a/include/svx/fmsrcimp.hxx
+++ b/include/svx/fmsrcimp.hxx
@@ -126,7 +126,7 @@ namespace svxform {
         css::uno::Reference< css::uno::XInterface >   m_xControl;
         // attribute access
     public:
-        css::uno::Reference< css::uno::XInterface >  getControl() const{ return m_xControl; }
+        const css::uno::Reference< css::uno::XInterface >&  getControl() const{ return m_xControl; }
     public:
         ControlTextWrapper(const css::uno::Reference< css::uno::XInterface >& _xControl) { m_xControl = _xControl; }
         virtual ~ControlTextWrapper() { }
diff --git a/include/svx/galleryitem.hxx b/include/svx/galleryitem.hxx
index ecfe03f57a0c..988c4ef2074f 100644
--- a/include/svx/galleryitem.hxx
+++ b/include/svx/galleryitem.hxx
@@ -52,8 +52,8 @@ public:
     virtual ~SvxGalleryItem();
 
     sal_Int8 GetType() const { return m_nType; }
-    const rtl::OUString GetURL() const { return m_aURL; }
-    const css::uno::Reference< css::graphic::XGraphic > GetGraphic() const { return m_xGraphic; }
+    const rtl::OUString& GetURL() const { return m_aURL; }
+    const css::uno::Reference< css::graphic::XGraphic >& GetGraphic() const { return m_xGraphic; }
 
     // pure virtual methods from SfxPoolItem
     virtual bool         operator==( const SfxPoolItem& ) const override;
diff --git a/include/svx/galtheme.hxx b/include/svx/galtheme.hxx
index 6102248b0660..11e35a3c70b1 100644
--- a/include/svx/galtheme.hxx
+++ b/include/svx/galtheme.hxx
@@ -166,7 +166,7 @@ public:
     SAL_DLLPRIVATE void         AbortActualize() { bAbortActualize = true; }
 
     SAL_DLLPRIVATE Gallery*     GetParent() const { return pParent; }
-    SAL_DLLPRIVATE tools::SvRef<SotStorage> GetSvDrawStorage() const;
+    SAL_DLLPRIVATE const tools::SvRef<SotStorage>& GetSvDrawStorage() const;
 
 public:
 
diff --git a/include/svx/gridctrl.hxx b/include/svx/gridctrl.hxx
index f4540571063a..85c63469f5ee 100644
--- a/include/svx/gridctrl.hxx
+++ b/include/svx/gridctrl.hxx
@@ -494,7 +494,7 @@ public:
     */
     void        ForceHideScrollbars();
 
-    css::uno::Reference< css::uno::XComponentContext >
+    const css::uno::Reference< css::uno::XComponentContext >&
         getContext() const { return m_xContext; }
 
     /// returns <TRUE/> if the text of the given cell can be copied into the clipboard
diff --git a/include/svx/nbdtmg.hxx b/include/svx/nbdtmg.hxx
index 55059bc56776..fddb962e260d 100644
--- a/include/svx/nbdtmg.hxx
+++ b/include/svx/nbdtmg.hxx
@@ -225,7 +225,7 @@ class SVX_DLLPUBLIC NBOTypeMgrBase
         // store the attributes passed from pSet
         void SetItems(const SfxItemSet* pArg) { pSet = pArg;StoreBulCharFmtName_impl();StoreMapUnit_impl();}
     protected:
-        OUString GetBulCharFmtName() { return aNumCharFmtName;}
+        const OUString& GetBulCharFmtName() { return aNumCharFmtName;}
         SfxMapUnit GetMapUnit() { return eCoreUnit;}
     protected:
         bool    bIsLoading;
diff --git a/include/svx/scene3d.hxx b/include/svx/scene3d.hxx
index 70d77e8373f8..f8642e02b540 100644
--- a/include/svx/scene3d.hxx
+++ b/include/svx/scene3d.hxx
@@ -126,7 +126,7 @@ public:
 
     virtual E3dScene* GetScene() const override;
     void SetCamera(const Camera3D& rNewCamera);
-    Camera3D GetCamera() const { return aCamera; }
+    const Camera3D& GetCamera() const { return aCamera; }
     void removeAllNonSelectedObjects();
 
     virtual E3dScene* Clone() const override;
diff --git a/include/svx/sdr/overlay/overlaymanager.hxx b/include/svx/sdr/overlay/overlaymanager.hxx
index f91571e79e7f..64199a2ec550 100644
--- a/include/svx/sdr/overlay/overlaymanager.hxx
+++ b/include/svx/sdr/overlay/overlaymanager.hxx
@@ -117,11 +117,11 @@ namespace sdr
             virtual void invalidateRange(const basegfx::B2DRange& rRange);
 
             // stripe support ColA
-            Color getStripeColorA() const { return maStripeColorA; }
+            const Color& getStripeColorA() const { return maStripeColorA; }
             void setStripeColorA(Color aNew= Color(COL_BLACK));
 
             // stripe support ColB
-            Color getStripeColorB() const { return maStripeColorB; }
+            const Color& getStripeColorB() const { return maStripeColorB; }
             void setStripeColorB(Color aNew = Color(COL_WHITE));
 
             // stripe support StripeLengthPixel
diff --git a/include/svx/sdr/overlay/overlayobject.hxx b/include/svx/sdr/overlay/overlayobject.hxx
index 6a4da0b11336..1130a6da65a5 100644
--- a/include/svx/sdr/overlay/overlayobject.hxx
+++ b/include/svx/sdr/overlay/overlayobject.hxx
@@ -144,7 +144,7 @@ namespace sdr
             const basegfx::B2DRange& getBaseRange() const;
 
             // access to baseColor
-            Color getBaseColor() const { return maBaseColor; }
+            const Color& getBaseColor() const { return maBaseColor; }
             void setBaseColor(Color aNew);
 
             // execute event from base class sdr::animation::Event. Default
diff --git a/include/svx/sdr/overlay/overlaypolypolygon.hxx b/include/svx/sdr/overlay/overlaypolypolygon.hxx
index a59cfe0b2a98..7cc20f481aee 100644
--- a/include/svx/sdr/overlay/overlaypolypolygon.hxx
+++ b/include/svx/sdr/overlay/overlaypolypolygon.hxx
@@ -44,7 +44,7 @@ namespace sdr
             virtual ~OverlayPolyPolygonStripedAndFilled();
 
             // change geometry
-            basegfx::B2DPolyPolygon getLinePolyPolygon() const { return maLinePolyPolygon; }
+            const basegfx::B2DPolyPolygon& getLinePolyPolygon() const { return maLinePolyPolygon; }
 
             // react on stripe definition change
             virtual void stripeDefinitionHasChanged() override;
diff --git a/include/svx/sidebar/AreaPropertyPanelBase.hxx b/include/svx/sidebar/AreaPropertyPanelBase.hxx
index 4c67637756b7..e4ec0dfa2564 100644
--- a/include/svx/sidebar/AreaPropertyPanelBase.hxx
+++ b/include/svx/sidebar/AreaPropertyPanelBase.hxx
@@ -80,7 +80,7 @@ public:
     const static sal_Int32 DEFAULT_ENDVALUE;
     const static sal_Int32 DEFAULT_BORDER;
 
-    XGradient GetGradient (const css::awt::GradientStyle eStyle) const;
+    const XGradient& GetGradient (const css::awt::GradientStyle eStyle) const;
     void SetGradient (const XGradient& rGradient);
     sal_Int32 GetSelectedTransparencyTypeIndex() const;
 
diff --git a/include/svx/svdedxv.hxx b/include/svx/svdedxv.hxx
index 9ef93a14d382..519ba6eb5e2e 100644
--- a/include/svx/svdedxv.hxx
+++ b/include/svx/svdedxv.hxx
@@ -258,7 +258,7 @@ public:
 
     virtual void MarkListHasChanged() override;
 
-    rtl::Reference< sdr::SelectionController > getSelectionController() const { return mxSelectionController; }
+    const rtl::Reference< sdr::SelectionController >& getSelectionController() const { return mxSelectionController; }
 
     /** returns true if the shape identified by its inventor and identifier supports format paint brush operation */
     static bool SupportsFormatPaintbrush( sal_uInt32 nObjectInventor, sal_uInt16 nObjectIdentifier );
diff --git a/include/svx/svdhdl.hxx b/include/svx/svdhdl.hxx
index 9214f5acd8ec..f93386efd757 100644
--- a/include/svx/svdhdl.hxx
+++ b/include/svx/svdhdl.hxx
@@ -270,7 +270,7 @@ public:
 
     bool IsUseLuminance() const { return bUseLuminance; }
 
-    Color GetColor() const { return aMarkerColor; }
+    const Color& GetColor() const { return aMarkerColor; }
     void SetColor(Color aNew, bool bCallLink = false);
 
     void SetSize(const Size& rNew);
diff --git a/include/svx/svdmodel.hxx b/include/svx/svdmodel.hxx
index 9525e0cb3328..350acefe74dc 100644
--- a/include/svx/svdmodel.hxx
+++ b/include/svx/svdmodel.hxx
@@ -536,7 +536,7 @@ public:
 
     // Accessor methods for Palettes, Lists and Tables
     // FIXME: this badly needs re-factoring...
-    XPropertyListRef GetPropertyList( XPropertyListType t ) const { return maProperties[ t ]; }
+    const XPropertyListRef& GetPropertyList( XPropertyListType t ) const { return maProperties[ t ]; }
     void             SetPropertyList( XPropertyListRef p ) { maProperties[ p->Type() ] = p; }
 
     // friendlier helpers
diff --git a/include/svx/svdobj.hxx b/include/svx/svdobj.hxx
index a73485a9b881..3b8c5b9848ae 100644
--- a/include/svx/svdobj.hxx
+++ b/include/svx/svdobj.hxx
@@ -369,7 +369,7 @@ public:
     const double* GetRelativeHeight() const;
     sal_Int16 GetRelativeHeightRelation() const;
     // evil calc grid/shape drawlayer syncing
-    Point GetGridOffset() const { return aGridOffset; }
+    const Point& GetGridOffset() const { return aGridOffset; }
     void SetGridOffset( const Point& rGridOffset ){ aGridOffset = rGridOffset; }
 protected:
     Rectangle ImpDragCalcRect(const SdrDragStat& rDrag) const;
@@ -875,7 +875,7 @@ public:
 
     // access to the UNO representation of the shape
     virtual css::uno::Reference< css::uno::XInterface > getUnoShape();
-    css::uno::WeakReference< css::uno::XInterface > getWeakUnoShape() const { return maWeakUnoShape; }
+    const css::uno::WeakReference< css::uno::XInterface >& getWeakUnoShape() const { return maWeakUnoShape; }
 
     static SdrObject* getSdrObjectFromXShape( const css::uno::Reference< css::uno::XInterface >& xInt );
 
@@ -929,7 +929,7 @@ public:
     // the following methods are used to control it;
     // usually this data makes no sense after the import is finished, since the object
     // might be resized
-    Rectangle GetBLIPSizeRectangle() const { return maBLIPSizeRectangle;}
+    const Rectangle& GetBLIPSizeRectangle() const { return maBLIPSizeRectangle;}
     void SetBLIPSizeRectangle( const Rectangle& aRect );
 
     /// @see mbDoNotInsertIntoPageAutomatically
diff --git a/include/svx/svdoole2.hxx b/include/svx/svdoole2.hxx
index 00525e60bc9b..cd21df9241b4 100644
--- a/include/svx/svdoole2.hxx
+++ b/include/svx/svdoole2.hxx
@@ -93,7 +93,7 @@ public:
 
     // OLE object has got a separate PersistName member now;
     // !!! use ::SetPersistName( ... ) only, if you know what you do !!!
-    OUString      GetPersistName() const;
+    const OUString& GetPersistName() const;
     void        SetPersistName( const OUString& rPersistName );
 
     // One can add an application name to a SdrOle2Obj, which can be queried for
diff --git a/include/svx/svdouno.hxx b/include/svx/svdouno.hxx
index 6e0e7b88a878..59af7fd0a593 100644
--- a/include/svx/svdouno.hxx
+++ b/include/svx/svdouno.hxx
@@ -88,7 +88,7 @@ public:
 
     virtual void SetContextWritingMode( const sal_Int16 _nContextWritingMode ) override;
 
-    css::uno::Reference< css::awt::XControlModel > GetUnoControlModel() const {return xUnoControlModel;}
+    const css::uno::Reference< css::awt::XControlModel >& GetUnoControlModel() const {return xUnoControlModel;}
     css::uno::Reference< css::awt::XControl > GetUnoControl(const SdrView& _rView, const OutputDevice& _rOut) const;
 
     /** Retrieves a temporary XControl instance for a given output device
diff --git a/include/svx/svdpagv.hxx b/include/svx/svdpagv.hxx
index d6e1c713bc0c..f5b982117886 100644
--- a/include/svx/svdpagv.hxx
+++ b/include/svx/svdpagv.hxx
@@ -258,11 +258,11 @@ public:
     // #103834# Set background color for svx at SdrPageViews
     void SetApplicationBackgroundColor(Color aBackgroundColor);
 
-    Color GetApplicationBackgroundColor() const { return maBackgroundColor;}
+    const Color& GetApplicationBackgroundColor() const { return maBackgroundColor;}
 
     // #103911# Set/Get document color for svx at SdrPageViews
     void SetApplicationDocumentColor(Color aDocumentColor);
-    Color GetApplicationDocumentColor() const { return maDocumentColor;}
+    const Color& GetApplicationDocumentColor() const { return maDocumentColor;}
 };
 
 
diff --git a/include/svx/svdpntv.hxx b/include/svx/svdpntv.hxx
index a3d00a04ea1b..b270076cfec7 100644
--- a/include/svx/svdpntv.hxx
+++ b/include/svx/svdpntv.hxx
@@ -390,7 +390,7 @@ public:
     /// Draw Help line in fron of the objects or beging them
     bool IsHlplFront() const { return mbHlplFront  ; }
 
-    Color GetGridColor() const { return maGridColor;}
+    const Color& GetGridColor() const { return maGridColor;}
     void SetPageVisible(bool bOn = true) { mbPageVisible=bOn; InvalidateAllWin(); }
     void SetPageShadowVisible(bool bOn = true) { mbPageShadowVisible=bOn; InvalidateAllWin(); }
     void SetPageBorderVisible(bool bOn = true) { mbPageBorderVisible=bOn; InvalidateAllWin(); }
diff --git a/include/svx/svxgrahicitem.hxx b/include/svx/svxgrahicitem.hxx
index 00e2f0d1c089..794f68ffaa4d 100644
--- a/include/svx/svxgrahicitem.hxx
+++ b/include/svx/svxgrahicitem.hxx
@@ -38,7 +38,7 @@ public:
     virtual bool            operator==( const SfxPoolItem& ) const override;
     virtual SfxPoolItem*    Clone( SfxItemPool *pPool = nullptr ) const override;
 
-    Graphic             GetGraphic() const { return aGraphic; }
+    const Graphic&        GetGraphic() const { return aGraphic; }
 
 };
 
diff --git a/include/svx/ucsubset.hxx b/include/svx/ucsubset.hxx
index ccdc3d92f509..26464d934f38 100644
--- a/include/svx/ucsubset.hxx
+++ b/include/svx/ucsubset.hxx
@@ -45,7 +45,7 @@ public:
 
     sal_UCS4        GetRangeMin() const { return mnRangeMin;}
     sal_UCS4        GetRangeMax() const { return mnRangeMax;}
-    const OUString  GetName()     const { return maRangeName;}
+    const OUString& GetName()     const { return maRangeName;}
 
 private:
     sal_UCS4        mnRangeMin;
diff --git a/include/svx/unoshape.hxx b/include/svx/unoshape.hxx
index be0b4b5b3195..08c377040891 100644
--- a/include/svx/unoshape.hxx
+++ b/include/svx/unoshape.hxx
@@ -896,7 +896,7 @@ public:
     SvxDummyShapeContainer( css::uno::Reference< css::drawing::XShapes > xWrappedObject );
     virtual ~SvxDummyShapeContainer() throw();
 
-    css::uno::Reference< css::drawing::XShapes > getWrappedShape()
+    const css::uno::Reference< css::drawing::XShapes >& getWrappedShape()
             { return m_xDummyObject; }
 
 };
diff --git a/include/svx/xbitmap.hxx b/include/svx/xbitmap.hxx
index 7f514d190250..32a8e9fab11b 100644
--- a/include/svx/xbitmap.hxx
+++ b/include/svx/xbitmap.hxx
@@ -55,7 +55,7 @@ public:
     void SetBackgroundColor( const Color& rColor )      { aBckgrColor = rColor; bGraphicDirty = true; }
 
     Bitmap                  GetBitmap() const;
-    Color                   GetBackgroundColor() const  { return aBckgrColor; }
+    const Color&            GetBackgroundColor() const  { return aBckgrColor; }
 };
 
 #endif
diff --git a/include/svx/xgrad.hxx b/include/svx/xgrad.hxx
index 68cbae987ac3..c9dc407c11ef 100644
--- a/include/svx/xgrad.hxx
+++ b/include/svx/xgrad.hxx
@@ -63,9 +63,9 @@ public:
     void SetSteps(sal_uInt16 nSteps)                    { nStepCount = nSteps; }
 
     css::awt::GradientStyle GetGradientStyle() const         { return eStyle; }
-    Color          GetStartColor() const            { return aStartColor; }
-    Color          GetEndColor() const              { return aEndColor; }
-    long           GetAngle() const                 { return nAngle; }
+    const Color&       GetStartColor() const            { return aStartColor; }
+    const Color&       GetEndColor() const              { return aEndColor; }
+    long               GetAngle() const                 { return nAngle; }
     sal_uInt16         GetBorder() const                { return nBorder; }
     sal_uInt16         GetXOffset() const               { return nOfsX; }
     sal_uInt16         GetYOffset() const               { return nOfsY; }
diff --git a/include/svx/xhatch.hxx b/include/svx/xhatch.hxx
index 52010cd5f71f..cf97801b0787 100644
--- a/include/svx/xhatch.hxx
+++ b/include/svx/xhatch.hxx
@@ -49,7 +49,7 @@ public:
     void            SetAngle(long nNewAngle) { nAngle = nNewAngle; }
 
     css::drawing::HatchStyle     GetHatchStyle() const { return eStyle; }
-    Color           GetColor() const { return aColor; }
+    const Color&    GetColor() const { return aColor; }
     long            GetDistance() const { return nDistance; }
     long            GetAngle() const { return nAngle; }
 };
diff --git a/include/svx/xlnedit.hxx b/include/svx/xlnedit.hxx
index 930408b5b45c..24c579fd2f2c 100644
--- a/include/svx/xlnedit.hxx
+++ b/include/svx/xlnedit.hxx
@@ -54,7 +54,7 @@ public:
                                     SfxMapUnit ePresMetric,
                                     OUString &rText, const IntlWrapper * = nullptr ) const override;
 
-    basegfx::B2DPolyPolygon GetLineEndValue() const { return maPolyPolygon;}
+    const basegfx::B2DPolyPolygon& GetLineEndValue() const { return maPolyPolygon;}
     void SetLineEndValue(const basegfx::B2DPolyPolygon& rPolyPolygon) { maPolyPolygon = rPolyPolygon; Detach(); }
 
     XLineEndItem* checkForUniqueItem( SdrModel* pModel ) const;
diff --git a/include/svx/xlnstit.hxx b/include/svx/xlnstit.hxx
index 58b9200a59f2..50bba7b9547d 100644
--- a/include/svx/xlnstit.hxx
+++ b/include/svx/xlnstit.hxx
@@ -54,7 +54,7 @@ public:
                                     SfxMapUnit ePresMetric,
                                     OUString &rText, const IntlWrapper * = nullptr ) const override;
 
-    basegfx::B2DPolyPolygon GetLineStartValue() const { return maPolyPolygon;}
+    const basegfx::B2DPolyPolygon& GetLineStartValue() const { return maPolyPolygon;}
     void SetLineStartValue(const basegfx::B2DPolyPolygon& rPolyPolygon) { maPolyPolygon = rPolyPolygon; Detach(); }
 
     XLineStartItem* checkForUniqueItem( SdrModel* pModel ) const;