Jean-Marc Lasgouttes a écrit :
"Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
I am not against adding a clear() method, but would not it be
better to have MenuBackend::expand clear the menu first? It seems
to be the assumption in all frontends.
Abdelrazak> Please tell me if you are going to add it or not. If not I
Abdelrazak> will put a work-around in the qt4 frontend (which does not
Abdelrazak> compile without it).
I am going to change expand so that it clears the menu first. Is that
OK? I'll send a patch tomorrow (I am a bit busy these days, sorry).
No problem. Thanks.
Also, could you explain why it should speed up operation?
Abdelrazak> Two things: 1) no construction 2) no need to copy the
Abdelrazak> FuncRequest in QLAction (a const ref is used instead)
2) is because you keep the expanded menu around, right?
Yes.
JMarc
