On 10/27/2010 09:38 PM, Uwe Stöhr wrote:
While debugging LyX to fix http://www.lyx.org/trac/ticket/6985 I
noticed the following general dialog and context-menu issues:
- right-clicking on a table cell (to see the context menu) invokes
InsetTabular::getStatus 4 times
This is expected: We're checking to see if the various commands that
will appear on the menu are enabled. There are probably four of them
that end up at the table level. So the argument to getStatus() is
different each time.
- opening the table settings dialog (by clicking on the Settings entry
of the context menu) invokes
InsetTabular::getStatus 1 time
GuiTabular::dialogToParams() 3 times
InsetTabular::getStatus 1 time
The multiple calls to dialogToParams() are probably unnecessary, but one
would need to work through the code to see why we keep calling it and
how it might be prevented. I've thought about having some kind of lock
variable here, so that we'd only do the call once, at the end. But I
haven't worked on it.
rh
