diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2019-10-03 20:45:02 +0200 |
|---|---|---|
| committer | Julien Nabet <serval2412@yahoo.fr> | 2019-10-03 22:22:20 +0200 |
| commit | 30308a8b4c89ea806dd61a20707bbe28c27286dc (patch) | |
| tree | 9e3a7a8ae77266edf07aec281fa383122d4b54ca /svl | |
| parent | 34095197fcb992d3c1ea63888918495fcc6f6994 (diff) | |
Following "Simplify a bit SfxAllEnumItem (svl)"
As Mike suggested in:
https://gerrit.libreoffice.org/#/c/80089/
Change-Id: Ie33cb1464907215ec23bf7be7cf5ce3fafdf6113
Reviewed-on: https://gerrit.libreoffice.org/80161
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Diffstat (limited to 'svl')
| -rw-r--r-- | svl/source/items/aeitem.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/svl/source/items/aeitem.cxx b/svl/source/items/aeitem.cxx index cafa6fe3a3b7..2d2ce6d10ea5 100644 --- a/svl/source/items/aeitem.cxx +++ b/svl/source/items/aeitem.cxx @@ -108,10 +108,9 @@ void SfxAllEnumItem::InsertValue( sal_uInt16 nValue, const OUString &rValue ) SfxAllEnumValue_Impl aVal; aVal.nValue = nValue; aVal.aText = rValue; - sal_uInt16 nPos = GetPosByValue(nValue); if ( !pValues ) pValues.reset( new SfxAllEnumValueArr ); - else if ( nPos != USHRT_MAX ) + else if ( sal_uInt16 nPos = GetPosByValue(nValue); nPos != USHRT_MAX ) { // remove when exists pValues->erase( pValues->begin() + nPos ); |