richedit: Send EN_SELCHANGE after wrapping but before EN_CHANGE.

The call to ME_SendSelChange cannot be done in ME_CommitUndo like it was before, because it must be done after wrapping. Native richedit controls send EN_CHANGE first, which is sent in ME_UpdateRepaint, so ME_SendSelChange must be before this notification in ME_UpdateRepaint.

richedit: Send EN_SELCHANGE after wrapping but before EN_CHANGE.
dcbc22b9 · Dylan Smith · Alexandre Julliard · d312d4e3 · dcbc22b9 · dcbc22b9
Commit dcbc22b9 authored May 19, 2010 by Dylan Smith Committed by Alexandre Julliard May 19, 2010
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

paint.c dlls/riched20/paint.c +2 -1

undo.c dlls/riched20/undo.c +0 -2

No files found.
--- a/dlls/riched20/paint.c
+++ b/dlls/riched20/paint.c
@@ -146,6 +146,8 @@ void ME_UpdateRepaint(ME_TextEditor *editor)
  /* Ensure that the cursor is visible */
  ME_EnsureVisible(editor, &editor->pCursors[0]);

+  ME_SendSelChange(editor);
+
  /* send EN_CHANGE if the event mask asks for it */
  if(editor->nEventMask & ENM_CHANGE)
  {
@@ -154,7 +156,6 @@ void ME_UpdateRepaint(ME_TextEditor *editor)
    editor->nEventMask |= ENM_CHANGE;
  }
  ME_Repaint(editor);
-  ME_SendSelChange(editor);
 }

 void

--- a/dlls/riched20/undo.c
+++ b/dlls/riched20/undo.c
@@ -203,7 +203,6 @@ void ME_CommitUndo(ME_TextEditor *editor) {
  }

  ME_AddUndoItem(editor, diUndoEndTransaction, NULL);
-  ME_SendSelChange(editor);
 }

 /**
@@ -269,7 +268,6 @@ void ME_CommitCoalescingUndo(ME_TextEditor *editor)
    return;

  ME_AddUndoItem(editor, diUndoPotentialEndTransaction, NULL);
-  ME_SendSelChange(editor);
 }

 static void ME_PlayUndoItem(ME_TextEditor *editor, ME_DisplayItem *pItem)