diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-03-19 12:36:34 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-03-19 15:37:57 +0000 |
| commit | eecb2b8db8125294d85a37cbb184743b659eeaf3 (patch) | |
| tree | d2a4dcf92219f09b103ca4e5c9db998f3a98182e /lotuswordpro | |
| parent | 69255b7ca8e6c5c6166ae88fe892d64b185cc62a (diff) | |
coverity#735758 Unchecked dynamic_cast
Change-Id: I7d995f0f54034e1ab4927e1aaa3632b975b8d7fc
Diffstat (limited to 'lotuswordpro')
| -rw-r--r-- | lotuswordpro/source/filter/lwppara.cxx | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/lotuswordpro/source/filter/lwppara.cxx b/lotuswordpro/source/filter/lwppara.cxx index 1dc1ad02bcd6..7a0027860434 100644 --- a/lotuswordpro/source/filter/lwppara.cxx +++ b/lotuswordpro/source/filter/lwppara.cxx @@ -377,7 +377,7 @@ void LwpPara::RegisterStyle() pOverStyle->SetStyleName(""); LwpParaProperty* pProps = m_pProps; sal_uInt32 PropType; - LwpParaStyle* pParaStyle = dynamic_cast<LwpParaStyle*>(m_ParaStyle.obj()); + LwpParaStyle& rParaStyle = dynamic_cast<LwpParaStyle&>(*m_ParaStyle.obj()); while (pProps) { PropType = pProps->GetType(); @@ -385,12 +385,12 @@ void LwpPara::RegisterStyle() { case PP_LOCAL_ALIGN: { - if (!pParaStyle->GetAlignment()) + if (!rParaStyle.GetAlignment()) OverrideAlignment(NULL,static_cast<LwpParaAlignProperty*>(pProps)->GetAlignment(),pOverStyle); else { boost::scoped_ptr<LwpAlignmentOverride> const pAlign( - pParaStyle->GetAlignment()->clone()); + rParaStyle.GetAlignment()->clone()); OverrideAlignment(pAlign.get(), static_cast<LwpParaAlignProperty*>(pProps)->GetAlignment(), pOverStyle); @@ -400,7 +400,7 @@ void LwpPara::RegisterStyle() case PP_LOCAL_INDENT: { noIndent = sal_False; - if (!pParaStyle->GetIndent()) + if (!rParaStyle.GetIndent()) OverrideIndent(NULL,static_cast<LwpParaIndentProperty*>(pProps)->GetIndent(),pOverStyle); else @@ -412,12 +412,12 @@ void LwpPara::RegisterStyle() case PP_LOCAL_SPACING: { noSpacing = sal_False; - if (!pParaStyle->GetSpacing()) + if (!rParaStyle.GetSpacing()) OverrideSpacing(NULL,static_cast<LwpParaSpacingProperty*>(pProps)->GetSpacing(),pOverStyle); else { boost::scoped_ptr<LwpSpacingOverride> const - pSpacing(pParaStyle->GetSpacing()->clone()); + pSpacing(rParaStyle.GetSpacing()->clone()); OverrideSpacing(pSpacing.get(), static_cast<LwpParaSpacingProperty*>(pProps)->GetSpacing(), pOverStyle); @@ -450,7 +450,7 @@ void LwpPara::RegisterStyle() case PP_LOCAL_TABRACK: { //, 01/28/05 - /*LwpTabOverride* pTabOverride=pParaStyle->GetTabOverride(); + /*LwpTabOverride* pTabOverride=rParaStyle.GetTabOverride(); if(!pTabOverride) { OverrideTab(NULL,static_cast<LwpParaTabRackProperty*>(pProps)->GetTab(),pOverStyle); @@ -464,11 +464,11 @@ void LwpPara::RegisterStyle() case PP_LOCAL_BACKGROUND: { /* LwpBackgroundOverride aBackground; - if (!pParaStyle->GetBackground()) + if (!rParaStyle.GetBackground()) OverrideBackground(NULL,static_cast<LwpParaBackGroundProperty*>(pProps)->GetBackground(),pOverStyle); else { - aBackground = *(pParaStyle->GetaBackground()); + aBackground = *(rParaStyle.GetaBackground()); OverrideBackground(&aBackground,static_cast<LwpParaBackGroundProperty*>(pProps)->GetBackground(),pOverStyle); } */ |