diff options
author | Michael Weghorn <m.weghorn@posteo.de> | 2022-03-01 10:27:11 +0000 |
---|---|---|
committer | Michael Weghorn <m.weghorn@posteo.de> | 2022-03-01 15:52:04 +0100 |
commit | 45ae2fb7a7de4da4c3d6c7c359aaf92ccf0688a2 (patch) | |
tree | 8d8f38b1c7ebcb0f2b731d7b894e20c75f295ea5 /winaccessibility | |
parent | 774364243b5541328aa633d7ae535fa4ae716105 (diff) |
wina11y: Use more range-based for
And use const reference to css::style::TabStop instead of
creating a copy.
Change-Id: I4092cbefcfbdaa6a6697974b7f40242e26cc6aad
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/130758
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Diffstat (limited to 'winaccessibility')
-rw-r--r-- | winaccessibility/source/UAccCOM/MAccessible.cxx | 28 |
1 files changed, 11 insertions, 17 deletions
diff --git a/winaccessibility/source/UAccCOM/MAccessible.cxx b/winaccessibility/source/UAccCOM/MAccessible.cxx index a24536c8f36f..56542b10028f 100644 --- a/winaccessibility/source/UAccCOM/MAccessible.cxx +++ b/winaccessibility/source/UAccCOM/MAccessible.cxx @@ -2712,43 +2712,37 @@ OUString CMAccessible::get_StringFromAny(Any const & pAny) OUStringBuffer pString; - int count = val.getLength(); + for (const OUString& rElem : val) + pString.append(rElem); - for( int iIndex = 0;iIndex < count;iIndex++ ) - { - pString.append(val[iIndex]); - } return pString.makeStringAndClear(); } else if (pAny.getValueType() == cppu::UnoType<Sequence< css::style::TabStop >>::get()) { Sequence < css::style::TabStop > val; pAny >>= val; - int count = val.getLength(); OUStringBuffer buf; - for( int iIndex = 0;iIndex < count;iIndex++ ) + for (const css::style::TabStop& rSingleVal : val) { - css::style::TabStop sigleVal = val[iIndex]; - buf.append("Position="); - buf.append(sigleVal.Position); + buf.append(rSingleVal.Position); buf.append(",TabAlign="); - buf.append(sal_Int32(sigleVal.Alignment)); + buf.append(sal_Int32(rSingleVal.Alignment)); buf.append(","); buf.append("DecimalChar="); - if(sigleVal.DecimalChar==';' || sigleVal.DecimalChar == ':' || sigleVal.DecimalChar == ',' || - sigleVal.DecimalChar == '=' || sigleVal.DecimalChar == '\\') + if (rSingleVal.DecimalChar==';' || rSingleVal.DecimalChar == ':' || rSingleVal.DecimalChar == ',' || + rSingleVal.DecimalChar == '=' || rSingleVal.DecimalChar == '\\') buf.append('\\'); - buf.append(sigleVal.DecimalChar); + buf.append(rSingleVal.DecimalChar); buf.append(","); buf.append("FillChar="); - if(sigleVal.FillChar==';' || sigleVal.FillChar == ':' || sigleVal.FillChar == ',' || - sigleVal.FillChar == '=' || sigleVal.FillChar == '\\') + if (rSingleVal.FillChar==';' || rSingleVal.FillChar == ':' || rSingleVal.FillChar == ',' || + rSingleVal.FillChar == '=' || rSingleVal.FillChar == '\\') buf.append('\\'); - buf.append(sigleVal.FillChar); + buf.append(rSingleVal.FillChar); buf.append(","); } return buf.makeStringAndClear(); |