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 /sw/source/filter/basflt/shellio.cxx | |
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 'sw/source/filter/basflt/shellio.cxx')
-rw-r--r-- | sw/source/filter/basflt/shellio.cxx | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sw/source/filter/basflt/shellio.cxx b/sw/source/filter/basflt/shellio.cxx index 8162258b726a..a5eb5e414c90 100644 --- a/sw/source/filter/basflt/shellio.cxx +++ b/sw/source/filter/basflt/shellio.cxx @@ -203,15 +203,15 @@ sal_uLong SwReader::Read( const Reader& rOptions ) { *pUndoPam->GetMark() = *pPam->GetPoint(); pUndoPam->GetPoint()->nNode++; - SwNode* pNd = pUndoPam->GetNode(); - if( pNd->IsCntntNode() ) + SwNode& rNd = pUndoPam->GetNode(); + if( rNd.IsCntntNode() ) pUndoPam->GetPoint()->nContent.Assign( - (SwCntntNode*)pNd, nSttCntnt ); + (SwCntntNode*)&rNd, nSttCntnt ); else pUndoPam->GetPoint()->nContent.Assign( 0, 0 ); - bool bChkHeaderFooter = pNd->FindHeaderStartNode() || - pNd->FindFooterStartNode(); + bool bChkHeaderFooter = rNd.FindHeaderStartNode() || + rNd.FindFooterStartNode(); // search all new Fly's, and store them as individual Undo Objects for( sal_uInt16 n = 0; n < pDoc->GetSpzFrmFmts()->size(); ++n ) |