Robert:

> As for "lazy" Undo/Redo, I went ahead and tweaked SetScrollBars() to
> return a bool which will be true if a Redraw() was done.  There are a
> *lot* of places (in Editor.cxx) that do a SetScrollBars() followed
> immediately by a Redraw()... or there were. :)

   Does this achieve any noticeable effect? Redraw adds the client
rectangle to the invalid area of the window. If this is done twice, I
expect it would very lightweight. Unless there is a measurable effect
its not worth churning the code.

> ONE place that had Redraw() FOLLOWED by SetScrollBars() is the end of
> Editor::SetDocPointer()... I was so tempted that I used the above code
> there too, so you may need to explain why that is a "Bad Idea(tm)".

   When swapping the document you will always have to redraw the whole
window. Order doesn't matter much so I suppose it can be swapped for
consistency.

   Neil

_______________________________________________
Scintilla-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scintilla-interest

Reply via email to