diff options
| author | Matteo Casalin <matteo.casalin@yahoo.com> | 2015-05-03 18:09:34 +0200 |
|---|---|---|
| committer | Matteo Casalin <matteo.casalin@yahoo.com> | 2015-05-03 22:31:58 +0200 |
| commit | e2a394d7d6f5961ab9c3da983f5f1390225ff6d6 (patch) | |
| tree | d5a327c781b37c7b813736cba0193a5c932060c9 /sw | |
| parent | cf8ae83b87ff0861ab9f2e67035d9daa98881604 (diff) | |
Reduce checks in loop
Change-Id: Ic4650f083ce501b75cbefd2711fcfe348e1d61ea
Diffstat (limited to 'sw')
| -rw-r--r-- | sw/source/core/fields/expfld.cxx | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/sw/source/core/fields/expfld.cxx b/sw/source/core/fields/expfld.cxx index 28f222c863e0..97774cf6b80a 100644 --- a/sw/source/core/fields/expfld.cxx +++ b/sw/source/core/fields/expfld.cxx @@ -559,12 +559,10 @@ sal_uInt16 SwSetExpFieldType::SetSeqRefNo( SwSetExpField& rFld ) if( USHRT_MAX != nNum ) { for( n = 0; n < aArr.size(); ++n ) - if( aArr[ n ] > nNum ) - return nNum; // no -> use it - else if( aArr[ n ] == nNum ) - break; // yes -> create new + if( aArr[ n ] >= nNum ) + break; - if( n == aArr.size() ) + if( n == aArr.size() || aArr[ n ] > nNum ) return nNum; // no -> use it } |