diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2019-02-06 14:51:25 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2019-02-06 17:32:24 +0100 |
| commit | c06b120b1813a79c82aed465c8ba5cf5373c45bb (patch) | |
| tree | 07dd3fadcfd20abd6de12e9a2ee0a8394aded2ad | |
| parent | 70e9a35b575ba29431a421ec01af6298dfc57377 (diff) | |
nNumElemVert is already known to be >= 4
cause the outer if is...
if ( pAny && ( *pAny >>= seqCoordinates ) && ( seqCoordinates.getLength() >= 4 ) )
{
... nNumElemVert = seqCoordinates.getLength();
}
so this check is always true
Change-Id: I512b9440969775224c04d8566401b63f819711aa
Reviewed-on: https://gerrit.libreoffice.org/67464
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
| -rw-r--r-- | filter/source/msfilter/msdffimp.cxx | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/filter/source/msfilter/msdffimp.cxx b/filter/source/msfilter/msdffimp.cxx index 2a934272186a..45c8836f9d10 100644 --- a/filter/source/msfilter/msdffimp.cxx +++ b/filter/source/msfilter/msdffimp.cxx @@ -4541,11 +4541,10 @@ SdrObject* SvxMSDffManager::ImportShape( const DffRecordHeader& rHd, SvStream& r aXP[ static_cast<sal_uInt16>(nPtNum) ] = aP; } aPolyBoundRect = aXP.GetBoundRect(); - if ( nNumElemVert >= 3 ) - { // arc first command is always wr -- clockwise arc - // the parameters are : (left,top),(right,bottom),start(x,y),end(x,y) - aStartPt = aXP[2]; - } + + // arc first command is always wr -- clockwise arc + // the parameters are : (left,top),(right,bottom),start(x,y),end(x,y) + aStartPt = aXP[2]; } else aPolyBoundRect = tools::Rectangle( -21600, 0, 21600, 43200 ); // defaulting |