Huw Davies : riched20: Use more cell helpers in some of the table functions.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: b692fb5e08cc7e635e38a49c18bb5704040d8e5a
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=b692fb5e08cc7e635e38a49c18bb5704040d8e5a

Author: Huw Davies <huw at codeweavers.com>
Date:   Fri Nov  6 08:32:20 2020 +0000

riched20: Use more cell helpers in some of the table functions.

Signed-off-by: Huw Davies <huw at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/riched20/table.c | 35 +++++++++++++++++------------------
 1 file changed, 17 insertions(+), 18 deletions(-)

diff --git a/dlls/riched20/table.c b/dlls/riched20/table.c
index 9d0d24fa1c3..83b089d6dbc 100644
--- a/dlls/riched20/table.c
+++ b/dlls/riched20/table.c
@@ -96,7 +96,7 @@ ME_Paragraph* table_insert_row_start_at_para( ME_TextEditor *editor, ME_Paragrap
 
     while (para != end_para)
     {
-        para->pCell = prev_para->pCell;
+        para->pCell = cell_get_di( para_cell( prev_para ) );
         para->nFlags |= MEPF_CELL;
         para->nFlags &= ~(MEPF_ROWSTART | MEPF_ROWEND);
         para->fmt.dwMask |= PFM_TABLE | PFM_TABLEROWDELIMITER;
@@ -128,32 +128,31 @@ ME_Paragraph* table_insert_row_end( ME_TextEditor *editor, ME_Cursor *cursor )
 
 ME_Paragraph* table_row_end( ME_Paragraph *para )
 {
-  ME_DisplayItem *cell;
-  assert( para );
+  ME_Cell *cell;
+
   if (para->nFlags & MEPF_ROWEND) return para;
   if (para->nFlags & MEPF_ROWSTART) para = para_next( para );
-  cell = para->pCell;
-  assert(cell && cell->type == diCell);
-  while (cell->member.cell.next_cell)
-    cell = cell->member.cell.next_cell;
+  cell = para_cell( para );
+  while (cell_next( cell ))
+    cell = cell_next( cell );
 
-  para = &ME_FindItemFwd( cell, diParagraph )->member.para;
+  para = &ME_FindItemFwd( cell_get_di( cell ), diParagraph )->member.para;
   assert( para && para->nFlags & MEPF_ROWEND );
   return para;
 }
 
 ME_Paragraph* table_row_start( ME_Paragraph *para )
 {
-  ME_DisplayItem *cell;
-  assert( para );
+  ME_Cell *cell;
+
   if (para->nFlags & MEPF_ROWSTART) return para;
   if (para->nFlags & MEPF_ROWEND) para = para_prev( para );
-  cell = para->pCell;
-  assert(cell && cell->type == diCell);
-  while (cell->member.cell.prev_cell)
-    cell = cell->member.cell.prev_cell;
+  cell = para_cell( para );
+
+  while (cell_prev( cell ))
+    cell = cell_prev( cell );
 
-  para = &ME_FindItemBack( cell, diParagraph )->member.para;
+  para = &ME_FindItemBack( cell_get_di( cell ), diParagraph )->member.para;
   assert( para && para->nFlags & MEPF_ROWSTART );
   return para;
 }
@@ -161,11 +160,11 @@ ME_Paragraph* table_row_start( ME_Paragraph *para )
 ME_Paragraph* table_outer_para( ME_Paragraph *para )
 {
   if (para->nFlags & MEPF_ROWEND) para = para_prev( para );
-  while (para->pCell)
+  while (para_cell( para ))
   {
     para = table_row_start( para );
-    if (!para->pCell) break;
-    para = &ME_FindItemBack( para->pCell, diParagraph )->member.para;
+    if (!para_cell( para )) break;
+    para = &ME_FindItemBack( cell_get_di( para_cell( para ) ), diParagraph )->member.para;
   }
   return para;
 }