Martin Vermeer wrote:

> What does this do? I don't get it.
> 
>                 if (!lyx::support::contains("tcb", cmd.argument[0])) {
>                         flag.enable(false);
>                         break;
>                 }
>                 flag.setOnOff(cmd.argument[0] == v_align_);

I guess it was originally meant to check the alignment flag. I experimented
with it, but it does not work anymore. Maybe we should leave the #if 0
block in and add a comment that somebody who knows what it did originally
should remove or fix it.

BTW, both your and my code rely on the fact that cur.row() and cur.idx() are
always valid. I am not sure wether this assumption is safe because I don't
really know hor the cursor works. If it is not safe, we need to add checks.


Georg


Reply via email to