diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2024-11-05 11:19:51 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2024-11-07 17:20:09 +0100 |
commit | 5fdfc074c93447f9eb1c9a351ee4690126ba782c (patch) | |
tree | 76f5c4c3d4de02117fd8267cbdf06db3ae7fef27 /extensions/source/abpilot | |
parent | 0fe2b2545da977cd462c06e2a6ec6551b9903497 (diff) |
loplugin:passstuffbyref make some small improvements
Change-Id: Ib14a2e6b41165887fcf99c3d155850faa8564822
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176218
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'extensions/source/abpilot')
-rw-r--r-- | extensions/source/abpilot/datasourcehandling.cxx | 4 | ||||
-rw-r--r-- | extensions/source/abpilot/datasourcehandling.hxx | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/extensions/source/abpilot/datasourcehandling.cxx b/extensions/source/abpilot/datasourcehandling.cxx index fb65c997187a..82013fecea3d 100644 --- a/extensions/source/abpilot/datasourcehandling.cxx +++ b/extensions/source/abpilot/datasourcehandling.cxx @@ -441,10 +441,10 @@ namespace abp } - OUString ODataSource::getName() const + const OUString & ODataSource::getName() const { if ( !isValid() ) - return OUString(); + return EMPTY_OUSTRING; return m_pImpl->sName; } diff --git a/extensions/source/abpilot/datasourcehandling.hxx b/extensions/source/abpilot/datasourcehandling.hxx index c6058e45e30a..4a755096a8d3 100644 --- a/extensions/source/abpilot/datasourcehandling.hxx +++ b/extensions/source/abpilot/datasourcehandling.hxx @@ -123,7 +123,7 @@ namespace abp // TODO: put this into the context class /// returns the name of the data source - OUString + const OUString & getName() const; /// renames the data source |