Migrate from GitHub to SourceForge with this tool. Check out all of SourceForge's recent improvements.
Close

#1374 SCI_MARKERDELETE(l, m >= 0) deletes all markers m

Bug
closed-fixed
Scintilla (796)
5
2012-06-18
2012-06-17
No

#SCI_MARKERDELETE: "If you added the same marker more than once to the line, this will delete one copy each time it is used". However, while LineMarkers::DeleteMark() handles "all" correctly, MarkerHandleSet::RemoveNumber(m) removes all markers m instead of the first one. There are at least 2 possible fixes: handle "all" in RemoveNumber(), or change it to remove one marker at a time. The former is slightly more effective.

Discussion

  • Neil Hodgson

    Neil Hodgson - 2012-06-18
    • assigned_to: nobody --> nyamatongwe
    • status: open --> open-fixed
     
  • Dimitar Zhekov

    Dimitar Zhekov - 2012-06-18

    No, that's somebody else. The only patch I can remember writing for sci* was xsmp support for scite/gtk+, but that was rejected.

     
  • Dimitar Zhekov

    Dimitar Zhekov - 2012-06-18
    • status: open-fixed --> closed-fixed
     

Log in to post a comment.