#1886 ScintillaWin.cxx: EM_SETSEL sets selection wrongly

Bug
closed-fixed
5
2016-12-05
2016-11-22
No

According to MSDN on EM_SETSEL:

Edit controls: The control displays a flashing caret at the end position regardless of the relative values of start and end.

ScintillaWin.cxx in line 1672 does however respect the relative values of start and end:

        if (nStart > nEnd) {
            SetSelection(nEnd, nStart);
        } else {
            SetSelection(nStart, nEnd);
        }

The if condition and the else branch can be dropped; then the code of the if branch will comply with the specification.

Discussion

  • Neil Hodgson

    Neil Hodgson - 2016-11-25
    • labels: --> scintilla, win32
    • status: open --> open-fixed
    • assigned_to: Neil Hodgson
     
  • Neil Hodgson

    Neil Hodgson - 2016-12-05
    • status: open-fixed --> closed-fixed
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks