Andre Poenitz wrote:

> On Mon, Nov 22, 2004 at 05:12:03PM +0100, Georg Baum wrote:
>> - I assume that we can never enter e.g. addCol() if getStatus returns
>> false for "append-column". Therefore I removed the error messages in
>> addCol() and delCol(). Is this assumption true?
> 
> In theory, yes ;-)

Good.
 
> Whatever you do, issuing a warning in the minibuffer would be in
> order I suppose.

I think something like this is intended with the following code in
LyXFunc::dispatch():

        // We cannot use this function here
        if (!getStatus(cmd).enabled()) {
                lyxerr[Debug::ACTION] << "LyXFunc::dispatch: "
                       << lyxaction.getActionName(action)
                       << " [" << action << "] is disabled at this location"
                       << endl;
                setErrorMessage(getStatusMessage());

        } else {
...


The only problem is: setErrorMessage() does nothing but set the variable
dispatch_buffer which is never used ;-) It seems like some code is missing
here. Can somebody who knows this part of the code please have a look?


Georg

Reply via email to