diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-23 07:50:57 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-23 11:48:22 +0200 |
commit | 51d1ec7883444e6216f1020122084d3a996c9ecb (patch) | |
tree | 8b24c91daecfa2ccb922b6e1c91e29c3bf89e7dc /filter/source/xsltfilter | |
parent | tdf#120794 crash while selecting a drawing line (diff) | |
download | core-51d1ec7883444e6216f1020122084d3a996c9ecb.tar.gz core-51d1ec7883444e6216f1020122084d3a996c9ecb.zip |
clang-tidy performance-unnecessary-copy-init in editeng..i18npool
Change-Id: I2ee2c8d15d8700cfaa1697363da4557c741a5f36
Reviewed-on: https://gerrit.libreoffice.org/62216
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'filter/source/xsltfilter')
-rw-r--r-- | filter/source/xsltfilter/LibXSLTTransformer.cxx | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/filter/source/xsltfilter/LibXSLTTransformer.cxx b/filter/source/xsltfilter/LibXSLTTransformer.cxx index b27546e1343f..acc81d4e1196 100644 --- a/filter/source/xsltfilter/LibXSLTTransformer.cxx +++ b/filter/source/xsltfilter/LibXSLTTransformer.cxx @@ -421,10 +421,8 @@ namespace XSLT void LibXSLTTransformer::start() { - ListenerList* l = &m_listeners; - for (auto const& elem : *l) + for (const css::uno::Reference<XStreamListener>& xl : m_listeners) { - css::uno::Reference<XStreamListener> xl = elem; xl.get()->started(); } OSL_ENSURE(!m_Reader.is(), "Somebody forgot to call terminate *and* holds a reference to this LibXSLTTransformer instance"); @@ -435,12 +433,10 @@ namespace XSLT void LibXSLTTransformer::error(const OUString& msg) { - ListenerList* l = &m_listeners; Any arg; arg <<= Exception(msg, *this); - for (auto const& elem : *l) + for (const css::uno::Reference<XStreamListener>& xl : m_listeners) { - css::uno::Reference<XStreamListener> xl = elem; if (xl.is()) { xl.get()->error(arg); @@ -451,10 +447,8 @@ namespace XSLT void LibXSLTTransformer::done() { - ListenerList* l = &m_listeners; - for (auto const& elem : *l) + for (const css::uno::Reference<XStreamListener>& xl : m_listeners) { - css::uno::Reference<XStreamListener> xl = elem; if (xl.is()) { xl.get()->closed(); |