diff options
author | Caolán McNamara <caolanm@redhat.com> | 2015-10-14 10:01:06 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2015-10-14 10:59:46 +0100 |
commit | 6dd3c3558291f8b23e339efd967a1ef76021b1cf (patch) | |
tree | 4461e9aab50cfc4f3571c1f237a77b5df4fcd123 /sw | |
parent | d7b75ef4ff20318c16ede48b7daa7b8c3240496d (diff) |
coverity#735965 Unchecked dynamic_cast
Change-Id: Iaeccbd37ae67a6893690d2cff5897678b81b23a0
Diffstat (limited to 'sw')
-rw-r--r-- | sw/source/filter/xml/xmlexpit.cxx | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/sw/source/filter/xml/xmlexpit.cxx b/sw/source/filter/xml/xmlexpit.cxx index bc58820ca82e..3753d2e84029 100644 --- a/sw/source/filter/xml/xmlexpit.cxx +++ b/sw/source/filter/xml/xmlexpit.cxx @@ -1071,33 +1071,32 @@ bool SvXMLExportItemMapper::QueryXMLValue( case RES_FRM_SIZE: { - const SwFormatFrmSize* pFrmSize = dynamic_cast<const SwFormatFrmSize*>( &rItem ); - OSL_ENSURE( pFrmSize != NULL, "Wrong Which-ID" ); + const SwFormatFrmSize& rFrmSize = dynamic_cast<const SwFormatFrmSize&>(rItem); bool bOutHeight = false; switch( nMemberId ) { case MID_FRMSIZE_REL_WIDTH: - if( pFrmSize->GetWidthPercent() ) + if (rFrmSize.GetWidthPercent()) { ::sax::Converter::convertPercent( - aOut, pFrmSize->GetWidthPercent() ); + aOut, rFrmSize.GetWidthPercent() ); bOk = true; } break; case MID_FRMSIZE_MIN_HEIGHT: - if( ATT_MIN_SIZE == pFrmSize->GetHeightSizeType() ) + if( ATT_MIN_SIZE == rFrmSize.GetHeightSizeType() ) bOutHeight = true; break; case MID_FRMSIZE_FIX_HEIGHT: - if( ATT_FIX_SIZE == pFrmSize->GetHeightSizeType() ) + if( ATT_FIX_SIZE == rFrmSize.GetHeightSizeType() ) bOutHeight = true; break; } if( bOutHeight ) { - rUnitConverter.convertMeasureToXML(aOut, pFrmSize->GetHeight()); + rUnitConverter.convertMeasureToXML(aOut, rFrmSize.GetHeight()); bOk = true; } } |