refactor IsRedlineAtEnd

should be no logic change, just added a comment as to
what is going on here and ready for extension

Change-Id: Ibf1a3893ece9a09ad342bb06f39e119c1340e0db

1 files changed, 12 insertions, 8 deletions

diff --git a/sw/source/filter/ww8/wrtw8nds.cxx b/sw/source/filter/ww8/wrtw8nds.cxx
index e000918604c1..fbd1d54ae8a6 100644
--- a/sw/source/filter/ww8/wrtw8nds.cxx
+++ b/sw/source/filter/ww8/wrtw8nds.cxx
@@ -1258,16 +1258,20 @@ bool SwWW8AttrIter::IsRedlineAtEnd( sal_Int32 nEnd ) const
     for( sal_uInt16 nPos = nCurRedlinePos;
         nPos < m_rExport.pDoc->getIDocumentRedlineAccess().GetRedlineTbl().size(); ++nPos )
     {
-        const SwPosition* pEnd = m_rExport.pDoc->getIDocumentRedlineAccess().GetRedlineTbl()[ nPos ]->End();
-        if( pEnd->nNode != rNd )
-        {
-            break;
-        }
-
-        if( pEnd->nContent.GetIndex() == nEnd )
+        const SwRangeRedline *pRange = m_rExport.pDoc->getIDocumentRedlineAccess().GetRedlineTbl()[nPos];
+        const SwPosition* pEnd = pRange->End();
+        if (pEnd->nNode == rNd)
         {
-            return true;
+            // In word the paragraph end marker is a real character, in writer it is not.
+            if (pEnd->nContent.GetIndex() == nEnd)
+            {
+                // This condition detects if the pseudo-char we will export is affected
+                // by redlining
+                return true;
+            }
         }
+        else
+            break;
     }
     return false;
 }