Jean-Marc Lasgouttes wrote:
While investigating bug 4607 (that Abdel solved in the meantime), I
came up with this patch to handle LyXView::getStatus.
I post it here to insist on how things are supposed to work.
LyXView::getStatus has been changed to return true if it has something
to say, and false otherwise. This is how different getStatus function
(but the main one) work, and I'd like it to stay that way. For some
reason BufferView::getStatus has been changed to another model...
Now, there is no knowledge in LyXFunc class of what LFUNs are handled
at LyXView level. I think it is how it should be.
Abdel, does the patch make sense to you?
Yes. I also planned to do this sometimes in the future but I was afraid
of potential bugs too...
I'd rather have a nod before
applying because it has potential for strange bugs...
On second thinking it should be fine. Only those LFUNs handled by
GuiView should be affected so I think this is OK.
Ack from me.
Abdel.
