To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=80966





------- Additional comments from [EMAIL PROTECTED] Wed Sep  5 08:28:22 +0000 
2007 -------
If you add SwSectionNode and SwTableNode to the condition, the condition will
become false in nearly every situation. But this code has to be performed in
some situations, e.g. if you insert a table into an empty document and delete it
with Table/Delete/Table. With your change the Undo operation would crash.
The problem is nNdDiff. The value of nNdDiff at the end of the SwUndoDelete
represents the number of SwNodes which has been deleted (as a side effect) in
front of the "main" deletion. E.g. if you delete some paragraphs in the document
(the "main" deletion) and this paragraphs contains footnotes, as a side effect
the footnote content will be deleted as well. Unfortunately this footnote
content is in our SwNode array in front of the body content. The Undo has to
take care of this "shifts" in the SwNode array.
For simple deletions of text nodes nNdDiff is the difference between the start
index of the deletion before (nSttNode) and after (rPam.GetPoint) the deletion
has been performed. But if the start node and the end nodes of the deletion are
_no_ text nodes, it becomes tricky. In line 259 the watched index will be
decremented and in our problem code in line 387 this decrementation will be
corrected. In our bug case something goes wrong with this decrementation and
correction.

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to