On 08/09/2011 05:31 PM, Pavel Sanda wrote:
> Vincent van Ravesteijn wrote:
>>> Author: vfr
>>> - string const s = cmd.getArg(1);
>>> // FIXME: We only check for the very first argument...
>>> + string const s = cmd.getArg(1);
>>> + // We always enable the lfun if it is coming from the dialog
>>> + // because the dialog makes sure all the settings are valid,
>>> + // even though the first argument might not be valid now.
>>> + if (s == "from-dialog") {
>>> + status.setEnabled(true);
>>> + return true;
>>> + }
>>> +
>>> int action = Tabular::LAST_ACTION;
>>> int i = 0;
>>> for (; tabularFeature[i].action != Tabular::LAST_ACTION; ++i) {
>> You forgot to strip the "from-dialog" parameter from the argument in the
>> handling of INSET_MODIFY in doDispatch. Now from-dialog is interpreted as
>> tabular feature.
> did i miss something or is it vfr talking to vfr? :)
>
Vincent has been away too long. ;-)
rh
