diff options
| author | Noel Grandin <noel@peralex.com> | 2015-07-03 11:31:14 +0200 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2015-07-06 07:04:50 +0000 |
| commit | e9c3583c2cc27fc88ee81047c236ec99dd51e8de (patch) | |
| tree | b3e8394ca1ec402a31b227339366fc790124c1f8 /sc/inc | |
| parent | 89c77994d4638c86635c70535fab6508e2f3d900 (diff) | |
improve the returnbyref loplugin
Change-Id: I1b510a6194282dfa4a9001d473127c5ebc8b44eb
Reviewed-on: https://gerrit.libreoffice.org/16731
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'sc/inc')
| -rw-r--r-- | sc/inc/column.hxx | 2 | ||||
| -rw-r--r-- | sc/inc/detdata.hxx | 2 | ||||
| -rw-r--r-- | sc/inc/dpdimsave.hxx | 4 | ||||
| -rw-r--r-- | sc/inc/dpobject.hxx | 4 | ||||
| -rw-r--r-- | sc/inc/userlist.hxx | 4 |
5 files changed, 8 insertions, 8 deletions
diff --git a/sc/inc/column.hxx b/sc/inc/column.hxx index 9a1ac59b9a86..9d073ef0f4bc 100644 --- a/sc/inc/column.hxx +++ b/sc/inc/column.hxx @@ -429,7 +429,7 @@ public: void PreprocessDBDataUpdate( sc::EndListeningContext& rEndListenCxt, sc::CompileFormulaContext& rCompileCxt ); - const SfxPoolItem* GetAttr( SCROW nRow, sal_uInt16 nWhich ) const; + const SfxPoolItem& GetAttr( SCROW nRow, sal_uInt16 nWhich ) const; const ScPatternAttr* GetPattern( SCROW nRow ) const; const ScPatternAttr* GetMostUsedPattern( SCROW nStartRow, SCROW nEndRow ) const; diff --git a/sc/inc/detdata.hxx b/sc/inc/detdata.hxx index e222cdd36713..110f4a11e316 100644 --- a/sc/inc/detdata.hxx +++ b/sc/inc/detdata.hxx @@ -76,7 +76,7 @@ public: void Append( ScDetOpData* pData ); ScDetOpDataVector& GetDataVector() { return aDetOpDataVector; } - const ScDetOpData* GetObject( size_t nPos ) const; + const ScDetOpData& GetObject( size_t nPos ) const; bool HasAddError() const { return bHasAddError; } size_t Count() const { return aDetOpDataVector.size(); } diff --git a/sc/inc/dpdimsave.hxx b/sc/inc/dpdimsave.hxx index 49c400cbabea..3c410643336d 100644 --- a/sc/inc/dpdimsave.hxx +++ b/sc/inc/dpdimsave.hxx @@ -111,8 +111,8 @@ public: bool HasOnlyHidden(const ScDPUniqueStringSet& rVisible); long GetGroupCount() const; - const ScDPSaveGroupItem* GetGroupByIndex( long nIndex ) const; - ScDPSaveGroupItem* GetGroupAccByIndex( long nIndex ); + const ScDPSaveGroupItem& GetGroupByIndex( long nIndex ) const; + ScDPSaveGroupItem& GetGroupAccByIndex( long nIndex ); void Rename( const OUString& rNewName ); diff --git a/sc/inc/dpobject.hxx b/sc/inc/dpobject.hxx index 9004151b6d8a..a167194683a2 100644 --- a/sc/inc/dpobject.hxx +++ b/sc/inc/dpobject.hxx @@ -373,8 +373,8 @@ public: bool ReloadGroupsInCache(ScDPObject* pDPObj, std::set<ScDPObject*>& rRefs); SC_DLLPUBLIC size_t GetCount() const; - SC_DLLPUBLIC ScDPObject* operator[](size_t nIndex); - SC_DLLPUBLIC const ScDPObject* operator[](size_t nIndex) const; + SC_DLLPUBLIC ScDPObject& operator[](size_t nIndex); + SC_DLLPUBLIC const ScDPObject& operator[](size_t nIndex) const; const ScDPObject* GetByName(const OUString& rName) const; diff --git a/sc/inc/userlist.hxx b/sc/inc/userlist.hxx index 8cba99245fb3..d47e795a0bb1 100644 --- a/sc/inc/userlist.hxx +++ b/sc/inc/userlist.hxx @@ -76,8 +76,8 @@ public: /// If the list in rStr is already inserted bool HasEntry( const OUString& rStr ) const; - const ScUserListData* operator[](size_t nIndex) const; - ScUserListData* operator[](size_t nIndex); + const ScUserListData& operator[](size_t nIndex) const; + ScUserListData& operator[](size_t nIndex); ScUserList& operator= ( const ScUserList& r ); bool operator==( const ScUserList& r ) const; bool operator!=( const ScUserList& r ) const; |