>>>>> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> Abdelrazak Younes wrote: >>>> You may be right indeed. But it seems that updateLayoutChoice >>>> does a setLayout. Is it necessary to do another one? >>> Maybe not. Actually it may well be that the >>> BufferView::Pimpl::firstLayout() method can go altogether. >>> updateLayoutChoice() will set the layout that match the current >>> cursor position anyway. An the cursor is always valid right? >>> >>> The attached patch should work, will test. >> Yes it works, I will propose a full patch that gets rid of >> firstLayout() once I have committed my pending patch. Abdelrazak> Here is the patch, objection? It looks good to me. This firstLayout method is weird anyway, I did not even know it existed. JMarc
