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 /dtrans/source | |
| 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 'dtrans/source')
| -rw-r--r-- | dtrans/source/generic/generic_clipboard.cxx | 6 | ||||
| -rw-r--r-- | dtrans/source/win32/dtobj/DOTransferable.cxx | 4 |
2 files changed, 5 insertions, 5 deletions
diff --git a/dtrans/source/generic/generic_clipboard.cxx b/dtrans/source/generic/generic_clipboard.cxx index 4fa49b3d5ebe..4f358db37336 100644 --- a/dtrans/source/generic/generic_clipboard.cxx +++ b/dtrans/source/generic/generic_clipboard.cxx @@ -45,10 +45,10 @@ void SAL_CALL GenericClipboard::initialize( const Sequence< Any >& aArguments ) { if (!m_bInitialized) { - for (sal_Int32 n = 0, nmax = aArguments.getLength(); n < nmax; n++) - if (aArguments[n].getValueType() == cppu::UnoType<OUString>::get()) + for (Any const & arg : aArguments) + if (arg.getValueType() == cppu::UnoType<OUString>::get()) { - aArguments[0] >>= m_aName; + arg >>= m_aName; break; } } diff --git a/dtrans/source/win32/dtobj/DOTransferable.cxx b/dtrans/source/win32/dtobj/DOTransferable.cxx index f90406462c81..8f27e7124e0a 100644 --- a/dtrans/source/win32/dtobj/DOTransferable.cxx +++ b/dtrans/source/win32/dtobj/DOTransferable.cxx @@ -294,8 +294,8 @@ sal_Bool SAL_CALL CDOTransferable::isDataFlavorSupported( const DataFlavor& aFla { OSL_ASSERT( isValidFlavor( aFlavor ) ); - for ( sal_Int32 i = 0; i < m_FlavorList.getLength( ); i++ ) - if ( compareDataFlavors( aFlavor, m_FlavorList[i] ) ) + for ( DataFlavor const & df : std::as_const(m_FlavorList) ) + if ( compareDataFlavors( aFlavor, df ) ) return true; return false; |