To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=56229
------- Additional comments from [EMAIL PROTECTED] Wed Oct 17 10:56:38 +0000 2007 ------- fme->lijian: Thank you very much. Almost good, I think. Just two more remarks: 1) I think the first if statement in the for loop is not correct. Imagine the case Table Row Cell 1 Section Para 1 /Section Para2 /Cell 1 Cell 2 Para 3 /Cell 1 /Row /Table Not, if cell1 is protected, lcl_FindNextCell does not correctly set the index to Para3, instead it returns a value != 0 indicating a failure. So I think the if statement should check if we have already reached the end of the (outer) table. And aTmp only has to be incremented in case that pNd is a start node. 2) The last content of a cell does not necessarily have to be a text node in the outer table, if can also be an inner table. Just try it: Select the whole content of a table cell and execute "insert table". The outer table cell will only contain a inner table cell with no surrounding text nodes. We should consider this case as well. --------------------------------------------------------------------- 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]