commit 333365369674b7f1f8d00be4d0c6686c235100a0
author Huw Davies <huw@codeweavers.com> Tue Dec 10 11:19:08 2013 +0000
committer Alexandre Julliard <julliard@winehq.org> Wed Jan 08 21:27:28 2014 +0100
tree 4a2d7776c4e376ee2f1ed142c34c07c907fe4aba
parent 79054f2fd43d02d14d204a8c30a65f78186a45bf

riched20: Ensure the cursors are correctly ordered in the case of a zero (logical) length selection bridging two runs.

(cherry picked from commit b0f177b61913b9fac1cf8952582d77576705bad1)

dlls/riched20/caret.c

diff --git a/dlls/riched20/caret.c b/dlls/riched20/caret.c
index fd7034d..4b1fea5 100644
--- a/dlls/riched20/caret.c
+++ b/dlls/riched20/caret.c
@@ -56,7 +56,19 @@
 
 int ME_GetSelection(ME_TextEditor *editor, ME_Cursor **from, ME_Cursor **to)
 {
-  if (ME_GetCursorOfs(&editor->pCursors[0]) < ME_GetCursorOfs(&editor->pCursors[1]))
+  int from_ofs = ME_GetCursorOfs( &editor->pCursors[0] );
+  int to_ofs   = ME_GetCursorOfs( &editor->pCursors[1] );
+  BOOL swap = (from_ofs > to_ofs);
+
+  if (from_ofs == to_ofs)
+  {
+      /* If cursor[0] is at the beginning of a run and cursor[1] at the end
+         of the prev run then we need to swap. */
+      if (editor->pCursors[0].nOffset < editor->pCursors[1].nOffset)
+          swap = TRUE;
+  }
+
+  if (!swap)
   {
     *from = &editor->pCursors[0];
     *to = &editor->pCursors[1];