Abdelrazak Younes <[EMAIL PROTECTED]> writes:

 | This needsredraw() function is right now necessary because the
| BufferView::update() method is used all other the place. But the plan
| is to remove the needRedraw() method.

Ok... so the need to redraw of not will eventually be done through a
function return value?
 
| Eventually, when the cleanup is done the command flow will be like this:
| 
| User press a key.
| 
| WorkArea::processKey(...) {
|    BufferView::processKey(...) {
|      ...
|      BufferView::update(); // _no_ call to any LyXView method in there.
|    }
|    WorkArea::redraw();

How does the WorkArea::redraw() know whether to redraw anything or
not?

| > My plan was to do this merge yeasterday, but real life intervened.
| 
| You should have started with this instead of merging the most recent patch.

Why?

The recent path was ortogonal to the first.

-- 
        Lgb

Reply via email to