diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2022-11-10 19:01:49 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2022-11-11 12:30:03 +0100 |
commit | ed38659127ac801666de1bc423855d2b666f7150 (patch) | |
tree | e6762c61f29545c0c4309bf73bc708a3e42a2cc3 /comphelper | |
parent | dbbfcfed6f877ad2446daefe1083f42994f28a1f (diff) |
Drop counterproductive top-level const
...that was apparently left by accident by
24c709655e5c1ca51c6c6da7ef0cb733354f6d20 "clang-tidy modernize-pass-by-value in
comphelper", thwarting the use of std::move here.
And clean up the declarations in the include file, too.
(I came across this code with an upcoming loplugin:constmove that flags
suspicious uses of std::move involving const-qualified types.)
Change-Id: I8178ad09bc4e7d4f57c5dfd9b0e2103e6d4fa72a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142566
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'comphelper')
-rw-r--r-- | comphelper/source/container/enumhelper.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/comphelper/source/container/enumhelper.cxx b/comphelper/source/container/enumhelper.cxx index bc9ae43e44ad..66ba15198232 100644 --- a/comphelper/source/container/enumhelper.cxx +++ b/comphelper/source/container/enumhelper.cxx @@ -36,7 +36,7 @@ OEnumerationByName::OEnumerationByName(css::uno::Reference<css::container::XName } -OEnumerationByName::OEnumerationByName(const css::uno::Reference<css::container::XNameAccess> _xAccess, +OEnumerationByName::OEnumerationByName(css::uno::Reference<css::container::XNameAccess> _xAccess, std::vector<OUString> _aNames ) :m_aNames(std::move(_aNames)) ,m_xAccess(std::move(_xAccess)) |