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/lwptaboverride.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/lwptaboverride.hxx')
-rw-r--r-- | lotuswordpro/source/filter/lwptaboverride.hxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lotuswordpro/source/filter/lwptaboverride.hxx b/lotuswordpro/source/filter/lwptaboverride.hxx index b9c28e21e5c2..8abcb540b38a 100644 --- a/lotuswordpro/source/filter/lwptaboverride.hxx +++ b/lotuswordpro/source/filter/lwptaboverride.hxx @@ -87,7 +87,7 @@ public: pStrm->SkipExtra(); } - inline LwpObjectID* GetTabRackID(); + inline LwpObjectID& GetTabRackID(); inline bool IsTabRackOverridden(); inline void Override(LwpTabOverride* pOther); inline void OverrideTabRack(LwpObjectID* pTabRackID); @@ -106,9 +106,9 @@ private: }; }; -inline LwpObjectID* LwpTabOverride::GetTabRackID() +inline LwpObjectID& LwpTabOverride::GetTabRackID() { - return &m_aTabRackID; + return m_aTabRackID; } inline void LwpTabOverride::Override(LwpTabOverride* pOther) @@ -117,7 +117,7 @@ inline void LwpTabOverride::Override(LwpTabOverride* pOther) { if (IsTabRackOverridden()) //m_aTabRackID = *(pOther->GetTabRackID()); - pOther->OverrideTabRack(GetTabRackID()); + pOther->OverrideTabRack(&GetTabRackID()); } } |