diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-02 15:10:58 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-03 08:07:41 +0100 |
commit | acdba3c2eee18ef0c079b7c41cd4165e06c956c7 (patch) | |
tree | 18c6da7b7d46bf901b6dd4f7a5b280caa7f62280 /basctl | |
parent | Remove unused header. (diff) | |
download | core-acdba3c2eee18ef0c079b7c41cd4165e06c956c7.tar.gz core-acdba3c2eee18ef0c079b7c41cd4165e06c956c7.zip |
loplugin:passstuffbyref more return improvements
slightly less restrictive check when calling functions
Change-Id: I35e268ac611797b1daa83777cda02288a635aa32
Reviewed-on: https://gerrit.libreoffice.org/47259
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'basctl')
-rw-r--r-- | basctl/source/basicide/baside2.hxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/basctl/source/basicide/baside2.hxx b/basctl/source/basicide/baside2.hxx index 1e97b1cf9040..79428e116b9e 100644 --- a/basctl/source/basicide/baside2.hxx +++ b/basctl/source/basicide/baside2.hxx @@ -422,7 +422,7 @@ public: void BasicRemoveWatch (); Color const & GetBackgroundColor () const { return aSyntaxColors.GetBackgroundColor(); } Color const & GetFontColor () const { return aSyntaxColors.GetFontColor(); } - Color GetSyntaxColor (TokenType eType) const { return aSyntaxColors.GetColor(eType); } + Color const & GetSyntaxColor (TokenType eType) const { return aSyntaxColors.GetColor(eType); } protected: // Window: @@ -449,7 +449,7 @@ private: public: Color const & GetBackgroundColor () const { return m_aBackgroundColor; }; Color const & GetFontColor () const { return m_aFontColor; } - Color GetColor (TokenType eType) const { return aColors[eType]; } + Color const & GetColor(TokenType eType) const { return aColors[eType]; } private: virtual void ConfigurationChanged (utl::ConfigurationBroadcaster*, ConfigurationHints) override; |