diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-05-20 08:43:33 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-05-20 12:11:11 +0200 |
commit | 1553d3787cbe0cdababf31382bf3376a3640d8cf (patch) | |
tree | b829cc1f97dac33abdf1e592a636d6fb24497f13 /dbaccess/source/ui | |
parent | c2ead5a142be19cb74127294641ec35da9e0f5c5 (diff) |
use for-range on Sequence in d*
and fix bug in GenericClipboard::initialize, where it was looping through
the arguments, but always reading the first one.
I'm guessing it was never an issue because it is always called with
only one argument
Change-Id: I8f72b6bce8c77a69c7d75115e34630e2c308261e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94553
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'dbaccess/source/ui')
-rw-r--r-- | dbaccess/source/ui/inc/sbamultiplex.hxx | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/dbaccess/source/ui/inc/sbamultiplex.hxx b/dbaccess/source/ui/inc/sbamultiplex.hxx index 920978437036..c71c9cbbeea5 100644 --- a/dbaccess/source/ui/inc/sbamultiplex.hxx +++ b/dbaccess/source/ui/inc/sbamultiplex.hxx @@ -264,11 +264,10 @@ namespace dbaui sal_Int32 classname::getOverallLen() const \ { \ sal_Int32 nLen = 0; \ - css::uno::Sequence< OUString > aContained = m_aListeners.getContainedTypes(); \ - const OUString* pContained = aContained.getConstArray(); \ - for ( sal_Int32 i=0; i<aContained.getLength(); ++i, ++pContained) \ + const css::uno::Sequence< OUString > aContained = m_aListeners.getContainedTypes(); \ + for ( OUString const & s : aContained) \ { \ - ::cppu::OInterfaceContainerHelper* pListeners = m_aListeners.getContainer(*pContained); \ + ::cppu::OInterfaceContainerHelper* pListeners = m_aListeners.getContainer(s); \ if (!pListeners) \ continue; \ nLen += pListeners->getLength(); \ |