diff options
author | Noel Grandin <noel@peralex.com> | 2014-06-18 12:14:29 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2014-06-24 11:34:21 +0200 |
commit | e2080e70fe8b085f18e868e46340454720fa94ca (patch) | |
tree | 4038d1d57b41b68a47d5ebbbe6ad390648ec6303 /lotuswordpro/source/filter/lwplayout.hxx | |
parent | f910280b8704ed9c289150a4ca3c8d60e15d0d97 (diff) |
new compilerplugin returnbyref
Find places where we are returning a pointer to something, where we can
be returning a reference.
e.g.
class A {
struct X x;
public X* getX() { return &x; }
}
which can be:
public X& getX() { return x; }
Change-Id: I796fd23fd36a18aedf6e36bc28f8fab4f518c6c7
Diffstat (limited to 'lotuswordpro/source/filter/lwplayout.hxx')
-rw-r--r-- | lotuswordpro/source/filter/lwplayout.hxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lotuswordpro/source/filter/lwplayout.hxx b/lotuswordpro/source/filter/lwplayout.hxx index 38c667377487..e1ffb4fc8b87 100644 --- a/lotuswordpro/source/filter/lwplayout.hxx +++ b/lotuswordpro/source/filter/lwplayout.hxx @@ -228,8 +228,8 @@ public: LwpAssociatedLayouts(){} public: void Read(LwpObjectStream* pStrm); - LwpObjectID* GetOnlyLayout(){return &m_OnlyLayout;} - LwpDLVListHeadTail* GetLayouts(){return &m_Layouts;} + LwpObjectID& GetOnlyLayout() { return m_OnlyLayout;} + LwpDLVListHeadTail& GetLayouts() { return m_Layouts;} LwpVirtualLayout* GetLayout(LwpVirtualLayout* pStartLayout); protected: LwpObjectID m_OnlyLayout; //LwpVirtualLayout @@ -348,7 +348,7 @@ protected: LwpObjectID m_LayBackgroundStuff; LwpObjectID m_LayExtBorderStuff; public: - LwpObjectID* GetContent(){return &m_Content;} + LwpObjectID& GetContent() { return m_Content; } LwpTabOverride* GetTabOverride(); }; @@ -393,7 +393,7 @@ public: virtual bool IsUseOnAllEvenPages() SAL_OVERRIDE; virtual bool IsUseOnAllOddPages() SAL_OVERRIDE; virtual bool IsUseOnPage() SAL_OVERRIDE; - LwpObjectID* GetPosition(){ return &m_Positon;} + LwpObjectID& GetPosition(){ return m_Positon;} virtual LwpVirtualLayout* GetContainerLayout() SAL_OVERRIDE; }; |