>>>>> "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).

>> 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?

JMarc

Reply via email to