9831640
From fdb1d62a09f7320ee5c2828aa4ce84248a6e3e4e Mon Sep 17 00:00:00 2001
9831640
From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= <caolanm@redhat.com>
9831640
Date: Thu, 17 Jul 2014 13:48:30 +0100
9831640
Subject: [PATCH] Related: rhbz#1117853 nStartPara of EE_PARA_NOT_FOUND
9831640
9831640
we know that nStartPara is EE_PARA_NOT_FOUND so rOutliner.GetAbsPos must have
9831640
returned that, but we don't know the circumstances that lead to that yet.
9831640
9831640
Change-Id: I53a488317d154b4a3c050248b8737da0a611ca43
9831640
---
9831640
 editeng/source/outliner/outliner.cxx | 3 +--
9831640
 sd/source/ui/view/outlnvsh.cxx       | 5 ++---
9831640
 2 files changed, 3 insertions(+), 5 deletions(-)
9831640
9831640
diff --git a/editeng/source/outliner/outliner.cxx b/editeng/source/outliner/outliner.cxx
9831640
index 4d7214b..aaa843d 100644
9831640
--- a/editeng/source/outliner/outliner.cxx
9831640
+++ b/editeng/source/outliner/outliner.cxx
9831640
@@ -384,7 +384,6 @@ sal_Int32 Outliner::GetBulletsNumberingStatus() const
9831640
 
9831640
 OutlinerParaObject* Outliner::CreateParaObject( sal_Int32 nStartPara, sal_Int32 nCount ) const
9831640
 {
9831640
-
9831640
     if ( static_cast<sal_uLong>(nStartPara) + nCount >
9831640
             static_cast<sal_uLong>(pParaList->GetParagraphCount()) )
9831640
         nCount = pParaList->GetParagraphCount() - nStartPara;
9831640
@@ -394,7 +393,7 @@ OutlinerParaObject* Outliner::CreateParaObject( sal_Int32 nStartPara, sal_Int32
9831640
     if ( ( nStartPara + nCount ) > pEditEngine->GetParagraphCount() )
9831640
         nCount = pEditEngine->GetParagraphCount() - nStartPara;
9831640
 
9831640
-    if( !nCount )
9831640
+    if (nCount <= 0)
9831640
         return NULL;
9831640
 
9831640
     EditTextObject* pText = pEditEngine->CreateTextObject( nStartPara, nCount );
9831640
diff --git a/sd/source/ui/view/outlnvsh.cxx b/sd/source/ui/view/outlnvsh.cxx
9831640
index 03e899d..e3b4aa7 100644
9831640
--- a/sd/source/ui/view/outlnvsh.cxx
9831640
+++ b/sd/source/ui/view/outlnvsh.cxx
9831640
@@ -1658,7 +1658,6 @@ bool OutlineViewShell::UpdateTitleObject( SdPage* pPage, Paragraph* pPara )
9831640
 
611aa43
     ::Outliner*             pOutliner = pOlView->GetOutliner();
9831640
     SdrTextObj*         pTO  = pOlView->GetTitleTextObject( pPage );
9831640
-    OutlinerParaObject* pOPO = NULL;
9831640
 
611aa43
     OUString aTest = pOutliner->GetText(pPara);
9831640
     bool    bText = !aTest.isEmpty();
9831640
@@ -1675,9 +1674,9 @@ bool OutlineViewShell::UpdateTitleObject( SdPage* pPage, Paragraph* pPara )
9831640
         }
9831640
 
9831640
         // if we have a title object and a text, set the text
9831640
-        if( pTO )
0cecb79
+        OutlinerParaObject* pOPO = pTO ? pOutliner->CreateParaObject(pOutliner->GetAbsPos(pPara), 1) : NULL;
9831640
+        if (pOPO)
9831640
         {
611aa43
-            pOPO = pOutliner->CreateParaObject( pOutliner->GetAbsPos( pPara ), 1 );
9831640
             pOPO->SetOutlinerMode( OUTLINERMODE_TITLEOBJECT );
9831640
             pOPO->SetVertical( pTO->IsVerticalWriting() );
9831640
             if( pTO->GetOutlinerParaObject() && (pOPO->GetTextObject() == pTO->GetOutlinerParaObject()->GetTextObject()) )
9831640
-- 
9831640
1.9.3
9831640