Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| Abdelrazak Younes wrote:
| > Hello,
| > You are going to be happy Lars, this patch removes the needRedraw
| > interface and simplify the screen update procedure. It also fixes
| > all crash problems.
| > The performance may suffer a bit because we do the second drawing
| > step in all cases. This could be optimized out by checking the
| > return value of the BufferView::update() method.
|
| This updated patch implement this bool return value in preparation for
| this potential optimization. It also delete the commented out code.
but how do you plan to communicate the need for a second updateMetrics
step to the frontend? After all the frontend should not know
beforehand if a update is needed or not due to some event.
(And it cannot even assume that an update is needed...)
And IMHO it is nice to keep those two parts of the frontend ignorant
of each other: the event handling and the drawing.
But the change is good regardless.
--
Lgb
