diff options
author | Albert Astals Cid <aacid@kde.org> | 2019-03-11 23:37:25 +0100 |
---|---|---|
committer | Albert Astals Cid <aacid@kde.org> | 2019-03-11 23:37:31 +0100 |
commit | 7e55f08343ef888dbdcd7a521f437102f95681ab (patch) | |
tree | db55ece64a89a8cbce5c2a47a7a357585751f9db /src/core/bufferfragment_p.h | |
parent | 5a2ca3652ff6deb450470f95863bfbff7c3da00d (diff) | |
download | kconfig-7e55f08343ef888dbdcd7a521f437102f95681ab.tar.gz kconfig-7e55f08343ef888dbdcd7a521f437102f95681ab.tar.bz2 |
Remove reference from const KConfigIniBackend::BufferFragment &
Summary: According to clazy since KConfigIniBackend::BufferFragment is very small it's faster to just copy it
Reviewers: apol
Reviewed By: apol
Subscribers: apol, kde-frameworks-devel
Tags: #frameworks
Differential Revision: https://phabricator.kde.org/D19666
Diffstat (limited to 'src/core/bufferfragment_p.h')
-rw-r--r-- | src/core/bufferfragment_p.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/core/bufferfragment_p.h b/src/core/bufferfragment_p.h index bbc09a2b..b4aff92a 100644 --- a/src/core/bufferfragment_p.h +++ b/src/core/bufferfragment_p.h @@ -150,7 +150,7 @@ public: return (other.size() != (int)len || memcmp(d, other.constData(), len) != 0); } - bool operator==(const BufferFragment &other) const + bool operator==(const BufferFragment other) const { return other.len == len && !memcmp(d, other.d, len); } @@ -195,7 +195,7 @@ private: unsigned int len; }; -uint qHash(const KConfigIniBackend::BufferFragment &fragment) +uint qHash(const KConfigIniBackend::BufferFragment fragment) { const uchar *p = reinterpret_cast<const uchar*>(fragment.constData()); const int len = fragment.length(); |