November 2020 - Wine-Devel - list.winehq.org

[PATCH] gdi32: Fix font rendering issue when child font is used.
by Akihiro Sagawa Nov. 6, 2020

Nov. 6, 2020

Otherwise, freetype_get_glyph_outline() fails at FT_Load_Glyph due to combination of base_font's ft_face and child font's glyph index. i.e., ft_face was updated at the following point: static DWORD CDECL freetype_get_glyph_outline( struct gdi_font *incoming_gdi_font, UINT glyph, UINT format, GLYPHMETRICS *lpgm, ABC *abc, DWORD buflen, void *buf, const MAT2 *lpmat ) { : : : FT_Face ft_face = incoming_font->ft_face; : : : if(format & GGO_GLYPH_INDEX) { : : : } else { BOOL vert; get_glyph_index_linked(incoming_gdi_font, glyph, &gdi_font, &glyph_index, &vert); font = get_font_ptr( gdi_font ); ft_face = font->ft_face; // <-------- HERE if (!vert && tategaki) tategaki = check_unicode_tategaki(glyph); } : : : } Fixes a regression introduced by 044315c0b3890676d9fd751d8191b0b7381e0b97. Signed-off-by: Akihiro Sagawa <sagawa.aki(a)gmail.com> --- dlls/gdi32/freetype.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

1 0

[PATCH 5/5] riched20: Use paragraph and run ptrs in the cursor structure.
by Huw Davies Nov. 6, 2020

Nov. 6, 2020

Signed-off-by: Huw Davies <huw(a)codeweavers.com> --- dlls/riched20/caret.c | 185 +++++++++++++++---------------- dlls/riched20/clipboard.c | 2 +- dlls/riched20/editor.c | 221 ++++++++++++++++++-------------------- dlls/riched20/editstr.h | 4 +- dlls/riched20/paint.c | 10 +- dlls/riched20/para.c | 28 ++--- dlls/riched20/richole.c | 14 +-- dlls/riched20/row.c | 12 +-- dlls/riched20/run.c | 88 +++++++-------- dlls/riched20/style.c | 6 +- dlls/riched20/table.c | 82 +++++++------- dlls/riched20/undo.c | 14 +-- dlls/riched20/wrap.c | 8 +- dlls/riched20/writer.c | 64 +++++------ 14 files changed, 359 insertions(+), 379 deletions(-) diff --git a/dlls/riched20/caret.c b/dlls/riched20/caret.c index 68bf40c76de..813074599a7 100644 --- a/dlls/riched20/caret.c +++ b/dlls/riched20/caret.c @@ -26,16 +26,16 @@ WINE_DEFAULT_DEBUG_CHANNEL(richedit); void ME_SetCursorToStart(ME_TextEditor *editor, ME_Cursor *cursor) { - cursor->pPara = para_get_di( editor_first_para( editor ) ); - cursor->pRun = run_get_di( para_first_run( &cursor->pPara->member.para ) ); + cursor->para = editor_first_para( editor ); + cursor->run = para_first_run( cursor->para ); cursor->nOffset = 0; } static void ME_SetCursorToEnd(ME_TextEditor *editor, ME_Cursor *cursor, BOOL final_eop) { - cursor->pPara = para_get_di( para_prev( editor_end_para( editor ) ) ); - cursor->pRun = run_get_di( para_end_run( &cursor->pPara->member.para ) ); - cursor->nOffset = final_eop ? cursor->pRun->member.run.len : 0; + cursor->para = para_prev( editor_end_para( editor ) ); + cursor->run = para_end_run( cursor->para ); + cursor->nOffset = final_eop ? cursor->run->len : 0; } @@ -205,12 +205,12 @@ int set_selection_cursors(ME_TextEditor *editor, int from, int to) editor->pCursors[0] = editor->pCursors[1]; ME_MoveCursorChars(editor, &editor->pCursors[0], to - from, FALSE); /* Selection is not allowed in the middle of an end paragraph run. */ - if (editor->pCursors[1].pRun->member.run.nFlags & MERF_ENDPARA) + if (editor->pCursors[1].run->nFlags & MERF_ENDPARA) editor->pCursors[1].nOffset = 0; - if (editor->pCursors[0].pRun->member.run.nFlags & MERF_ENDPARA) + if (editor->pCursors[0].run->nFlags & MERF_ENDPARA) { if (to > len) - editor->pCursors[0].nOffset = editor->pCursors[0].pRun->member.run.len; + editor->pCursors[0].nOffset = editor->pCursors[0].run->len; else editor->pCursors[0].nOffset = 0; } @@ -222,8 +222,8 @@ void cursor_coords( ME_TextEditor *editor, ME_Cursor *cursor, int *x, int *y, int *height ) { ME_Row *row; - ME_Run *run = &cursor->pRun->member.run; - ME_Paragraph *para = &cursor->pPara->member.para; + ME_Run *run = cursor->run; + ME_Paragraph *para = cursor->para; ME_Run *size_run = run, *prev; ME_Context c; int run_x; @@ -302,7 +302,7 @@ BOOL ME_InternalDeleteText(ME_TextEditor *editor, ME_Cursor *start, /* Prevent deletion past last end of paragraph run. */ nChars = min(nChars, text_len - nOfs); if (nChars == text_len) delete_all = TRUE; - start_para = &c.pPara->member.para; + start_para = c.para; if (!bForce) { @@ -318,17 +318,17 @@ BOOL ME_InternalDeleteText(ME_TextEditor *editor, ME_Cursor *start, { /* We aren't deleting anything in this run, so we will go back to the * last run we are deleting text in. */ - c.pRun = run_get_di( run_prev_all_paras( &c.pRun->member.run ) ); - c.pPara = para_get_di( c.pRun->member.run.para ); - c.nOffset = c.pRun->member.run.len; + c.run = run_prev_all_paras( c.run ); + c.para = c.run->para; + c.nOffset = c.run->len; } - run = &c.pRun->member.run; + run = c.run; if (run->nFlags & MERF_ENDPARA) { - int eollen = c.pRun->member.run.len; + int eollen = c.run->len; BOOL keepFirstParaFormat; - if (!para_next( para_next( &c.pPara->member.para ) )) return TRUE; + if (!para_next( para_next( c.para ) )) return TRUE; keepFirstParaFormat = (totalChars == nChars && nChars <= eollen && run->nCharOfs); @@ -354,7 +354,7 @@ BOOL ME_InternalDeleteText(ME_TextEditor *editor, ME_Cursor *start, keepFirstParaFormat = TRUE; } } - para_join( editor, &c.pPara->member.para, keepFirstParaFormat ); + para_join( editor, c.para, keepFirstParaFormat ); /* ME_SkipAndPropagateCharOffset(p->pRun, shift); */ ME_CheckCharOffsets(editor); nChars -= (eollen < nChars) ? eollen : nChars; @@ -368,7 +368,7 @@ BOOL ME_InternalDeleteText(ME_TextEditor *editor, ME_Cursor *start, c.nOffset -= nCharsToDelete; - para_mark_rewrap( editor, c.pRun->member.run.para ); + para_mark_rewrap( editor, c.run->para ); cursor = c; /* nChars is the number of characters that should be deleted from the @@ -393,7 +393,7 @@ BOOL ME_InternalDeleteText(ME_TextEditor *editor, ME_Cursor *start, for (i=-1; i<editor->nCursors; i++) { ME_Cursor *pThisCur = editor->pCursors + i; if (i == -1) pThisCur = &c; - if (pThisCur->pRun == cursor.pRun) { + if (pThisCur->run == cursor.run) { if (pThisCur->nOffset > cursor.nOffset) { if (pThisCur->nOffset-cursor.nOffset < nCharsToDelete) pThisCur->nOffset = cursor.nOffset; @@ -404,8 +404,8 @@ BOOL ME_InternalDeleteText(ME_TextEditor *editor, ME_Cursor *start, } if (pThisCur->nOffset == run->len) { - pThisCur->pRun = run_get_di( run_next( &pThisCur->pRun->member.run ) ); - assert( pThisCur->pRun ); + pThisCur->run = run_next( pThisCur->run ); + assert( pThisCur->run ); pThisCur->nOffset = 0; } } @@ -413,22 +413,19 @@ BOOL ME_InternalDeleteText(ME_TextEditor *editor, ME_Cursor *start, /* c = updated data now */ - if (c.pRun == cursor.pRun) - ME_SkipAndPropagateCharOffset(c.pRun, shift); + if (c.run == cursor.run) + ME_SkipAndPropagateCharOffset( run_get_di( c.run ), shift ); else - ME_PropagateCharOffset(c.pRun, shift); + ME_PropagateCharOffset( run_get_di( c.run ), shift ); - if (!cursor.pRun->member.run.len) + if (!cursor.run->len) { TRACE("Removing empty run\n"); - ME_Remove(cursor.pRun); - ME_DestroyDisplayItem(cursor.pRun); + ME_Remove( run_get_di( cursor.run )); + ME_DestroyDisplayItem( run_get_di( cursor.run )); } shift = 0; - /* - ME_CheckCharOffsets(editor); - */ continue; } } @@ -590,15 +587,15 @@ void ME_InsertTextFromCursor(ME_TextEditor *editor, int nCursor, eol_len = 1; } - if (cursor->nOffset == cursor->pRun->member.run.len) + if (cursor->nOffset == cursor->run->len) { - run = run_next( &cursor->pRun->member.run ); - if (!run) run = &cursor->pRun->member.run; + run = run_next( cursor->run ); + if (!run) run = cursor->run; } else { if (cursor->nOffset) run_split( editor, cursor ); - run = &cursor->pRun->member.run; + run = cursor->run; } new_para = para_split( editor, run, style, eol_str, eol_len, 0 ); @@ -611,11 +608,11 @@ void ME_InsertTextFromCursor(ME_TextEditor *editor, int nCursor, int i; for (i = 0; i < editor->nCursors; i++) { - if (editor->pCursors[i].pRun == run_get_di( prev ) && + if (editor->pCursors[i].run == prev && editor->pCursors[i].nOffset == prev->len) { - editor->pCursors[i].pPara = para_get_di( new_para ); - editor->pCursors[i].pRun = run_get_di( run ); + editor->pCursors[i].para = new_para; + editor->pCursors[i].run = run; editor->pCursors[i].nOffset = 0; } } @@ -638,18 +635,18 @@ int ME_MoveCursorChars(ME_TextEditor *editor, ME_Cursor *cursor, int nRelOfs, BO cursor->nOffset += nRelOfs; if (cursor->nOffset < 0) { - cursor->nOffset += cursor->pRun->member.run.nCharOfs; + cursor->nOffset += cursor->run->nCharOfs; if (cursor->nOffset >= 0) { /* new offset in the same paragraph */ do { - cursor->pRun = run_get_di( run_prev( &cursor->pRun->member.run ) ); - } while (cursor->nOffset < cursor->pRun->member.run.nCharOfs); - cursor->nOffset -= cursor->pRun->member.run.nCharOfs; + cursor->run = run_prev( cursor->run ); + } while (cursor->nOffset < cursor->run->nCharOfs); + cursor->nOffset -= cursor->run->nCharOfs; return nRelOfs; } - cursor->nOffset += cursor->pPara->member.para.nCharOfs; + cursor->nOffset += cursor->para->nCharOfs; if (cursor->nOffset <= 0) { /* moved to the start of the text */ @@ -660,30 +657,29 @@ int ME_MoveCursorChars(ME_TextEditor *editor, ME_Cursor *cursor, int nRelOfs, BO /* new offset in a previous paragraph */ do { - cursor->pPara = para_get_di( para_prev( &cursor->pPara->member.para ) ); - } while (cursor->nOffset < cursor->pPara->member.para.nCharOfs); - cursor->nOffset -= cursor->pPara->member.para.nCharOfs; - - cursor->pRun = run_get_di( para_end_run( &cursor->pPara->member.para ) ); - while (cursor->nOffset < cursor->pRun->member.run.nCharOfs) { - cursor->pRun = run_get_di( run_prev( &cursor->pRun->member.run ) ); - } - cursor->nOffset -= cursor->pRun->member.run.nCharOfs; + cursor->para = para_prev( cursor->para ); + } while (cursor->nOffset < cursor->para->nCharOfs); + cursor->nOffset -= cursor->para->nCharOfs; + + cursor->run = para_end_run( cursor->para ); + while (cursor->nOffset < cursor->run->nCharOfs) + cursor->run = run_prev( cursor->run ); + cursor->nOffset -= cursor->run->nCharOfs; } - else if (cursor->nOffset >= cursor->pRun->member.run.len) + else if (cursor->nOffset >= cursor->run->len) { ME_Paragraph *next_para; int new_offset; new_offset = ME_GetCursorOfs(cursor); - next_para = para_next( &cursor->pPara->member.para ); + next_para = para_next( cursor->para ); if (new_offset < next_para->nCharOfs) { /* new offset in the same paragraph */ do { - cursor->nOffset -= cursor->pRun->member.run.len; - cursor->pRun = run_get_di( run_next( &cursor->pRun->member.run ) ); - } while (cursor->nOffset >= cursor->pRun->member.run.len); + cursor->nOffset -= cursor->run->len; + cursor->run = run_next( cursor->run ); + } while (cursor->nOffset >= cursor->run->len); return nRelOfs; } @@ -697,16 +693,16 @@ int ME_MoveCursorChars(ME_TextEditor *editor, ME_Cursor *cursor, int nRelOfs, BO /* new offset in a following paragraph */ do { - cursor->pPara = para_get_di( next_para ); + cursor->para = next_para; next_para = para_next( next_para ); } while (new_offset >= next_para->nCharOfs); - cursor->nOffset = new_offset - cursor->pPara->member.para.nCharOfs; - cursor->pRun = run_get_di( para_first_run( &cursor->pPara->member.para ) ); - while (cursor->nOffset >= cursor->pRun->member.run.len) + cursor->nOffset = new_offset - cursor->para->nCharOfs; + cursor->run = para_first_run( cursor->para ); + while (cursor->nOffset >= cursor->run->len) { - cursor->nOffset -= cursor->pRun->member.run.len; - cursor->pRun = run_get_di( run_next( &cursor->pRun->member.run ) ); + cursor->nOffset -= cursor->run->len; + cursor->run = run_next( cursor->run ); } } /* else new offset is in the same run */ return nRelOfs; @@ -716,8 +712,8 @@ int ME_MoveCursorChars(ME_TextEditor *editor, ME_Cursor *cursor, int nRelOfs, BO BOOL ME_MoveCursorWords(ME_TextEditor *editor, ME_Cursor *cursor, int nRelOfs) { - ME_Run *run = &cursor->pRun->member.run, *other_run; - ME_Paragraph *para = &cursor->pPara->member.para; + ME_Run *run = cursor->run, *other_run; + ME_Paragraph *para = cursor->para; int nOffset = cursor->nOffset; if (nRelOfs == -1) @@ -732,7 +728,7 @@ ME_MoveCursorWords(ME_TextEditor *editor, ME_Cursor *cursor, int nRelOfs) { if (ME_CallWordBreakProc( editor, get_text( other_run, 0 ), other_run->len, other_run->len - 1, WB_ISDELIMITER ) && !(run->nFlags & MERF_ENDPARA) - && !(&cursor->pRun->member.run == run && cursor->nOffset == 0) + && !(cursor->run == run && cursor->nOffset == 0) && !ME_CallWordBreakProc( editor, get_text( run, 0 ), run->len, 0, WB_ISDELIMITER )) break; run = other_run; @@ -740,7 +736,7 @@ ME_MoveCursorWords(ME_TextEditor *editor, ME_Cursor *cursor, int nRelOfs) } else { - if (&cursor->pRun->member.run == run && cursor->nOffset == 0) + if (cursor->run == run && cursor->nOffset == 0) { para = run->para; /* Skip empty start of table row paragraph */ @@ -776,21 +772,21 @@ ME_MoveCursorWords(ME_TextEditor *editor, ME_Cursor *cursor, int nRelOfs) else { para = para_next( para ); - if (para_get_di( para )->type == diTextEnd) + if (!para_next( para )) { - if (&cursor->pRun->member.run == run) return FALSE; + if (cursor->run == run) return FALSE; nOffset = 0; break; } if (para->nFlags & MEPF_ROWSTART) para = para_next( para ); - if (&cursor->pRun->member.run == run) run = para_first_run( para ); + if (cursor->run == run) run = para_first_run( para ); nOffset = 0; break; } } } - cursor->pPara = para_get_di( para ); - cursor->pRun = run_get_di( run ); + cursor->para = para; + cursor->run = run; cursor->nOffset = nOffset; return TRUE; } @@ -818,11 +814,11 @@ ME_SelectByType(ME_TextEditor *editor, ME_SelectionType selectionType) case stParagraph: editor->pCursors[1] = editor->pCursors[0]; - editor->pCursors[0].pRun = run_get_di( para_end_run( &editor->pCursors[0].pPara->member.para ) ); - editor->pCursors[0].pPara = para_get_di( editor->pCursors[0].pRun->member.run.para ); - editor->pCursors[0].nOffset = editor->pCursors[0].pRun->member.run.len; + editor->pCursors[0].run = para_end_run( editor->pCursors[0].para ); + editor->pCursors[0].para = editor->pCursors[0].run->para; + editor->pCursors[0].nOffset = editor->pCursors[0].run->len; - editor->pCursors[1].pRun = run_get_di( para_first_run( &editor->pCursors[1].pPara->member.para ) ); + editor->pCursors[1].run = para_first_run( editor->pCursors[1].para ); editor->pCursors[1].nOffset = 0; break; case stLine: @@ -847,8 +843,7 @@ ME_SelectByType(ME_TextEditor *editor, ME_SelectionType selectionType) int ME_GetCursorOfs(const ME_Cursor *cursor) { - return cursor->pPara->member.para.nCharOfs - + cursor->pRun->member.run.nCharOfs + cursor->nOffset; + return cursor->para->nCharOfs + cursor->run->nCharOfs + cursor->nOffset; } /* Helper function for cursor_from_virtual_coords() to find paragraph within tables */ @@ -910,8 +905,8 @@ static BOOL row_cursor( ME_TextEditor *editor, ME_Row *row, int x, if (x >= run->pt.x && x < run->pt.x + run->nWidth) { cursor->nOffset = ME_CharFromPoint( editor, x - run->pt.x, run, TRUE, TRUE ); - cursor->pRun = run_get_di( run ); - cursor->pPara = para_get_di( run->para ); + cursor->run = run; + cursor->para = run->para; return exact; } last = run; @@ -920,8 +915,8 @@ static BOOL row_cursor( ME_TextEditor *editor, ME_Row *row, int x, run = last; - cursor->pRun = run_get_di( run ); - cursor->pPara = para_get_di( run->para ); + cursor->run = run; + cursor->para = run->para; return FALSE; } @@ -1057,7 +1052,7 @@ static void ME_ExtendAnchorSelection(ME_TextEditor *editor) } case stParagraph: - editor->pCursors[1].pRun = run_get_di( para_first_run( &editor->pCursors[1].pPara->member.para ) ); + editor->pCursors[1].run = para_first_run( editor->pCursors[1].para ); editor->pCursors[1].nOffset = 0; break; @@ -1083,9 +1078,9 @@ static void ME_ExtendAnchorSelection(ME_TextEditor *editor) } case stParagraph: - editor->pCursors[0].pRun = run_get_di( para_end_run( &editor->pCursors[0].pPara->member.para ) ); - editor->pCursors[0].pPara = para_get_di( editor->pCursors[0].pRun->member.run.para ); - editor->pCursors[0].nOffset = editor->pCursors[0].pRun->member.run.len; + editor->pCursors[0].run = para_end_run( editor->pCursors[0].para ); + editor->pCursors[0].para = editor->pCursors[0].run->para; + editor->pCursors[0].nOffset = editor->pCursors[0].run->len; break; default: @@ -1188,7 +1183,7 @@ void ME_MouseMove(ME_TextEditor *editor, int x, int y) static int ME_GetXForArrow(ME_TextEditor *editor, ME_Cursor *pCursor) { - ME_Run *run = &pCursor->pRun->member.run; + ME_Run *run = pCursor->run; int x; if (editor->nUDArrowX != -1) @@ -1206,8 +1201,8 @@ static int ME_GetXForArrow(ME_TextEditor *editor, ME_Cursor *pCursor) static void ME_MoveCursorLines(ME_TextEditor *editor, ME_Cursor *pCursor, int nRelOfs, BOOL extend) { - ME_DisplayItem *pRun = pCursor->pRun; - ME_Paragraph *old_para = &pCursor->pPara->member.para, *new_para; + ME_DisplayItem *pRun = run_get_di( pCursor->run ); + ME_Paragraph *old_para = pCursor->para, *new_para; ME_DisplayItem *pItem; int x = ME_GetXForArrow(editor, pCursor); @@ -1277,8 +1272,6 @@ ME_MoveCursorLines(ME_TextEditor *editor, ME_Cursor *pCursor, int nRelOfs, BOOL return; } row_cursor( editor, &pItem->member.row, x, pCursor ); - assert(pCursor->pRun); - assert(pCursor->pRun->type == diRun); } static void ME_ArrowPageUp(ME_TextEditor *editor, ME_Cursor *pCursor) @@ -1293,7 +1286,7 @@ static void ME_ArrowPageUp(ME_TextEditor *editor, ME_Cursor *pCursor) * Doesn't make sense, but we try to be bug for bug compatible. */ editor->nUDArrowX = -1; } else { - ME_DisplayItem *pRun = pCursor->pRun; + ME_DisplayItem *pRun = run_get_di( pCursor->run ); ME_DisplayItem *pLast; int x, y, yd, yp; int yOldScrollPos = editor->vert_si.nPos; @@ -1328,8 +1321,6 @@ static void ME_ArrowPageUp(ME_TextEditor *editor, ME_Cursor *pCursor) row_cursor( editor, &pLast->member.row, x, pCursor ); } - assert(pCursor->pRun); - assert(pCursor->pRun->type == diRun); } static void ME_ArrowPageDown(ME_TextEditor *editor, ME_Cursor *pCursor) @@ -1348,7 +1339,7 @@ static void ME_ArrowPageDown(ME_TextEditor *editor, ME_Cursor *pCursor) { ME_SetCursorToEnd(editor, pCursor, FALSE); } else { - ME_DisplayItem *pRun = pCursor->pRun; + ME_DisplayItem *pRun = run_get_di( pCursor->run ); ME_DisplayItem *p; int yd, yp; int yOldScrollPos = editor->vert_si.nPos; @@ -1382,8 +1373,6 @@ static void ME_ArrowPageDown(ME_TextEditor *editor, ME_Cursor *pCursor) row_cursor( editor, &pLast->member.row, x, pCursor ); } - assert(pCursor->pRun); - assert(pCursor->pRun->type == diRun); } static void ME_ArrowHome( ME_TextEditor *editor, ME_Cursor *cursor ) @@ -1412,7 +1401,7 @@ static void ME_ArrowCtrlEnd(ME_TextEditor *editor, ME_Cursor *pCursor) BOOL ME_IsSelection(ME_TextEditor *editor) { - return editor->pCursors[0].pRun != editor->pCursors[1].pRun || + return editor->pCursors[0].run != editor->pCursors[1].run || editor->pCursors[0].nOffset != editor->pCursors[1].nOffset; } diff --git a/dlls/riched20/clipboard.c b/dlls/riched20/clipboard.c index f74ca2dae1a..10a814214be 100644 --- a/dlls/riched20/clipboard.c +++ b/dlls/riched20/clipboard.c @@ -348,7 +348,7 @@ static HGLOBAL get_unicode_text(ME_TextEditor *editor, const ME_Cursor *start, i int nEnd = ME_GetCursorOfs(start) + nChars; /* count paragraphs in range */ - para = &start->pPara->member.para; + para = start->para; while ((para = para_next( para )) && para->nCharOfs <= nEnd) pars++; diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index 36918c9dd79..2f7383e59a1 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -593,9 +593,9 @@ void ME_RTFParAttrHook(RTF_Info *info) /* We are just after a table row. */ RTFFlushOutputBuffer(info); cursor = info->editor->pCursors[0]; - para = &cursor.pPara->member.para; + para = cursor.para; if (para == para_next( info->tableDef->row_start ) - && !cursor.nOffset && !cursor.pRun->member.run.nCharOfs) + && !cursor.nOffset && !cursor.run->nCharOfs) { /* Since the table row end, no text has been inserted, and the \intbl * control word has not be used. We can confirm that we are not in a @@ -632,7 +632,7 @@ void ME_RTFParAttrHook(RTF_Info *info) ME_Cursor cursor; WCHAR endl = '\r'; cursor = info->editor->pCursors[0]; - if (cursor.nOffset || cursor.pRun->member.run.nCharOfs) + if (cursor.nOffset || cursor.run->nCharOfs) ME_InsertTextFromCursor(info->editor, 0, &endl, 1, info->style); tableDef->row_start = table_insert_row_start( info->editor, info->editor->pCursors ); } @@ -658,7 +658,7 @@ void ME_RTFParAttrHook(RTF_Info *info) if (tableDef->row_start && tableDef->row_start->nFlags & MEPF_ROWEND) para = para_next( tableDef->row_start ); else - para = &info->editor->pCursors[0].pPara->member.para; + para = info->editor->pCursors[0].para; tableDef->row_start = table_insert_row_start_at_para( info->editor, para ); @@ -901,7 +901,7 @@ void ME_RTFTblAttrHook(RTF_Info *info) { /* Tab stops were used to store cell positions before v4.1 but v4.1 * still seems to set the tabstops without using them. */ - PARAFORMAT2 *fmt = &info->editor->pCursors[0].pPara->member.para.fmt; + PARAFORMAT2 *fmt = &info->editor->pCursors[0].para->fmt; fmt->rgxTabs[cellNum] &= ~0x00FFFFFF; fmt->rgxTabs[cellNum] |= 0x00FFFFFF & info->rtfParam; } @@ -971,7 +971,7 @@ void ME_RTFSpecialCharHook(RTF_Info *info) } else /* v1.0 - v3.0 */ { - ME_Paragraph *para = &info->editor->pCursors[0].pPara->member.para; + ME_Paragraph *para = info->editor->pCursors[0].para; if (para_in_table( para ) && tableDef->numCellsInserted < tableDef->numCellsDefined) { @@ -1043,13 +1043,12 @@ void ME_RTFSpecialCharHook(RTF_Info *info) } run = para_first_run( cell_first_para( cell ) ); - if (&info->editor->pCursors[0].pRun->member.run != run || - info->editor->pCursors[0].nOffset) + if (info->editor->pCursors[0].run != run || info->editor->pCursors[0].nOffset) { int nOfs, nChars; /* Delete inserted cells that aren't defined. */ - info->editor->pCursors[1].pRun = run_get_di( run ); - info->editor->pCursors[1].pPara = para_get_di( run->para ); + info->editor->pCursors[1].run = run; + info->editor->pCursors[1].para = run->para; info->editor->pCursors[1].nOffset = 0; nOfs = ME_GetCursorOfs(&info->editor->pCursors[1]); nChars = ME_GetCursorOfs(&info->editor->pCursors[0]) - nOfs; @@ -1085,7 +1084,7 @@ void ME_RTFSpecialCharHook(RTF_Info *info) { WCHAR endl = '\r'; - para = &info->editor->pCursors[0].pPara->member.para; + para = info->editor->pCursors[0].para; para->fmt.dxOffset = info->tableDef->gapH; para->fmt.dxStartIndent = info->tableDef->leftEdge; @@ -1110,7 +1109,7 @@ void ME_RTFSpecialCharHook(RTF_Info *info) ME_Paragraph *para; RTFFlushOutputBuffer(info); - para = &info->editor->pCursors[0].pPara->member.para; + para = info->editor->pCursors[0].para; if (para_in_table( para )) { /* rtfPar is treated like a space within a table. */ @@ -1609,20 +1608,20 @@ static LRESULT ME_StreamIn(ME_TextEditor *editor, DWORD format, EDITSTREAM *stre /* Don't insert text at the end of the table row */ if (!editor->bEmulateVersion10) /* v4.1 */ { - ME_Paragraph *para = &editor->pCursors->pPara->member.para; + ME_Paragraph *para = editor->pCursors->para; if (para->nFlags & (MEPF_ROWSTART | MEPF_ROWEND)) { para = para_next( para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[0].nOffset = 0; } editor->pCursors[1] = editor->pCursors[0]; } else /* v1.0 - 3.0 */ { - if (editor->pCursors[0].pRun->member.run.nFlags & MERF_ENDPARA && - para_in_table( &editor->pCursors[0].pPara->member.para )) + if (editor->pCursors[0].run->nFlags & MERF_ENDPARA && + para_in_table( editor->pCursors[0].para )) return 0; } } @@ -1635,7 +1634,7 @@ static LRESULT ME_StreamIn(ME_TextEditor *editor, DWORD format, EDITSTREAM *stre ME_GetTextLength(editor), FALSE); from = to = 0; ME_ClearTempStyle(editor); - editor_set_default_para_fmt( editor, &editor->pCursors[0].pPara->member.para.fmt ); + editor_set_default_para_fmt( editor, &editor->pCursors[0].para->fmt ); } @@ -1724,8 +1723,8 @@ static LRESULT ME_StreamIn(ME_TextEditor *editor, DWORD format, EDITSTREAM *stre para = para_next( para ); } - editor->pCursors[1].pPara = para_get_di( para ); - editor->pCursors[1].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[1].para = para; + editor->pCursors[1].run = para_first_run( para ); editor->pCursors[1].nOffset = 0; nOfs = ME_GetCursorOfs(&editor->pCursors[1]); nChars = ME_GetCursorOfs(&editor->pCursors[0]) - nOfs; @@ -1921,14 +1920,14 @@ ME_FindText(ME_TextEditor *editor, DWORD flags, const CHARRANGE *chrg, const WCH if ((flags & FR_WHOLEWORD) && nMin) { cursor_from_char_ofs( editor, nMin - 1, &cursor ); - wLastChar = *get_text( &cursor.pRun->member.run, cursor.nOffset ); + wLastChar = *get_text( cursor.run, cursor.nOffset ); ME_MoveCursorChars(editor, &cursor, 1, FALSE); } else cursor_from_char_ofs( editor, nMin, &cursor ); - while (cursor.pRun && ME_GetCursorOfs(&cursor) + nLen <= nMax) + while (cursor.run && ME_GetCursorOfs(&cursor) + nLen <= nMax) { - ME_Run *run = &cursor.pRun->member.run; + ME_Run *run = cursor.run; int nCurStart = cursor.nOffset; int nMatched = 0; @@ -1962,7 +1961,7 @@ ME_FindText(ME_TextEditor *editor, DWORD flags, const CHARRANGE *chrg, const WCH break; } - cursor.nOffset += cursor.pPara->member.para.nCharOfs + cursor.pRun->member.run.nCharOfs; + cursor.nOffset += cursor.para->nCharOfs + cursor.run->nCharOfs; if (chrgText) { chrgText->cpMin = cursor.nOffset; @@ -1983,16 +1982,16 @@ ME_FindText(ME_TextEditor *editor, DWORD flags, const CHARRANGE *chrg, const WCH wLastChar = ' '; cursor.nOffset++; - if (cursor.nOffset == cursor.pRun->member.run.len) + if (cursor.nOffset == cursor.run->len) { - if (run_next_all_paras( &cursor.pRun->member.run )) + if (run_next_all_paras( cursor.run )) { - cursor.pRun = run_get_di( run_next_all_paras( &cursor.pRun->member.run ) ); - cursor.pPara = para_get_di( cursor.pRun->member.run.para ); + cursor.run = run_next_all_paras( cursor.run ); + cursor.para = cursor.run->para; cursor.nOffset = 0; } else - cursor.pRun = NULL; + cursor.run = NULL; } } } @@ -2002,15 +2001,15 @@ ME_FindText(ME_TextEditor *editor, DWORD flags, const CHARRANGE *chrg, const WCH if ((flags & FR_WHOLEWORD) && nMax < nTextLen - 1) { cursor_from_char_ofs( editor, nMax + 1, &cursor ); - wLastChar = *get_text( &cursor.pRun->member.run, cursor.nOffset ); + wLastChar = *get_text( cursor.run, cursor.nOffset ); ME_MoveCursorChars(editor, &cursor, -1, FALSE); } else cursor_from_char_ofs( editor, nMax, &cursor ); - while (cursor.pRun && ME_GetCursorOfs(&cursor) - nLen >= nMin) + while (cursor.run && ME_GetCursorOfs(&cursor) - nLen >= nMin) { - ME_Run *run = &cursor.pRun->member.run; - ME_Paragraph *para = &cursor.pPara->member.para; + ME_Run *run = cursor.run; + ME_Paragraph *para = cursor.para; int nCurEnd = cursor.nOffset; int nMatched = 0; @@ -2080,14 +2079,14 @@ ME_FindText(ME_TextEditor *editor, DWORD flags, const CHARRANGE *chrg, const WCH cursor.nOffset--; if (cursor.nOffset < 0) { - if (run_prev_all_paras( &cursor.pRun->member.run ) ) + if (run_prev_all_paras( cursor.run ) ) { - cursor.pRun = run_get_di( run_prev_all_paras( &cursor.pRun->member.run ) ); - cursor.pPara = para_get_di( cursor.pRun->member.run.para ); - cursor.nOffset = cursor.pRun->member.run.len; + cursor.run = run_prev_all_paras( cursor.run ); + cursor.para = cursor.run->para; + cursor.nOffset = cursor.run->len; } else - cursor.pRun = NULL; + cursor.run = NULL; } } } @@ -2448,14 +2447,14 @@ static void ME_UpdateSelectionLinkAttribute(ME_TextEditor *editor) ME_GetSelection(editor, &from, &to); /* Find paragraph previous to the one that contains start cursor */ - start_para = &from->pPara->member.para; + start_para = from->para; if (para_prev( start_para )) start_para = para_prev( start_para ); /* Find paragraph that contains end cursor */ - end_para = para_next( &to->pPara->member.para ); + end_para = para_next( to->para ); - start.pPara = para_get_di( start_para ); - start.pRun = run_get_di( para_first_run( start_para ) ); + start.para = start_para; + start.run = para_first_run( start_para ); start.nOffset = 0; num_chars = end_para->nCharOfs - start_para->nCharOfs; @@ -2497,7 +2496,7 @@ static BOOL handle_enter(ME_TextEditor *editor) static const WCHAR endl = '\r'; static const WCHAR endlv10[] = {'\r','\n'}; ME_Cursor cursor = editor->pCursors[0]; - ME_Paragraph *para = &cursor.pPara->member.para; + ME_Paragraph *para = cursor.para; int from, to; ME_Style *style, *eop_style; @@ -2517,33 +2516,32 @@ static BOOL handle_enter(ME_TextEditor *editor) /* Add a new table row after this row. */ para = table_append_row( editor, para ); para = para_next( para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; ME_CommitUndo(editor); ME_UpdateRepaint(editor, FALSE); return TRUE; } - else if (para == &editor->pCursors[1].pPara->member.para && - cursor.nOffset + cursor.pRun->member.run.nCharOfs == 0 && + else if (para == editor->pCursors[1].para && + cursor.nOffset + cursor.run->nCharOfs == 0 && para_prev( para ) && para_prev( para )->nFlags & MEPF_ROWSTART && !para_prev( para )->nCharOfs) { /* Insert a newline before the table. */ para = para_prev( para ); para->nFlags &= ~MEPF_ROWSTART; - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[1] = editor->pCursors[0]; - ME_InsertTextFromCursor(editor, 0, &endl, 1, - editor->pCursors[0].pRun->member.run.style); + ME_InsertTextFromCursor( editor, 0, &endl, 1, editor->pCursors[0].run->style ); para = editor_first_para( editor ); editor_set_default_para_fmt( editor, &para->fmt ); para->nFlags = 0; para_mark_rewrap( editor, para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[1] = editor->pCursors[0]; para_next( para )->nFlags |= MEPF_ROWSTART; ME_CommitCoalescingUndo(editor); @@ -2553,17 +2551,17 @@ static BOOL handle_enter(ME_TextEditor *editor) } else /* v1.0 - 3.0 */ { - ME_Paragraph *para = &cursor.pPara->member.para; + ME_Paragraph *para = cursor.para; if (para_in_table( para )) { - if (cursor.pRun->member.run.nFlags & MERF_ENDPARA) + if (cursor.run->nFlags & MERF_ENDPARA) { if (from == to) { ME_ContinueCoalescingTransaction(editor); para = table_append_row( editor, para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; ME_CommitCoalescingUndo(editor); @@ -2574,27 +2572,26 @@ static BOOL handle_enter(ME_TextEditor *editor) else { ME_ContinueCoalescingTransaction(editor); - if (cursor.pRun->member.run.nCharOfs + cursor.nOffset == 0 && + if (cursor.run->nCharOfs + cursor.nOffset == 0 && para_prev( para ) && !para_in_table( para_prev( para ) )) { /* Insert newline before table */ - cursor.pRun = run_get_di( para_end_run( para_prev( para ) ) ); - if (cursor.pRun) + cursor.run = para_end_run( para_prev( para ) ); + if (cursor.run) { - editor->pCursors[0].pRun = cursor.pRun; - editor->pCursors[0].pPara = para_get_di( para_prev( para ) ); + editor->pCursors[0].run = cursor.run; + editor->pCursors[0].para = para_prev( para ); } editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; - ME_InsertTextFromCursor(editor, 0, &endl, 1, - editor->pCursors[0].pRun->member.run.style); + ME_InsertTextFromCursor( editor, 0, &endl, 1, editor->pCursors[0].run->style ); } else { editor->pCursors[1] = editor->pCursors[0]; para = table_append_row( editor, para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; } @@ -2795,7 +2792,7 @@ static LRESULT ME_Char(ME_TextEditor *editor, WPARAM charCode, if ((unsigned)wstr >= ' ' || wstr == '\t') { ME_Cursor cursor = editor->pCursors[0]; - ME_Paragraph *para = &cursor.pPara->member.para; + ME_Paragraph *para = cursor.para; int from, to; BOOL ctrl_is_down = GetKeyState(VK_CONTROL) & 0x8000; ME_GetSelectionOfs(editor, &from, &to); @@ -2806,7 +2803,7 @@ static LRESULT ME_Char(ME_TextEditor *editor, WPARAM charCode, BOOL selected_row = FALSE; if (ME_IsSelection(editor) && - cursor.pRun->member.run.nCharOfs + cursor.nOffset == 0 && + cursor.run->nCharOfs + cursor.nOffset == 0 && to == ME_GetCursorOfs(&editor->pCursors[0]) && para_prev( para )) { para = para_prev( para ); @@ -2827,8 +2824,8 @@ static LRESULT ME_Char(ME_TextEditor *editor, WPARAM charCode, { para = para_next( para ); if (para->nFlags & MEPF_ROWSTART) para = para_next( para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; } @@ -2836,9 +2833,7 @@ static LRESULT ME_Char(ME_TextEditor *editor, WPARAM charCode, } else /* v1.0 - 3.0 */ { - if (para_in_table( para ) && - cursor.pRun->member.run.nFlags & MERF_ENDPARA && - from == to) + if (para_in_table( para ) && cursor.run->nFlags & MERF_ENDPARA && from == to) { /* Text should not be inserted at the end of the table. */ MessageBeep(-1); @@ -2983,9 +2978,8 @@ static BOOL ME_SetCursor(ME_TextEditor *editor) ME_CharFromPos(editor, pt.x, pt.y, &cursor, &isExact); if (isExact) { - ME_Run *run; + ME_Run *run = cursor.run; - run = &cursor.pRun->member.run; if (is_link( run )) { ITextHost_TxSetCursor(editor->texthost, @@ -3039,10 +3033,8 @@ static LONG ME_GetSelectionType(ME_TextEditor *editor) ME_Cursor cursor; cursor_from_char_ofs( editor, start + i, &cursor ); - if (cursor.pRun->member.run.reobj) - object_count++; - else - character_count++; + if (cursor.run->reobj) object_count++; + else character_count++; if (character_count >= 2 && object_count >= 2) return (SEL_TEXT | SEL_MULTICHAR | SEL_OBJECT | SEL_MULTIOBJECT); } @@ -3138,7 +3130,7 @@ ME_TextEditor *ME_MakeEditor(ITextHost *texthost, BOOL bEmulateVersion10) ed->nUndoMode = umAddToUndo; ed->nParagraphs = 1; ed->nLastSelStart = ed->nLastSelEnd = 0; - ed->last_sel_start_para = ed->last_sel_end_para = &ed->pCursors[0].pPara->member.para; + ed->last_sel_start_para = ed->last_sel_end_para = ed->pCursors[0].para; ed->bHideSelection = FALSE; ed->pfnWordBreak = NULL; ed->lpOleCallback = NULL; @@ -3459,7 +3451,7 @@ static void ME_LinkNotify(ME_TextEditor *editor, UINT msg, WPARAM wParam, LPARAM ME_CharFromPos(editor, x, y, &cursor, &isExact); if (!isExact) return; - if (is_link( &cursor.pRun->member.run )) + if (is_link( cursor.run )) { /* The clicked run has CFE_LINK set */ info.nmhdr.hwndFrom = NULL; info.nmhdr.idFrom = 0; @@ -3471,13 +3463,13 @@ static void ME_LinkNotify(ME_TextEditor *editor, UINT msg, WPARAM wParam, LPARAM /* find the first contiguous run with CFE_LINK set */ info.chrg.cpMin = ME_GetCursorOfs(&cursor); - run = &cursor.pRun->member.run; + run = cursor.run; while ((run = run_prev( run )) && is_link( run )) info.chrg.cpMin -= run->len; /* find the last contiguous run with CFE_LINK set */ - info.chrg.cpMax = ME_GetCursorOfs(&cursor) + cursor.pRun->member.run.len; - run = &cursor.pRun->member.run; + info.chrg.cpMax = ME_GetCursorOfs(&cursor) + cursor.run->len; + run = cursor.run; while ((run = run_next( run )) && is_link( run )) info.chrg.cpMax += run->len; @@ -4253,13 +4245,13 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, row_first_cursor( row, &start ); row_end_cursor( row, &end, TRUE ); - run = &start.pRun->member.run; + run = start.run; while (nCharsLeft) { WCHAR *str; unsigned int nCopy; - int ofs = (run == &start.pRun->member.run) ? start.nOffset : 0; - int len = (run == &end.pRun->member.run) ? end.nOffset : run->len; + int ofs = (run == start.run) ? start.nOffset : 0; + int len = (run == end.run) ? end.nOffset : run->len; str = get_text( run, ofs ); nCopy = min( nCharsLeft, len ); @@ -4271,7 +4263,7 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, nCharsLeft, NULL, NULL); dest += nCopy * (unicode ? sizeof(WCHAR) : 1); nCharsLeft -= nCopy; - if (run == &end.pRun->member.run) break; + if (run == end.run) break; run = row_next_run( row, run ); } @@ -4457,18 +4449,17 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, nCharOfs = max(nCharOfs, 0); cursor_from_char_ofs( editor, nCharOfs, &cursor ); - pt.y = cursor.pRun->member.run.pt.y; - pt.x = cursor.pRun->member.run.pt.x + - ME_PointFromChar( editor, &cursor.pRun->member.run, cursor.nOffset, TRUE ); - pt.y += cursor.pPara->member.para.pt.y + editor->rcFormat.top; + pt.y = cursor.run->pt.y; + pt.x = cursor.run->pt.x + + ME_PointFromChar( editor, cursor.run, cursor.nOffset, TRUE ); + pt.y += cursor.para->pt.y + editor->rcFormat.top; pt.x += editor->rcFormat.left; pt.x -= editor->horz_si.nPos; pt.y -= editor->vert_si.nPos; - if (wParam >= 0x40000) { - *(POINTL *)wParam = pt; - } + if (wParam >= 0x40000) *(POINTL *)wParam = pt; + return (wParam >= 0x40000) ? 0 : MAKELONG( pt.x, pt.y ); } case WM_CREATE: @@ -4910,8 +4901,8 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, /* plain text can only have the default style. */ ME_ClearTempStyle(editor); ME_AddRefStyle(editor->pBuffer->pDefaultStyle); - ME_ReleaseStyle(editor->pCursors[0].pRun->member.run.style); - editor->pCursors[0].pRun->member.run.style = editor->pBuffer->pDefaultStyle; + ME_ReleaseStyle( editor->pCursors[0].run->style ); + editor->pCursors[0].run->style = editor->pBuffer->pDefaultStyle; } } /* FIXME: Currently no support for undo level and code page options */ @@ -5165,7 +5156,7 @@ int ME_GetTextW(ME_TextEditor *editor, WCHAR *buffer, int buflen, /* bCRLF flag is only honored in 2.0 and up. 1.0 must always return text verbatim */ if (editor->bEmulateVersion10) bCRLF = FALSE; - run = &start->pRun->member.run; + run = start->run; next_run = run_next_all_paras( run ); nLen = run->len - start->nOffset; @@ -5358,8 +5349,8 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, while (nChars > 0) { - WCHAR *str = get_text( &cursor.pRun->member.run, 0 ); - int run_len = cursor.pRun->member.run.len; + WCHAR *str = get_text( cursor.run, 0 ); + int run_len = cursor.run->len; nChars -= run_len - cursor.nOffset; @@ -5373,8 +5364,8 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, { *candidate_min = cursor; candidateStarted = TRUE; - neutral_end.pPara = NULL; - space_end.pPara = NULL; + neutral_end.para = NULL; + space_end.para = NULL; cursor.nOffset++; break; } @@ -5393,9 +5384,9 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, { if (quoted && c != '\r') { - if (!space_end.pPara) + if (!space_end.para) { - if (neutral_end.pPara) + if (neutral_end.para) space_end = neutral_end; else space_end = cursor; @@ -5408,15 +5399,15 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, { if (quoted && c == '>') { - neutral_end.pPara = NULL; - space_end.pPara = NULL; + neutral_end.para = NULL; + space_end.para = NULL; goto done; } - if (!neutral_end.pPara) + if (!neutral_end.para) neutral_end = cursor; } else - neutral_end.pPara = NULL; + neutral_end.para = NULL; cursor.nOffset++; } @@ -5430,9 +5421,9 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, done: if (candidateStarted) { - if (space_end.pPara) + if (space_end.para) *candidate_max = space_end; - else if (neutral_end.pPara) + else if (neutral_end.para) *candidate_max = neutral_end; else *candidate_max = cursor; @@ -5521,7 +5512,7 @@ static BOOL ME_UpdateLinkAttribute(ME_TextEditor *editor, ME_Cursor *start, int nChars = 0; } - if (startCur.pRun != candidateStart.pRun || + if (startCur.run != candidateStart.run || startCur.nOffset != candidateStart.nOffset) { /* CFE_LINK effect should be consistently unset */ @@ -5538,15 +5529,15 @@ static BOOL ME_UpdateLinkAttribute(ME_TextEditor *editor, ME_Cursor *start, int /* Update candidateEnd since setting character formats may split * runs, which can cause a cursor to be at an invalid offset within * a split run. */ - while (candidateEnd.nOffset >= candidateEnd.pRun->member.run.len) + while (candidateEnd.nOffset >= candidateEnd.run->len) { - candidateEnd.nOffset -= candidateEnd.pRun->member.run.len; - candidateEnd.pRun = ME_FindItemFwd(candidateEnd.pRun, diRun); + candidateEnd.nOffset -= candidateEnd.run->len; + candidateEnd.run = run_next_all_paras( candidateEnd.run ); } modified = TRUE; } } - if (candidateStart.pRun != candidateEnd.pRun || + if (candidateStart.run != candidateEnd.run || candidateStart.nOffset != candidateEnd.nOffset) { /* CFE_LINK effect should be consistently set */ diff --git a/dlls/riched20/editstr.h b/dlls/riched20/editstr.h index 9c2cdd685b1..204e078c986 100644 --- a/dlls/riched20/editstr.h +++ b/dlls/riched20/editstr.h @@ -274,8 +274,8 @@ typedef struct tagME_TextBuffer typedef struct tagME_Cursor { - ME_DisplayItem *pPara; - ME_DisplayItem *pRun; + ME_Paragraph *para; + ME_Run *run; int nOffset; } ME_Cursor; diff --git a/dlls/riched20/paint.c b/dlls/riched20/paint.c index 6228c760295..ad2ffb9abb9 100644 --- a/dlls/riched20/paint.c +++ b/dlls/riched20/paint.c @@ -437,7 +437,7 @@ static void ME_DebugWrite(HDC hDC, const POINT *pt, LPCWSTR szText) { static void draw_run( ME_Context *c, int x, int y, ME_Cursor *cursor ) { ME_Row *row; - ME_Run *run = &cursor->pRun->member.run; + ME_Run *run = cursor->run; int runofs = run_char_ofs( run, cursor->nOffset ); int nSelFrom, nSelTo; @@ -999,8 +999,8 @@ static void draw_paragraph( ME_Context *c, ME_Paragraph *para ) { ME_Cursor cursor; - cursor.pRun = run_get_di( run ); - cursor.pPara = para_get_di( para ); + cursor.run = run; + cursor.para = para; cursor.nOffset = 0; draw_run( c, c->pt.x + run->pt.x, c->pt.y + para->pt.y + run->pt.y + baseline, &cursor ); } @@ -1271,9 +1271,9 @@ void ME_UpdateScrollBar(ME_TextEditor *editor) void editor_ensure_visible( ME_TextEditor *editor, ME_Cursor *cursor ) { - ME_Run *run = &cursor->pRun->member.run; + ME_Run *run = cursor->run; ME_Row *row = row_from_cursor( cursor ); - ME_Paragraph *para = &cursor->pPara->member.para; + ME_Paragraph *para = cursor->para; int x, y, yheight; diff --git a/dlls/riched20/para.c b/dlls/riched20/para.c index d01a264765a..a8168ff0daa 100644 --- a/dlls/riched20/para.c +++ b/dlls/riched20/para.c @@ -581,10 +581,10 @@ ME_Paragraph *para_split( ME_TextEditor *editor, ME_Run *run, ME_Style *style, /* Update selection cursors to point to the correct paragraph. */ for (i = 0; i < editor->nCursors; i++) { - if (editor->pCursors[i].pPara == para_get_di( old_para ) && - run->nCharOfs <= editor->pCursors[i].pRun->member.run.nCharOfs) + if (editor->pCursors[i].para == old_para && + run->nCharOfs <= editor->pCursors[i].run->nCharOfs) { - editor->pCursors[i].pPara = para_get_di( new_para ); + editor->pCursors[i].para = new_para; } } @@ -708,10 +708,10 @@ ME_Paragraph *para_join( ME_TextEditor *editor, ME_Paragraph *para, BOOL use_fir /* null char format operation to store the original char format for the ENDPARA run */ ME_InitCharFormat2W(&fmt); - startCur.pPara = para_get_di( para ); - startCur.pRun = run_get_di( end_run ); - endCur.pPara = para_get_di( next ); - endCur.pRun = run_get_di( next_first_run ); + startCur.para = para; + startCur.run = end_run; + endCur.para = next; + endCur.run = next_first_run; startCur.nOffset = endCur.nOffset = 0; ME_SetCharFormat(editor, &startCur, &endCur, &fmt); @@ -757,13 +757,13 @@ ME_Paragraph *para_join( ME_TextEditor *editor, ME_Paragraph *para, BOOL use_fir * paragraph run, and point to the correct paragraph. */ for (i = 0; i < editor->nCursors; i++) { - if (editor->pCursors[i].pRun == run_get_di( end_run )) + if (editor->pCursors[i].run == end_run) { - editor->pCursors[i].pRun = run_get_di( next_first_run ); + editor->pCursors[i].run = next_first_run; editor->pCursors[i].nOffset = 0; } - else if (editor->pCursors[i].pPara == para_get_di( next )) - editor->pCursors[i].pPara = para_get_di( para ); + else if (editor->pCursors[i].para == next) + editor->pCursors[i].para = para; } for (tmp_run = next_first_run; tmp_run; tmp_run = run_next( tmp_run )) @@ -869,8 +869,8 @@ void editor_get_selection_paras( ME_TextEditor *editor, ME_Paragraph **para, ME_ { ME_Cursor *pEndCursor = &editor->pCursors[1]; - *para = &editor->pCursors[0].pPara->member.para; - *para_end = &editor->pCursors[1].pPara->member.para; + *para = editor->pCursors[0].para; + *para_end = editor->pCursors[1].para; if (*para == *para_end) return; @@ -885,7 +885,7 @@ void editor_get_selection_paras( ME_TextEditor *editor, ME_Paragraph **para, ME_ /* The paragraph at the end of a non-empty selection isn't included * if the selection ends at the start of the paragraph. */ - if (!pEndCursor->pRun->member.run.nCharOfs && !pEndCursor->nOffset) + if (!pEndCursor->run->nCharOfs && !pEndCursor->nOffset) *para_end = para_prev( *para_end ); } diff --git a/dlls/riched20/richole.c b/dlls/riched20/richole.c index 015326c5abc..5d73731dedb 100644 --- a/dlls/riched20/richole.c +++ b/dlls/riched20/richole.c @@ -1390,20 +1390,20 @@ IRichEditOle_fnGetObject(IRichEditOle *me, LONG iob, TRACE("character offset: %d\n", lpreobject->cp); cursor_from_char_ofs( This->editor, lpreobject->cp, &cursor ); - if (!cursor.pRun->member.run.reobj) + if (!cursor.run->reobj) return E_INVALIDARG; else - reobj = cursor.pRun->member.run.reobj; + reobj = cursor.run->reobj; } else if (iob == REO_IOB_SELECTION) { ME_Cursor *from, *to; ME_GetSelection(This->editor, &from, &to); - if (!from->pRun->member.run.reobj) + if (!from->run->reobj) return E_INVALIDARG; else - reobj = from->pRun->member.run.reobj; + reobj = from->run->reobj; } else { @@ -1659,7 +1659,7 @@ static HRESULT WINAPI ITextRange_fnGetText(ITextRange *me, BSTR *str) if (!*str) return E_OUTOFMEMORY; - bEOP = (end.pRun->next->type == diTextEnd && This->end > ME_GetTextLength(editor)); + bEOP = (!para_next( para_next( end.para )) && This->end > ME_GetTextLength(editor)); ME_GetTextW(editor, *str, length, &start, length, FALSE, bEOP); return S_OK; } @@ -1714,7 +1714,7 @@ static HRESULT range_GetChar(ME_TextEditor *editor, ME_Cursor *cursor, LONG *pch { WCHAR wch[2]; - ME_GetTextW(editor, wch, 1, cursor, 1, FALSE, cursor->pRun->next->type == diTextEnd); + ME_GetTextW(editor, wch, 1, cursor, 1, FALSE, !para_next( para_next( cursor->para ) )); *pch = wch[0]; return S_OK; @@ -4745,7 +4745,7 @@ static HRESULT WINAPI ITextSelection_fnGetText(ITextSelection *me, BSTR *pbstr) if (!*pbstr) return E_OUTOFMEMORY; - bEOP = (end->pRun->next->type == diTextEnd && endOfs > ME_GetTextLength(This->reOle->editor)); + bEOP = (!para_next( para_next( end->para ) ) && endOfs > ME_GetTextLength(This->reOle->editor)); ME_GetTextW(This->reOle->editor, *pbstr, nChars, start, nChars, FALSE, bEOP); TRACE("%s\n", wine_dbgstr_w(*pbstr)); diff --git a/dlls/riched20/row.c b/dlls/riched20/row.c index 9a87cf27fb5..f5d3ad15701 100644 --- a/dlls/riched20/row.c +++ b/dlls/riched20/row.c @@ -57,7 +57,7 @@ ME_Row *row_from_cursor( ME_Cursor *cursor ) { ME_DisplayItem *item; - item = ME_FindItemBack( cursor->pRun, diStartRow ); + item = ME_FindItemBack( run_get_di( cursor->run ), diStartRow ); return &item->member.row; } @@ -66,8 +66,8 @@ void row_first_cursor( ME_Row *row, ME_Cursor *cursor ) ME_DisplayItem *item; item = ME_FindItemFwd( row_get_di( row ), diRun ); - cursor->pRun = item; - cursor->pPara = para_get_di( cursor->pRun->member.run.para ); + cursor->run = &item->member.run; + cursor->para = cursor->run->para; cursor->nOffset = 0; } @@ -77,9 +77,9 @@ void row_end_cursor( ME_Row *row, ME_Cursor *cursor, BOOL include_eop ) item = ME_FindItemFwd( row_get_di( row ), diStartRowOrParagraphOrEnd ); run = ME_FindItemBack( item, diRun ); - cursor->pRun = run; - cursor->pPara = para_get_di( cursor->pRun->member.run.para ); - cursor->nOffset = (item->type == diStartRow || include_eop) ? cursor->pRun->member.run.len : 0; + cursor->run = &run->member.run; + cursor->para = cursor->run->para; + cursor->nOffset = (item->type == diStartRow || include_eop) ? cursor->run->len : 0; } ME_Row *row_from_row_number( ME_TextEditor *editor, int row_num ) diff --git a/dlls/riched20/run.c b/dlls/riched20/run.c index 978f6691356..db0ae0a6aa0 100644 --- a/dlls/riched20/run.c +++ b/dlls/riched20/run.c @@ -29,15 +29,15 @@ WINE_DECLARE_DEBUG_CHANNEL(richedit_lists); BOOL cursor_next_run( ME_Cursor *cursor, BOOL all_para ) { - ME_DisplayItem *p = cursor->pRun->next; + ME_DisplayItem *p = run_get_di( cursor->run )->next; while (p->type != diTextEnd) { if (p->type == diParagraph && !all_para) return FALSE; else if (p->type == diRun) { - cursor->pRun = p; - cursor->pPara = para_get_di( cursor->pRun->member.run.para ); + cursor->run = &p->member.run; + cursor->para = cursor->run->para; cursor->nOffset = 0; return TRUE; } @@ -48,15 +48,15 @@ BOOL cursor_next_run( ME_Cursor *cursor, BOOL all_para ) BOOL cursor_prev_run( ME_Cursor *cursor, BOOL all_para ) { - ME_DisplayItem *p = cursor->pRun->prev; + ME_DisplayItem *p = run_get_di( cursor->run )->prev; while (p->type != diTextStart) { if (p->type == diParagraph && !all_para) return FALSE; else if (p->type == diRun) { - cursor->pRun = p; - cursor->pPara = para_get_di( cursor->pRun->member.run.para ); + cursor->run = &p->member.run; + cursor->para = cursor->run->para; cursor->nOffset = 0; return TRUE; } @@ -69,12 +69,12 @@ ME_Run *run_next( ME_Run *run ) { ME_Cursor cursor; - cursor.pRun = run_get_di( run ); - cursor.pPara = para_get_di( run->para ); + cursor.run = run; + cursor.para = run->para; cursor.nOffset = 0; if (cursor_next_run( &cursor, FALSE )) - return &cursor.pRun->member.run; + return cursor.run; return NULL; } @@ -83,12 +83,12 @@ ME_Run *run_prev( ME_Run *run ) { ME_Cursor cursor; - cursor.pRun = run_get_di( run ); - cursor.pPara = para_get_di( run->para ); + cursor.run = run; + cursor.para = run->para; cursor.nOffset = 0; if (cursor_prev_run( &cursor, FALSE )) - return &cursor.pRun->member.run; + return cursor.run; return NULL; } @@ -97,12 +97,12 @@ ME_Run *run_next_all_paras( ME_Run *run ) { ME_Cursor cursor; - cursor.pRun = run_get_di( run ); - cursor.pPara = para_get_di( run->para ); + cursor.run = run; + cursor.para = run->para; cursor.nOffset = 0; if (cursor_next_run( &cursor, TRUE )) - return &cursor.pRun->member.run; + return cursor.run; return NULL; } @@ -111,12 +111,12 @@ ME_Run *run_prev_all_paras( ME_Run *run ) { ME_Cursor cursor; - cursor.pRun = run_get_di( run ); - cursor.pPara = para_get_di( run->para ); + cursor.run = run; + cursor.para = run->para; cursor.nOffset = 0; if (cursor_prev_run( &cursor, TRUE )) - return &cursor.pRun->member.run; + return cursor.run; return NULL; } @@ -282,8 +282,8 @@ void cursor_from_char_ofs( ME_TextEditor *editor, int char_ofs, ME_Cursor *curso char_ofs -= run->nCharOfs; - cursor->pPara = para_get_di( para ); - cursor->pRun = run_get_di( run ); + cursor->para = para; + cursor->run = run; cursor->nOffset = char_ofs; } @@ -304,9 +304,9 @@ void run_join( ME_TextEditor *editor, ME_Run *run ) /* Update all cursors so that they don't contain the soon deleted run */ for (i = 0; i < editor->nCursors; i++) { - if (&editor->pCursors[i].pRun->member.run == next) + if (editor->pCursors[i].run == next) { - editor->pCursors[i].pRun = run_get_di( run ); + editor->pCursors[i].run = run; editor->pCursors[i].nOffset += run->len; } } @@ -326,7 +326,7 @@ void run_join( ME_TextEditor *editor, ME_Run *run ) */ ME_Run *run_split( ME_TextEditor *editor, ME_Cursor *cursor ) { - ME_Run *run = &cursor->pRun->member.run, *new_run; + ME_Run *run = cursor->run, *new_run; int i; int nOffset = cursor->nOffset; @@ -337,7 +337,7 @@ ME_Run *run_split( ME_TextEditor *editor, ME_Cursor *cursor ) new_run->len = run->len - nOffset; new_run->para = run->para; run->len = nOffset; - cursor->pRun = run_get_di( new_run ); + cursor->run = new_run; cursor->nOffset = 0; ME_InsertBefore( run_get_di( run )->next, run_get_di( new_run ) ); @@ -346,10 +346,10 @@ ME_Run *run_split( ME_TextEditor *editor, ME_Cursor *cursor ) ME_UpdateRunFlags( editor, new_run ); for (i = 0; i < editor->nCursors; i++) { - if (editor->pCursors[i].pRun == run_get_di( run ) && + if (editor->pCursors[i].run == run && editor->pCursors[i].nOffset >= nOffset) { - editor->pCursors[i].pRun = run_get_di( new_run ); + editor->pCursors[i].run = new_run; editor->pCursors[i].nOffset -= nOffset; } } @@ -397,19 +397,19 @@ ME_Run *run_create( ME_Style *s, int flags ) ME_Run *run_insert( ME_TextEditor *editor, ME_Cursor *cursor, ME_Style *style, const WCHAR *str, int len, int flags ) { - ME_Run *insert_before = &cursor->pRun->member.run, *run, *prev; + ME_Run *insert_before = cursor->run, *run, *prev; if (cursor->nOffset) { if (cursor->nOffset == insert_before->len) { insert_before = run_next_all_paras( insert_before ); - if (!insert_before) insert_before = &cursor->pRun->member.run; /* Always insert before the final eop run */ + if (!insert_before) insert_before = cursor->run; /* Always insert before the final eop run */ } else { run_split( editor, cursor ); - insert_before = &cursor->pRun->member.run; + insert_before = cursor->run; } } @@ -433,10 +433,10 @@ ME_Run *run_insert( ME_TextEditor *editor, ME_Cursor *cursor, ME_Style *style, for (i = 0; i < editor->nCursors; i++) { - if (editor->pCursors[i].pRun == run_get_di( prev ) && + if (editor->pCursors[i].run == prev && editor->pCursors[i].nOffset == prev->len) { - editor->pCursors[i].pRun = run_get_di( run ); + editor->pCursors[i].run = run; editor->pCursors[i].nOffset = len; } } @@ -795,35 +795,35 @@ void ME_SetSelectionCharFormat(ME_TextEditor *editor, CHARFORMAT2W *pFmt) */ void ME_SetCharFormat( ME_TextEditor *editor, ME_Cursor *start, ME_Cursor *end, CHARFORMAT2W *fmt ) { - ME_Run *run, *start_run = &start->pRun->member.run, *end_run = NULL; + ME_Run *run, *start_run = start->run, *end_run = NULL; - if (end && start->pRun == end->pRun && start->nOffset == end->nOffset) + if (end && start->run == end->run && start->nOffset == end->nOffset) return; - if (start->nOffset == start->pRun->member.run.len) - start_run = run_next_all_paras( &start->pRun->member.run ); + if (start->nOffset == start->run->len) + start_run = run_next_all_paras( start->run ); else if (start->nOffset) { /* run_split() may or may not update the cursors, depending on whether they * are selection cursors, but we need to make sure they are valid. */ int split_offset = start->nOffset; ME_Run *split_run = run_split( editor, start ); - start_run = &start->pRun->member.run; - if (end && &end->pRun->member.run == split_run) + start_run = start->run; + if (end && end->run == split_run) { - end->pRun = start->pRun; + end->run = start->run; end->nOffset -= split_offset; } } if (end) { - if (end->nOffset == end->pRun->member.run.len) - end_run = run_next_all_paras( &end->pRun->member.run ); + if (end->nOffset == end->run->len) + end_run = run_next_all_paras( end->run ); else { if (end->nOffset) run_split( editor, end ); - end_run = &end->pRun->member.run; + end_run = end->run; } } @@ -893,16 +893,16 @@ void ME_GetCharFormat( ME_TextEditor *editor, const ME_Cursor *from, ME_Run *run, *run_end, *prev_run; CHARFORMAT2W tmp; - run = &from->pRun->member.run; + run = from->run; /* special case - if selection is empty, take previous char's formatting */ - if (from->pRun == to->pRun && from->nOffset == to->nOffset) + if (from->run == to->run && from->nOffset == to->nOffset) { if (!from->nOffset && (prev_run = run_prev( run ))) run = prev_run; run_copy_char_fmt( run, fmt ); return; } - run_end = &to->pRun->member.run; + run_end = to->run; if (!to->nOffset) run_end = run_prev_all_paras( run_end ); run_copy_char_fmt( run, fmt ); diff --git a/dlls/riched20/style.c b/dlls/riched20/style.c index 95d3d3b6b29..917a5bd8100 100644 --- a/dlls/riched20/style.c +++ b/dlls/riched20/style.c @@ -482,14 +482,14 @@ ME_Style *style_get_insert_style( ME_TextEditor *editor, ME_Cursor *cursor ) if (ME_IsSelection( editor )) { ME_GetSelection( editor, &from, &to ); - style = from->pRun->member.run.style; + style = from->run->style; } else if (editor->pBuffer->pCharStyle) style = editor->pBuffer->pCharStyle; - else if (!cursor->nOffset && (prev = run_prev( &cursor->pRun->member.run ))) + else if (!cursor->nOffset && (prev = run_prev( cursor->run ))) style = prev->style; else - style = cursor->pRun->member.run.style; + style = cursor->run->style; ME_AddRefStyle( style ); return style; diff --git a/dlls/riched20/table.c b/dlls/riched20/table.c index 8d58c3e4f8f..0ce5a2c32c0 100644 --- a/dlls/riched20/table.c +++ b/dlls/riched20/table.c @@ -64,10 +64,10 @@ static ME_Paragraph* table_insert_end_para( ME_TextEditor *editor, ME_Cursor *cu if (cursor->nOffset) run_split( editor, cursor ); - para = para_split( editor, &cursor->pRun->member.run, style, eol_str, eol_len, para_flags ); + para = para_split( editor, cursor->run, style, eol_str, eol_len, para_flags ); ME_ReleaseStyle( style ); - cursor->pPara = para_get_di( para ); - cursor->pRun = run_get_di( para_first_run( para ) ); + cursor->para = para; + cursor->run = para_first_run( para ); return para; } @@ -84,13 +84,13 @@ ME_Paragraph* table_insert_row_start_at_para( ME_TextEditor *editor, ME_Paragrap ME_Paragraph *prev_para, *end_para, *start_row; ME_Cursor cursor; - cursor.pPara = para_get_di( para ); - cursor.pRun = run_get_di( para_first_run( para ) ); + cursor.para = para; + cursor.run = para_first_run( para ); cursor.nOffset = 0; start_row = table_insert_row_start( editor, &cursor ); - end_para = para_next( &editor->pCursors[0].pPara->member.para ); + end_para = para_next( editor->pCursors[0].para ); prev_para = para_next( start_row ); para = para_next( prev_para ); @@ -220,20 +220,20 @@ void table_protect_partial_deletion( ME_TextEditor *editor, ME_Cursor *c, int *n { int start_ofs = ME_GetCursorOfs( c ); ME_Cursor c2 = *c; - ME_Paragraph *this_para = &c->pPara->member.para, *end_para; + ME_Paragraph *this_para = c->para, *end_para; ME_MoveCursorChars( editor, &c2, *num_chars, FALSE ); - end_para = &c2.pPara->member.para; - if (c2.pRun->member.run.nFlags & MERF_ENDPARA) + end_para = c2.para; + if (c2.run->nFlags & MERF_ENDPARA) { /* End offset might be in the middle of the end paragraph run. * If this is the case, then we need to use the next paragraph as the last * paragraphs. */ - int remaining = start_ofs + *num_chars - c2.pRun->member.run.nCharOfs - end_para->nCharOfs; + int remaining = start_ofs + *num_chars - c2.run->nCharOfs - end_para->nCharOfs; if (remaining) { - assert( remaining < c2.pRun->member.run.len ); + assert( remaining < c2.run->len ); end_para = para_next( end_para ); } } @@ -293,11 +293,11 @@ void table_protect_partial_deletion( ME_TextEditor *editor, ME_Cursor *c, int *n if ((this_para->nCharOfs != start_ofs || this_para == end_para) && para_in_table( this_para )) { - run = &c->pRun->member.run; + run = c->run; /* Find the next tab or end paragraph to use as a delete boundary */ while (!(run->nFlags & (MERF_TAB | MERF_ENDPARA))) run = run_next( run ); - chars_to_boundary = run->nCharOfs - c->pRun->member.run.nCharOfs - c->nOffset; + chars_to_boundary = run->nCharOfs - c->run->nCharOfs - c->nOffset; *num_chars = min( *num_chars, chars_to_boundary ); } else if (para_in_table( end_para )) @@ -341,8 +341,8 @@ ME_Paragraph* table_append_row( ME_TextEditor *editor, ME_Paragraph *table_row ) prev_table_end = table_row_end( table_row ); para = para_next( prev_table_end ); run = para_first_run( para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( run ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = run; editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; new_row_start = table_insert_row_start( editor, editor->pCursors ); @@ -368,12 +368,12 @@ ME_Paragraph* table_append_row( ME_TextEditor *editor, ME_Paragraph *table_row ) { run = para_end_run( table_row ); assert( para_in_table( table_row ) ); - editor->pCursors[0].pPara = para_get_di( table_row ); - editor->pCursors[0].pRun = run_get_di( run ); + editor->pCursors[0].para = table_row; + editor->pCursors[0].run = run; editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; ME_InsertTextFromCursor( editor, 0, &endl, 1, run->style ); - run = &editor->pCursors[0].pRun->member.run; + run = editor->pCursors[0].run; for (i = 0; i < table_row->fmt.cTabCount; i++) ME_InsertTextFromCursor( editor, 0, &tab, 1, run->style ); @@ -410,8 +410,8 @@ static void table_select_next_cell_or_append( ME_TextEditor *editor, ME_Run *run para = table_append_row( editor, table_row_start( para ) ); /* Put cursor at the start of the new table row */ para = para_next( para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; ME_WrapMarkedParagraphs(editor); @@ -419,11 +419,11 @@ static void table_select_next_cell_or_append( ME_TextEditor *editor, ME_Run *run } } /* Select cell */ - editor->pCursors[1].pPara = para_get_di( cell_first_para( cell ) ); - editor->pCursors[1].pRun = run_get_di( para_first_run( &editor->pCursors[1].pPara->member.para ) ); + editor->pCursors[1].para = cell_first_para( cell ); + editor->pCursors[1].run = para_first_run( editor->pCursors[1].para ); editor->pCursors[1].nOffset = 0; - editor->pCursors[0].pPara = para_get_di( cell_end_para( cell ) ); - editor->pCursors[0].pRun = run_get_di( para_end_run( &editor->pCursors[0].pPara->member.para ) ); + editor->pCursors[0].para = cell_end_para( cell ); + editor->pCursors[0].run = para_end_run( editor->pCursors[0].para ); editor->pCursors[0].nOffset = 0; } else /* v1.0 - 3.0 */ @@ -443,8 +443,8 @@ static void table_select_next_cell_or_append( ME_TextEditor *editor, ME_Run *run if (para_in_table( para )) { run = para_first_run( para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( run ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = run; editor->pCursors[0].nOffset = 0; i = 1; } @@ -453,8 +453,8 @@ static void table_select_next_cell_or_append( ME_TextEditor *editor, ME_Run *run /* Insert table row */ para = table_append_row( editor, para_prev( para ) ); /* Put cursor at the start of the new table row */ - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; ME_WrapMarkedParagraphs(editor); @@ -464,8 +464,8 @@ static void table_select_next_cell_or_append( ME_TextEditor *editor, ME_Run *run else run = run_next( run ); } if (i == 0) run = run_next_all_paras( run ); - editor->pCursors[i].pRun = run_get_di( run ); - editor->pCursors[i].pPara = para_get_di( run->para ); + editor->pCursors[i].run = run; + editor->pCursors[i].para = run->para; editor->pCursors[i].nOffset = 0; } } @@ -494,32 +494,32 @@ void table_handle_tab( ME_TextEditor *editor, BOOL selected_row ) } if (!editor->bEmulateVersion10) /* v4.1 */ { - if (!para_in_table( &toCursor.pPara->member.para )) + if (!para_in_table( toCursor.para )) { editor->pCursors[0] = toCursor; editor->pCursors[1] = toCursor; } - else table_select_next_cell_or_append( editor, &toCursor.pRun->member.run ); + else table_select_next_cell_or_append( editor, toCursor.run ); } else /* v1.0 - 3.0 */ { - if (!para_in_table( &fromCursor.pPara->member.para) ) + if (!para_in_table( fromCursor.para )) { editor->pCursors[0] = fromCursor; editor->pCursors[1] = fromCursor; /* FIXME: For some reason the caret is shown at the start of the * previous paragraph in v1.0 to v3.0 */ } - else if ((selected_row || !para_in_table( &toCursor.pPara->member.para ))) - table_select_next_cell_or_append( editor, &fromCursor.pRun->member.run ); + else if ((selected_row || !para_in_table( toCursor.para ))) + table_select_next_cell_or_append( editor, fromCursor.run ); else { - ME_Run *run = run_prev( &toCursor.pRun->member.run ); + ME_Run *run = run_prev( toCursor.run ); if (ME_IsSelection(editor) && !toCursor.nOffset && run && run->nFlags & MERF_TAB) table_select_next_cell_or_append( editor, run ); else - table_select_next_cell_or_append( editor, &toCursor.pRun->member.run ); + table_select_next_cell_or_append( editor, toCursor.run ); } } ME_InvalidateSelection(editor); @@ -532,15 +532,15 @@ void table_handle_tab( ME_TextEditor *editor, BOOL selected_row ) * without a selection. */ void table_move_from_row_start( ME_TextEditor *editor ) { - ME_Paragraph *para = &editor->pCursors[0].pPara->member.para; + ME_Paragraph *para = editor->pCursors[0].para; - if (para == &editor->pCursors[1].pPara->member.para && para->nFlags & MEPF_ROWSTART) + if (para == editor->pCursors[1].para && para->nFlags & MEPF_ROWSTART) { /* The cursors should not be at the hidden start row paragraph without * a selection, so the cursor is moved into the first cell. */ para = para_next( para ); - editor->pCursors[0].pPara = para_get_di( para ); - editor->pCursors[0].pRun = run_get_di( para_first_run( para ) ); + editor->pCursors[0].para = para; + editor->pCursors[0].run = para_first_run( para ); editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; } diff --git a/dlls/riched20/undo.c b/dlls/riched20/undo.c index d6665bfd104..d95ce121234 100644 --- a/dlls/riched20/undo.c +++ b/dlls/riched20/undo.c @@ -336,10 +336,10 @@ static void ME_PlayUndoItem(ME_TextEditor *editor, struct undo_item *undo) { ME_Cursor tmp; cursor_from_char_ofs( editor, undo->u.set_para_fmt.pos, &tmp ); - add_undo_set_para_fmt( editor, &tmp.pPara->member.para ); - tmp.pPara->member.para.fmt = undo->u.set_para_fmt.fmt; - tmp.pPara->member.para.border = undo->u.set_para_fmt.border; - para_mark_rewrap( editor, &tmp.pPara->member.para ); + add_undo_set_para_fmt( editor, tmp.para ); + tmp.para->fmt = undo->u.set_para_fmt.fmt; + tmp.para->border = undo->u.set_para_fmt.border; + para_mark_rewrap( editor, tmp.para ); break; } case undo_set_char_fmt: @@ -371,7 +371,7 @@ static void ME_PlayUndoItem(ME_TextEditor *editor, struct undo_item *undo) { ME_Cursor tmp; cursor_from_char_ofs( editor, undo->u.join_paras.pos, &tmp ); - para_join( editor, &tmp.pPara->member.para, TRUE ); + para_join( editor, tmp.para, TRUE ); break; } case undo_split_para: @@ -383,7 +383,7 @@ static void ME_PlayUndoItem(ME_TextEditor *editor, struct undo_item *undo) cursor_from_char_ofs( editor, undo->u.split_para.pos, &tmp ); if (tmp.nOffset) run_split( editor, &tmp ); - this_para = &tmp.pPara->member.para; + this_para = tmp.para; bFixRowStart = this_para->nFlags & MEPF_ROWSTART; if (bFixRowStart) { @@ -391,7 +391,7 @@ static void ME_PlayUndoItem(ME_TextEditor *editor, struct undo_item *undo) * is correct. */ this_para->nFlags &= ~MEPF_ROWSTART; } - new_para = para_split( editor, &tmp.pRun->member.run, tmp.pRun->member.run.style, + new_para = para_split( editor, tmp.run, tmp.run->style, undo->u.split_para.eol_str->szData, undo->u.split_para.eol_str->nLen, paraFlags ); if (bFixRowStart) new_para->nFlags |= MEPF_ROWSTART; diff --git a/dlls/riched20/wrap.c b/dlls/riched20/wrap.c index 5fc20dbf83b..0ca70b8a8ad 100644 --- a/dlls/riched20/wrap.c +++ b/dlls/riched20/wrap.c @@ -131,7 +131,7 @@ static ME_Run *split_run_extents( ME_WrapContext *wc, ME_Run *run, int nVChar ) { ME_TextEditor *editor = wc->context->editor; ME_Run *run2; - ME_Cursor cursor = {para_get_di( wc->para ), run_get_di( run ), nVChar}; + ME_Cursor cursor = { wc->para, run, nVChar }; assert( run->nCharOfs != -1 ); ME_CheckCharOffsets(editor); @@ -141,7 +141,7 @@ static ME_Run *split_run_extents( ME_WrapContext *wc, ME_Run *run, int nVChar ) run_split( editor, &cursor ); - run2 = &cursor.pRun->member.run; + run2 = cursor.run; run2->script_analysis = run->script_analysis; shape_run( wc->context, run ); @@ -157,7 +157,7 @@ static ME_Run *split_run_extents( ME_WrapContext *wc, ME_Run *run, int nVChar ) debugstr_run( run ), run->pt.x, run->pt.y, debugstr_run( run2 ), run2->pt.x, run2->pt.y); - return &cursor.pRun->member.run; + return cursor.run; } /****************************************************************************** @@ -777,7 +777,7 @@ static HRESULT itemize_para( ME_Context *c, ME_Paragraph *para ) if (run->nCharOfs + run->len > items[cur_item+1].iCharPos) { - ME_Cursor cursor = {para_get_di( para ), run_get_di( run ), items[cur_item+1].iCharPos - run->nCharOfs}; + ME_Cursor cursor = { para, run, items[cur_item + 1].iCharPos - run->nCharOfs }; run_split( c->editor, &cursor ); } } diff --git a/dlls/riched20/writer.c b/dlls/riched20/writer.c index 1fc2bfad51e..49cbb0e021f 100644 --- a/dlls/riched20/writer.c +++ b/dlls/riched20/writer.c @@ -1003,7 +1003,7 @@ static BOOL ME_StreamOutRTF(ME_TextEditor *editor, ME_OutStream *pStream, if (!ME_StreamOutRTFHeader(pStream, dwFormat)) return FALSE; - if (!stream_out_font_and_colour_tbls( pStream, &cursor.pRun->member.run, &endCur.pRun->member.run )) + if (!stream_out_font_and_colour_tbls( pStream, cursor.run, endCur.run )) return FALSE; /* TODO: stylesheet table */ @@ -1021,29 +1021,29 @@ static BOOL ME_StreamOutRTF(ME_TextEditor *editor, ME_OutStream *pStream, do { - if (&cursor.pPara->member.para != prev_para) + if (cursor.para != prev_para) { - prev_para = &cursor.pPara->member.para; - if (!stream_out_para_props( editor, pStream, &cursor.pPara->member.para )) + prev_para = cursor.para; + if (!stream_out_para_props( editor, pStream, cursor.para )) return FALSE; } - if (cursor.pRun == endCur.pRun && !endCur.nOffset) + if (cursor.run == endCur.run && !endCur.nOffset) break; - TRACE("flags %xh\n", cursor.pRun->member.run.nFlags); + TRACE("flags %xh\n", cursor.run->nFlags); /* TODO: emit embedded objects */ - if (cursor.pPara->member.para.nFlags & (MEPF_ROWSTART | MEPF_ROWEND)) + if (cursor.para->nFlags & (MEPF_ROWSTART | MEPF_ROWEND)) continue; - if (cursor.pRun->member.run.nFlags & MERF_GRAPHICS) + if (cursor.run->nFlags & MERF_GRAPHICS) { - if (!stream_out_graphics(editor, pStream, &cursor.pRun->member.run)) + if (!stream_out_graphics( editor, pStream, cursor.run )) return FALSE; } - else if (cursor.pRun->member.run.nFlags & MERF_TAB) + else if (cursor.run->nFlags & MERF_TAB) { if (editor->bEmulateVersion10 && /* v1.0 - 3.0 */ - para_in_table( &cursor.pPara->member.para )) + para_in_table( cursor.para )) { if (!ME_StreamOutPrint(pStream, "\\cell ")) return FALSE; @@ -1054,7 +1054,7 @@ static BOOL ME_StreamOutRTF(ME_TextEditor *editor, ME_OutStream *pStream, return FALSE; } } - else if (cursor.pRun->member.run.nFlags & MERF_ENDCELL) + else if (cursor.run->nFlags & MERF_ENDCELL) { if (pStream->nNestingLevel > 1) { @@ -1068,13 +1068,13 @@ static BOOL ME_StreamOutRTF(ME_TextEditor *editor, ME_OutStream *pStream, } nChars--; } - else if (cursor.pRun->member.run.nFlags & MERF_ENDPARA) + else if (cursor.run->nFlags & MERF_ENDPARA) { - if (!ME_StreamOutRTFCharProps(pStream, &cursor.pRun->member.run.style->fmt)) + if (!ME_StreamOutRTFCharProps( pStream, &cursor.run->style->fmt )) return FALSE; - if (para_in_table( &cursor.pPara->member.para) && - !(cursor.pPara->member.para.nFlags & (MEPF_ROWSTART | MEPF_ROWEND | MEPF_CELL))) + if (para_in_table( cursor.para ) && + !(cursor.para->nFlags & (MEPF_ROWSTART | MEPF_ROWEND | MEPF_CELL))) { if (!ME_StreamOutPrint(pStream, "\\row\r\n")) return FALSE; @@ -1085,9 +1085,9 @@ static BOOL ME_StreamOutRTF(ME_TextEditor *editor, ME_OutStream *pStream, return FALSE; } /* Skip as many characters as required by current line break */ - nChars = max(0, nChars - cursor.pRun->member.run.len); + nChars = max(0, nChars - cursor.run->len); } - else if (cursor.pRun->member.run.nFlags & MERF_ENDROW) + else if (cursor.run->nFlags & MERF_ENDROW) { if (!ME_StreamOutPrint(pStream, "\\line\r\n")) return FALSE; @@ -1097,17 +1097,17 @@ static BOOL ME_StreamOutRTF(ME_TextEditor *editor, ME_OutStream *pStream, { int nEnd; - TRACE("style %p\n", cursor.pRun->member.run.style); - if (!ME_StreamOutRTFCharProps(pStream, &cursor.pRun->member.run.style->fmt)) + TRACE("style %p\n", cursor.run->style); + if (!ME_StreamOutRTFCharProps( pStream, &cursor.run->style->fmt )) return FALSE; - nEnd = (cursor.pRun == endCur.pRun) ? endCur.nOffset : cursor.pRun->member.run.len; - if (!ME_StreamOutRTFText(pStream, get_text( &cursor.pRun->member.run, cursor.nOffset ), + nEnd = (cursor.run == endCur.run) ? endCur.nOffset : cursor.run->len; + if (!ME_StreamOutRTFText(pStream, get_text( cursor.run, cursor.nOffset ), nEnd - cursor.nOffset)) return FALSE; cursor.nOffset = 0; } - } while (cursor.pRun != endCur.pRun && cursor_next_run( &cursor, TRUE )); + } while (cursor.run != endCur.run && cursor_next_run( &cursor, TRUE )); if (!ME_StreamOutMove(pStream, "}\0", 2)) return FALSE; @@ -1125,7 +1125,7 @@ static BOOL ME_StreamOutText(ME_TextEditor *editor, ME_OutStream *pStream, int nBufLen = 0; BOOL success = TRUE; - if (!cursor.pRun) + if (!cursor.run) return FALSE; if (dwFormat & SF_USECODEPAGE) @@ -1133,10 +1133,11 @@ static BOOL ME_StreamOutText(ME_TextEditor *editor, ME_OutStream *pStream, /* TODO: Handle SF_TEXTIZED */ - while (success && nChars && cursor.pRun) { - nLen = min(nChars, cursor.pRun->member.run.len - cursor.nOffset); + while (success && nChars && cursor.run) + { + nLen = min(nChars, cursor.run->len - cursor.nOffset); - if (!editor->bEmulateVersion10 && cursor.pRun->member.run.nFlags & MERF_ENDPARA) + if (!editor->bEmulateVersion10 && cursor.run->nFlags & MERF_ENDPARA) { static const WCHAR szEOL[] = { '\r', '\n' }; @@ -1147,18 +1148,18 @@ static BOOL ME_StreamOutText(ME_TextEditor *editor, ME_OutStream *pStream, success = ME_StreamOutMove(pStream, "\r\n", 2); } else { if (dwFormat & SF_UNICODE) - success = ME_StreamOutMove(pStream, (const char *)(get_text( &cursor.pRun->member.run, cursor.nOffset )), + success = ME_StreamOutMove(pStream, (const char *)(get_text( cursor.run, cursor.nOffset )), sizeof(WCHAR) * nLen); else { int nSize; - nSize = WideCharToMultiByte(nCodePage, 0, get_text( &cursor.pRun->member.run, cursor.nOffset ), + nSize = WideCharToMultiByte(nCodePage, 0, get_text( cursor.run, cursor.nOffset ), nLen, NULL, 0, NULL, NULL); if (nSize > nBufLen) { buffer = heap_realloc(buffer, nSize); nBufLen = nSize; } - WideCharToMultiByte(nCodePage, 0, get_text( &cursor.pRun->member.run, cursor.nOffset ), + WideCharToMultiByte(nCodePage, 0, get_text( cursor.run, cursor.nOffset ), nLen, buffer, nSize, NULL, NULL); success = ME_StreamOutMove(pStream, buffer, nSize); } @@ -1166,8 +1167,7 @@ static BOOL ME_StreamOutText(ME_TextEditor *editor, ME_OutStream *pStream, nChars -= nLen; cursor.nOffset = 0; - cursor.pRun = run_next_all_paras( &cursor.pRun->member.run ) ? - run_get_di( run_next_all_paras( &cursor.pRun->member.run ) ) : NULL; + cursor.run = run_next_all_paras( cursor.run ); } heap_free(buffer); -- 2.23.0

1 0

[PATCH 4/5] riched20: Add next / prev run from cursor helpers.
by Huw Davies Nov. 6, 2020

Nov. 6, 2020

Signed-off-by: Huw Davies <huw(a)codeweavers.com> --- dlls/riched20/editor.c | 2 +- dlls/riched20/editor.h | 5 ++- dlls/riched20/list.c | 45 ------------------------ dlls/riched20/run.c | 78 +++++++++++++++++++++++++++++++++++------- dlls/riched20/writer.c | 2 +- 5 files changed, 70 insertions(+), 62 deletions(-) diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index b27bab48738..36918c9dd79 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -5423,7 +5423,7 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, } cursor.nOffset = 0; - if (!ME_NextRun(&cursor.pPara, &cursor.pRun, TRUE)) + if (!cursor_next_run( &cursor, TRUE )) goto done; } diff --git a/dlls/riched20/editor.h b/dlls/riched20/editor.h index df8e9d2b829..d06fcef332f 100644 --- a/dlls/riched20/editor.h +++ b/dlls/riched20/editor.h @@ -67,8 +67,6 @@ void ME_CharFormatFromLogFont(HDC hDC, const LOGFONTW *lf, CHARFORMAT2W *fmt) DE /* list.c */ void ME_InsertBefore(ME_DisplayItem *diWhere, ME_DisplayItem *diWhat) DECLSPEC_HIDDEN; void ME_Remove(ME_DisplayItem *diWhere) DECLSPEC_HIDDEN; -BOOL ME_NextRun(ME_DisplayItem **para, ME_DisplayItem **run, BOOL all_para) DECLSPEC_HIDDEN; -BOOL ME_PrevRun(ME_DisplayItem **para, ME_DisplayItem **run, BOOL all_para) DECLSPEC_HIDDEN; ME_DisplayItem *ME_FindItemBack(ME_DisplayItem *di, ME_DIType nTypeOrClass) DECLSPEC_HIDDEN; ME_DisplayItem *ME_FindItemFwd(ME_DisplayItem *di, ME_DIType nTypeOrClass) DECLSPEC_HIDDEN; ME_DisplayItem *ME_FindItemBackOrHere(ME_DisplayItem *di, ME_DIType nTypeOrClass) DECLSPEC_HIDDEN; @@ -126,7 +124,8 @@ static inline ME_DisplayItem *row_get_di( ME_Row *row ) /* run.c */ void cursor_from_char_ofs( ME_TextEditor *editor, int char_ofs, ME_Cursor *cursor ) DECLSPEC_HIDDEN; - +BOOL cursor_next_run( ME_Cursor *cursor, BOOL all_para ) DECLSPEC_HIDDEN; +BOOL cursor_prev_run( ME_Cursor *cursor, BOOL all_para ) DECLSPEC_HIDDEN; int run_char_ofs( ME_Run *run, int ofs ) DECLSPEC_HIDDEN; ME_Run *run_create( ME_Style *s, int nFlags ) DECLSPEC_HIDDEN; ME_Run *run_insert( ME_TextEditor *editor, ME_Cursor *cursor, diff --git a/dlls/riched20/list.c b/dlls/riched20/list.c index 23d1a38991d..891a52e605f 100644 --- a/dlls/riched20/list.c +++ b/dlls/riched20/list.c @@ -63,51 +63,6 @@ static BOOL ME_DITypesEqual(ME_DIType type, ME_DIType nTypeOrClass) } } -/* Modifies run pointer to point to the next run. - * If all_para is FALSE constrain the search to the current para, - * otherwise modify the paragraph pointer if moving into the next paragraph. - * - * Returns TRUE if next run is found, otherwise returns FALSE. */ -BOOL ME_NextRun(ME_DisplayItem **para, ME_DisplayItem **run, BOOL all_para) -{ - ME_DisplayItem *p = (*run)->next; - while (p->type != diTextEnd) - { - if (p->type == diParagraph) { - if (!all_para) return FALSE; - *para = p; - } else if (p->type == diRun) { - *run = p; - return TRUE; - } - p = p->next; - } - return FALSE; -} - -/* Modifies run pointer to point to the previous run. - * If all_para is FALSE constrain the search to the current para, - * otherwise modify the paragraph pointer if moving into the previous paragraph. - * - * Returns TRUE if previous run is found, otherwise returns FALSE. */ -BOOL ME_PrevRun(ME_DisplayItem **para, ME_DisplayItem **run, BOOL all_para) -{ - ME_DisplayItem *p = (*run)->prev; - while (p->type != diTextStart) - { - if (p->type == diParagraph) { - if (!all_para) return FALSE; - if (para && p->member.para.prev_para->type == diParagraph) - *para = p->member.para.prev_para; - } else if (p->type == diRun) { - *run = p; - return TRUE; - } - p = p->prev; - } - return FALSE; -} - ME_DisplayItem *ME_FindItemBack(ME_DisplayItem *di, ME_DIType nTypeOrClass) { if (!di) diff --git a/dlls/riched20/run.c b/dlls/riched20/run.c index b1436ad5060..978f6691356 100644 --- a/dlls/riched20/run.c +++ b/dlls/riched20/run.c @@ -27,42 +27,96 @@ WINE_DEFAULT_DEBUG_CHANNEL(richedit); WINE_DECLARE_DEBUG_CHANNEL(richedit_check); WINE_DECLARE_DEBUG_CHANNEL(richedit_lists); +BOOL cursor_next_run( ME_Cursor *cursor, BOOL all_para ) +{ + ME_DisplayItem *p = cursor->pRun->next; + + while (p->type != diTextEnd) + { + if (p->type == diParagraph && !all_para) return FALSE; + else if (p->type == diRun) + { + cursor->pRun = p; + cursor->pPara = para_get_di( cursor->pRun->member.run.para ); + cursor->nOffset = 0; + return TRUE; + } + p = p->next; + } + return FALSE; +} + +BOOL cursor_prev_run( ME_Cursor *cursor, BOOL all_para ) +{ + ME_DisplayItem *p = cursor->pRun->prev; + + while (p->type != diTextStart) + { + if (p->type == diParagraph && !all_para) return FALSE; + else if (p->type == diRun) + { + cursor->pRun = p; + cursor->pPara = para_get_di( cursor->pRun->member.run.para ); + cursor->nOffset = 0; + return TRUE; + } + p = p->prev; + } + return FALSE; +} + ME_Run *run_next( ME_Run *run ) { - ME_DisplayItem *item = run_get_di( run ); + ME_Cursor cursor; + + cursor.pRun = run_get_di( run ); + cursor.pPara = para_get_di( run->para ); + cursor.nOffset = 0; - if (ME_NextRun( NULL, &item, FALSE )) - return &item->member.run; + if (cursor_next_run( &cursor, FALSE )) + return &cursor.pRun->member.run; return NULL; } ME_Run *run_prev( ME_Run *run ) { - ME_DisplayItem *item = run_get_di( run ); + ME_Cursor cursor; + + cursor.pRun = run_get_di( run ); + cursor.pPara = para_get_di( run->para ); + cursor.nOffset = 0; - if (ME_PrevRun( NULL, &item, FALSE )) - return &item->member.run; + if (cursor_prev_run( &cursor, FALSE )) + return &cursor.pRun->member.run; return NULL; } ME_Run *run_next_all_paras( ME_Run *run ) { - ME_DisplayItem *item = run_get_di( run ), *dummy = para_get_di( run->para ); + ME_Cursor cursor; - if (ME_NextRun( &dummy, &item, TRUE )) - return &item->member.run; + cursor.pRun = run_get_di( run ); + cursor.pPara = para_get_di( run->para ); + cursor.nOffset = 0; + + if (cursor_next_run( &cursor, TRUE )) + return &cursor.pRun->member.run; return NULL; } ME_Run *run_prev_all_paras( ME_Run *run ) { - ME_DisplayItem *item = run_get_di( run ), *dummy = para_get_di( run->para ); + ME_Cursor cursor; + + cursor.pRun = run_get_di( run ); + cursor.pPara = para_get_di( run->para ); + cursor.nOffset = 0; - if (ME_PrevRun( &dummy, &item, TRUE )) - return &item->member.run; + if (cursor_prev_run( &cursor, TRUE )) + return &cursor.pRun->member.run; return NULL; } diff --git a/dlls/riched20/writer.c b/dlls/riched20/writer.c index 72242194127..1fc2bfad51e 100644 --- a/dlls/riched20/writer.c +++ b/dlls/riched20/writer.c @@ -1107,7 +1107,7 @@ static BOOL ME_StreamOutRTF(ME_TextEditor *editor, ME_OutStream *pStream, return FALSE; cursor.nOffset = 0; } - } while (cursor.pRun != endCur.pRun && ME_NextRun(&cursor.pPara, &cursor.pRun, TRUE)); + } while (cursor.pRun != endCur.pRun && cursor_next_run( &cursor, TRUE )); if (!ME_StreamOutMove(pStream, "}\0", 2)) return FALSE; -- 2.23.0

1 0

[PATCH 3/5] riched20: Use cell ptrs to link the cells.
by Huw Davies Nov. 6, 2020

Nov. 6, 2020

Signed-off-by: Huw Davies <huw(a)codeweavers.com> --- dlls/riched20/editstr.h | 4 ++-- dlls/riched20/para.c | 31 ++++++++++++++----------------- dlls/riched20/table.c | 8 +++----- 3 files changed, 19 insertions(+), 24 deletions(-) diff --git a/dlls/riched20/editstr.h b/dlls/riched20/editstr.h index bd839afb0c0..9c2cdd685b1 100644 --- a/dlls/riched20/editstr.h +++ b/dlls/riched20/editstr.h @@ -206,7 +206,7 @@ typedef struct tagME_Paragraph PARAFORMAT2 fmt; ME_String *text; - struct tagME_DisplayItem *pCell; /* v4.1 */ + struct tagME_Cell *cell; /* v4.1 */ ME_BorderRect border; int nCharOfs; @@ -228,7 +228,7 @@ typedef struct tagME_Cell /* v4.1 */ POINT pt; int nHeight, nWidth; int yTextOffset; /* The text offset is caused by the largest top border. */ - struct tagME_DisplayItem *prev_cell, *next_cell, *parent_cell; + struct tagME_Cell *prev_cell, *next_cell, *parent_cell; } ME_Cell; typedef struct tagME_Row diff --git a/dlls/riched20/para.c b/dlls/riched20/para.c index 47640a6fd50..d01a264765a 100644 --- a/dlls/riched20/para.c +++ b/dlls/riched20/para.c @@ -126,8 +126,7 @@ BOOL para_in_table( ME_Paragraph *para ) ME_Cell *para_cell( ME_Paragraph *para ) { - if (!para->pCell) return NULL; - return &para->pCell->member.cell; + return para->cell; } ME_Row *para_first_row( ME_Paragraph *para ) @@ -625,13 +624,13 @@ ME_Paragraph *para_split( ME_TextEditor *editor, ME_Run *run, ME_Style *style, { ME_Cell *cell = cell_create(); ME_InsertBefore( para_get_di( new_para ), cell_get_di( cell ) ); - new_para->pCell = cell_get_di( cell ); + new_para->cell = cell; cell->next_cell = NULL; if (paraFlags & MEPF_ROWSTART) { old_para->nFlags |= MEPF_ROWSTART; cell->prev_cell = NULL; - cell->parent_cell = old_para->pCell; + cell->parent_cell = old_para->cell; if (para_cell( old_para )) cell->nNestingLevel = para_cell( old_para )->nNestingLevel + 1; else @@ -639,8 +638,8 @@ ME_Paragraph *para_split( ME_TextEditor *editor, ME_Run *run, ME_Style *style, } else { - cell->prev_cell = old_para->pCell; - cell_prev( cell )->next_cell = cell_get_di( cell ); + cell->prev_cell = old_para->cell; + cell_prev( cell )->next_cell = cell; assert( old_para->nFlags & MEPF_CELL ); assert( !(old_para->nFlags & MEPF_ROWSTART) ); cell->nNestingLevel = cell_prev( cell )->nNestingLevel; @@ -650,19 +649,19 @@ ME_Paragraph *para_split( ME_TextEditor *editor, ME_Run *run, ME_Style *style, else if (paraFlags & MEPF_ROWEND) { old_para->nFlags |= MEPF_ROWEND; - old_para->pCell = old_para->pCell->member.cell.parent_cell; - new_para->pCell = old_para->pCell; + old_para->cell = old_para->cell->parent_cell; + new_para->cell = old_para->cell; assert( para_prev( old_para )->nFlags & MEPF_CELL ); assert( !(para_prev( old_para )->nFlags & MEPF_ROWSTART) ); - if (new_para->pCell != para_next( new_para )->pCell - && para_next( new_para )->pCell - && !para_next( new_para )->pCell->member.cell.prev_cell) + if (new_para->cell != para_next( new_para )->cell + && para_next( new_para )->cell + && !cell_prev( para_next( new_para )->cell )) { /* Row starts just after the row that was ended. */ new_para->nFlags |= MEPF_ROWSTART; } } - else new_para->pCell = old_para->pCell; + else new_para->cell = old_para->cell; table_update_flags( old_para ); table_update_flags( new_para ); @@ -721,7 +720,7 @@ ME_Paragraph *para_join( ME_TextEditor *editor, ME_Paragraph *para, BOOL use_fir { /* Table cell/row properties are always moved over from the removed para. */ para->nFlags = next->nFlags; - para->pCell = next->pCell; + para->cell = next->cell; /* Remove cell boundary if it is between the end paragraph run and the next * paragraph display item. */ @@ -740,10 +739,8 @@ ME_Paragraph *para_join( ME_TextEditor *editor, ME_Paragraph *para, BOOL use_fir if (cell) { ME_Remove( cell_get_di( cell ) ); - if (cell_prev( cell )) - cell_prev( cell )->next_cell = cell_next( cell ) ? cell_get_di( cell_next( cell ) ) : NULL; - if (cell_next( cell )) - cell_next( cell )->prev_cell = cell_prev( cell ) ? cell_get_di( cell_prev( cell ) ) : NULL; + if (cell_prev( cell )) cell_prev( cell )->next_cell = cell_next( cell ); + if (cell_next( cell )) cell_next( cell )->prev_cell = cell_prev( cell ); ME_DestroyDisplayItem( cell_get_di( cell ) ); } diff --git a/dlls/riched20/table.c b/dlls/riched20/table.c index 08f04e06ca7..8d58c3e4f8f 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 = cell_get_di( para_cell( prev_para ) ); + para->cell = para_cell( prev_para ); para->nFlags |= MEPF_CELL; para->nFlags &= ~(MEPF_ROWSTART | MEPF_ROWEND); para->fmt.dwMask |= PFM_TABLE | PFM_TABLEROWDELIMITER; @@ -193,14 +193,12 @@ ME_Cell *cell_create( void ) ME_Cell *cell_next( ME_Cell *cell ) { - if (!cell->next_cell) return NULL; - return &cell->next_cell->member.cell; + return cell->next_cell; } ME_Cell *cell_prev( ME_Cell *cell ) { - if (!cell->prev_cell) return NULL; - return &cell->prev_cell->member.cell; + return cell->prev_cell; } ME_Paragraph *cell_first_para( ME_Cell *cell ) -- 2.23.0

1 0

[PATCH 2/5] riched20: Use cell ptrs in the paragraph splitting and joining functions.
by Huw Davies Nov. 6, 2020

Nov. 6, 2020

Signed-off-by: Huw Davies <huw(a)codeweavers.com> --- dlls/riched20/editor.h | 1 + dlls/riched20/para.c | 60 +++++++++++++++++++++--------------------- dlls/riched20/table.c | 6 +++++ 3 files changed, 37 insertions(+), 30 deletions(-) diff --git a/dlls/riched20/editor.h b/dlls/riched20/editor.h index 24e750be1de..df8e9d2b829 100644 --- a/dlls/riched20/editor.h +++ b/dlls/riched20/editor.h @@ -289,6 +289,7 @@ ME_Paragraph *editor_end_para( ME_TextEditor *editor ) DECLSPEC_HIDDEN; ME_Paragraph *editor_first_para( ME_TextEditor *editor ) DECLSPEC_HIDDEN; /* table.c */ +ME_Cell *cell_create( void ) DECLSPEC_HIDDEN; ME_Paragraph *cell_end_para( ME_Cell *cell ) DECLSPEC_HIDDEN; ME_Paragraph *cell_first_para( ME_Cell *cell ) DECLSPEC_HIDDEN; ME_Cell *cell_next( ME_Cell *cell ) DECLSPEC_HIDDEN; diff --git a/dlls/riched20/para.c b/dlls/riched20/para.c index e7c3c6934b9..47640a6fd50 100644 --- a/dlls/riched20/para.c +++ b/dlls/riched20/para.c @@ -243,7 +243,7 @@ static void table_update_flags( ME_Paragraph *para ) { para->fmt.dwMask |= PFM_TABLE | PFM_TABLEROWDELIMITER; - if (para->pCell) para->nFlags |= MEPF_CELL; + if (para_cell( para )) para->nFlags |= MEPF_CELL; else para->nFlags &= ~MEPF_CELL; if (para->nFlags & MEPF_ROWEND) para->fmt.wEffects |= PFE_TABLEROWDELIMITER; @@ -623,29 +623,28 @@ ME_Paragraph *para_split( ME_TextEditor *editor, ME_Run *run, ME_Style *style, { if (paraFlags & (MEPF_ROWSTART | MEPF_CELL)) { - ME_DisplayItem *cell = ME_MakeDI(diCell); - ME_InsertBefore( para_get_di( new_para ), cell ); - new_para->pCell = cell; - cell->member.cell.next_cell = NULL; + ME_Cell *cell = cell_create(); + ME_InsertBefore( para_get_di( new_para ), cell_get_di( cell ) ); + new_para->pCell = cell_get_di( cell ); + cell->next_cell = NULL; if (paraFlags & MEPF_ROWSTART) { old_para->nFlags |= MEPF_ROWSTART; - cell->member.cell.prev_cell = NULL; - cell->member.cell.parent_cell = old_para->pCell; - if (old_para->pCell) - cell->member.cell.nNestingLevel = old_para->pCell->member.cell.nNestingLevel + 1; + cell->prev_cell = NULL; + cell->parent_cell = old_para->pCell; + if (para_cell( old_para )) + cell->nNestingLevel = para_cell( old_para )->nNestingLevel + 1; else - cell->member.cell.nNestingLevel = 1; + cell->nNestingLevel = 1; } else { - cell->member.cell.prev_cell = old_para->pCell; - assert(cell->member.cell.prev_cell); - cell->member.cell.prev_cell->member.cell.next_cell = cell; + cell->prev_cell = old_para->pCell; + cell_prev( cell )->next_cell = cell_get_di( cell ); assert( old_para->nFlags & MEPF_CELL ); assert( !(old_para->nFlags & MEPF_ROWSTART) ); - cell->member.cell.nNestingLevel = cell->member.cell.prev_cell->member.cell.nNestingLevel; - cell->member.cell.parent_cell = cell->member.cell.prev_cell->member.cell.parent_cell; + cell->nNestingLevel = cell_prev( cell )->nNestingLevel; + cell->parent_cell = cell_prev( cell )->parent_cell; } } else if (paraFlags & MEPF_ROWEND) @@ -653,11 +652,11 @@ ME_Paragraph *para_split( ME_TextEditor *editor, ME_Run *run, ME_Style *style, old_para->nFlags |= MEPF_ROWEND; old_para->pCell = old_para->pCell->member.cell.parent_cell; new_para->pCell = old_para->pCell; - assert( old_para->prev_para->member.para.nFlags & MEPF_CELL ); - assert( !(old_para->prev_para->member.para.nFlags & MEPF_ROWSTART) ); - if (new_para->pCell != new_para->next_para->member.para.pCell - && new_para->next_para->member.para.pCell - && !new_para->next_para->member.para.pCell->member.cell.prev_cell) + assert( para_prev( old_para )->nFlags & MEPF_CELL ); + assert( !(para_prev( old_para )->nFlags & MEPF_ROWSTART) ); + if (new_para->pCell != para_next( new_para )->pCell + && para_next( new_para )->pCell + && !para_next( new_para )->pCell->member.cell.prev_cell) { /* Row starts just after the row that was ended. */ new_para->nFlags |= MEPF_ROWSTART; @@ -686,9 +685,10 @@ ME_Paragraph *para_split( ME_TextEditor *editor, ME_Run *run, ME_Style *style, specified in use_first_fmt */ ME_Paragraph *para_join( ME_TextEditor *editor, ME_Paragraph *para, BOOL use_first_fmt ) { - ME_DisplayItem *tmp, *pCell = NULL; + ME_DisplayItem *tmp; ME_Paragraph *next = para_next( para ); ME_Run *end_run, *next_first_run, *tmp_run; + ME_Cell *cell = NULL; int i, shift; int end_len; CHARFORMAT2W fmt; @@ -729,22 +729,22 @@ ME_Paragraph *para_join( ME_TextEditor *editor, ME_Paragraph *para, BOOL use_fir { if (tmp->type == diCell) { - pCell = tmp; + cell = &tmp->member.cell; break; } } } - add_undo_split_para( editor, next, eol_str, pCell ? &pCell->member.cell : NULL ); + add_undo_split_para( editor, next, eol_str, cell ); - if (pCell) + if (cell) { - ME_Remove( pCell ); - if (pCell->member.cell.prev_cell) - pCell->member.cell.prev_cell->member.cell.next_cell = pCell->member.cell.next_cell; - if (pCell->member.cell.next_cell) - pCell->member.cell.next_cell->member.cell.prev_cell = pCell->member.cell.prev_cell; - ME_DestroyDisplayItem( pCell ); + ME_Remove( cell_get_di( cell ) ); + if (cell_prev( cell )) + cell_prev( cell )->next_cell = cell_next( cell ) ? cell_get_di( cell_next( cell ) ) : NULL; + if (cell_next( cell )) + cell_next( cell )->prev_cell = cell_prev( cell ) ? cell_get_di( cell_prev( cell ) ) : NULL; + ME_DestroyDisplayItem( cell_get_di( cell ) ); } if (!use_first_fmt) diff --git a/dlls/riched20/table.c b/dlls/riched20/table.c index 83b089d6dbc..08f04e06ca7 100644 --- a/dlls/riched20/table.c +++ b/dlls/riched20/table.c @@ -185,6 +185,12 @@ ME_Cell *table_row_end_cell( ME_Paragraph *para ) return cell_next( para_cell( para ) ); } +ME_Cell *cell_create( void ) +{ + ME_DisplayItem *item = ME_MakeDI( diCell ); + return &item->member.cell; +} + ME_Cell *cell_next( ME_Cell *cell ) { if (!cell->next_cell) return NULL; -- 2.23.0

1 0

[PATCH 1/5] riched20: Use more cell helpers in some of the table functions.
by Huw Davies Nov. 6, 2020

Nov. 6, 2020

Signed-off-by: Huw Davies <huw(a)codeweavers.com> --- 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; } -- 2.23.0

1 0

[PATCH] wininet: Fix error handling INTERNET_OPTION_PROXY in InternetSetOptionA.
by Jiangyi Chen Nov. 6, 2020

Nov. 6, 2020

1 0

[PATCH] iphlpapi: Update a comment.
by Haoyang Chen Nov. 6, 2020

Nov. 6, 2020

1 0

[PATCH] wine.inf: Update Pacific SA Time Zone.
by Daniel Lehman Nov. 6, 2020

Nov. 6, 2020

Signed-off-by: Daniel Lehman <dlehman25(a)gmail.com> --- loader/wine.inf.in | 23 +++++++++++++++++++---- 1 file changed, 19 insertions(+), 4 deletions(-) diff --git a/loader/wine.inf.in b/loader/wine.inf.in index 2699a982acb..b742b2df410 100644 --- a/loader/wine.inf.in +++ b/loader/wine.inf.in @@ -3463,10 +3463,25 @@ HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time,"Dlt",,"Pacific SA D HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time,"MUI_Dlt",,"@tzres.dll,-65009" HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time,"MUI_Std",,"@tzres.dll,-65008" HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time,"Std",,"Pacific SA Standard Time" -HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time,"TZI",1,b4,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00 -HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2015",1,b4,00,00,00,00,00,00,00,c4,ff,ff,ff,df,07,04,00,00,00,1a,00,01,00,00,00,00,00,00,00,e0,07,01,00,05,00,01,00,00,00,00,00,00,00,00,00 -HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"FirstEntry",0x10001,2015 -HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"LastEntry",0x10001,2015 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time,"TZI",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,04,00,00,00,01,00,00,00,00,00,00,00,00,00,00,00,09,00,00,00,01,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2005",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,03,00,00,00,02,00,00,00,00,00,00,00,00,00,00,00,0a,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2006",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,03,00,00,00,02,00,00,00,00,00,00,00,00,00,00,00,0a,00,00,00,03,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2007",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,03,00,00,00,02,00,00,00,00,00,00,00,00,00,00,00,0a,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2008",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,03,00,00,00,05,00,00,00,00,00,00,00,00,00,00,00,0a,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2009",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,03,00,00,00,03,00,00,00,00,00,00,00,00,00,00,00,0a,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2010",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,04,00,00,00,01,00,00,00,00,00,00,00,00,00,00,00,0a,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2011",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,05,00,00,00,02,00,00,00,00,00,00,00,00,00,00,00,08,00,00,00,03,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2012",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,04,00,00,00,05,00,00,00,00,00,00,00,00,00,00,00,09,00,00,00,01,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2013",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,04,00,00,00,05,00,00,00,00,00,00,00,00,00,00,00,09,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2014",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,04,00,00,00,05,00,00,00,00,00,00,00,00,00,00,00,09,00,00,00,01,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2015",1,b4,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2016",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,05,00,00,00,03,00,00,00,00,00,00,00,00,00,00,00,08,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2017",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,05,00,00,00,02,00,00,00,00,00,00,00,00,00,00,00,08,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2018",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,05,00,00,00,02,00,00,00,00,00,00,00,00,00,00,00,08,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2019",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,04,00,00,00,01,00,00,00,00,00,00,00,00,00,00,00,09,00,00,00,02,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"2020",1,f0,00,00,00,00,00,00,00,c4,ff,ff,ff,00,00,04,00,00,00,01,00,00,00,00,00,00,00,00,00,00,00,09,00,00,00,01,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"FirstEntry",0x10001,"2005" +HKLM,%CurrentVersionNT%\Time Zones\Pacific SA Standard Time\Dynamic DST,"LastEntry",0x10001,"2020" HKLM,%CurrentVersionNT%\Time Zones\Pacific Standard Time,"Display",,"America/Los_Angeles" HKLM,%CurrentVersionNT%\Time Zones\Pacific Standard Time,"Dlt",,"Pacific Daylight Time" HKLM,%CurrentVersionNT%\Time Zones\Pacific Standard Time,"MUI_Dlt",,"@tzres.dll,-11233" -- 2.25.1

1 0

[PATCH 1/2] wine.inf: Update Azerbaijan Time Zone.
by Daniel Lehman Nov. 6, 2020

Nov. 6, 2020

Signed-off-by: Daniel Lehman <dlehman25(a)gmail.com> --- loader/wine.inf.in | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/loader/wine.inf.in b/loader/wine.inf.in index ee816a5cdef..88fadca2342 100644 --- a/loader/wine.inf.in +++ b/loader/wine.inf.in @@ -2834,7 +2834,11 @@ HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time,"Dlt",,"Azerbaijan D HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time,"MUI_Dlt",,"@tzres.dll,-11313" HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time,"MUI_Std",,"@tzres.dll,-11312" HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time,"Std",,"Azerbaijan Standard Time" -HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time,"TZI",1,10,ff,ff,ff,00,00,00,00,c4,ff,ff,ff,00,00,0a,00,00,00,05,00,05,00,00,00,00,00,00,00,00,00,03,00,00,00,05,00,04,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time,"TZI",1,10,ff,ff,ff,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time\Dynamic DST,"2015",1,10,ff,ff,ff,00,00,00,00,c4,ff,ff,ff,00,00,0a,00,00,00,05,00,05,00,00,00,00,00,00,00,00,00,03,00,00,00,05,00,04,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time\Dynamic DST,"2016",1,10,ff,ff,ff,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time\Dynamic DST,"FirstEntry",0x10001,"2015" +HKLM,%CurrentVersionNT%\Time Zones\Azerbaijan Standard Time\Dynamic DST,"LastEntry",0x10001,"2016" HKLM,%CurrentVersionNT%\Time Zones\Azores Standard Time,"Display",,"Atlantic/Azores" HKLM,%CurrentVersionNT%\Time Zones\Azores Standard Time,"Dlt",,"Azores Daylight Time" HKLM,%CurrentVersionNT%\Time Zones\Azores Standard Time,"MUI_Dlt",,"@tzres.dll,-18833" -- 2.25.1

1 1