SCI_GOTOPOS: Remove Redraw() call?

Brought to you by: antoniolinares, johnsoonj, kapix93, nyamatongwe, and 2 others

#764 SCI_GOTOPOS: Remove Redraw() call?

Status: closed

Owner: Neil Hodgson

Labels: Scintilla (391)

Priority: 5

Updated: 2011-02-03

Created: 2010-12-27

Creator: Jordan Russell

Private: No

In the handling for SCI_GOTOPOS, is the call to Redraw() really necessary?

It appears to paint correctly without it, and the call isn't present in the handling for other commands that move the caret / clear the selection, e.g. SCI_GOTOLINE, SCI_SETSEL, SCI_CHARRIGHT.

(I ask because I'm finding Redraw() to be a real perf-killer under VMware.)

Discussion

Neil Hodgson - 2010-12-28

Removing the Redraw is probably safe. Committed.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Neil Hodgson - 2010-12-28

assigned_to: nobody --> nyamatongwe
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Neil Hodgson - 2011-02-03

status: open --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

SCI_GOTOPOS: Remove Redraw() call?

Group

Searches

Help

#764 SCI_GOTOPOS: Remove Redraw() call?

Discussion