On Thu, Sep 19, 2002 at 04:35:37PM +0300, Martin Vermeer wrote:
> Another thing: I ran into a bug in math_fontinset. If you have two
> fontinsets inside each other, only the outer one calls validate.
Yes. It should call MathNestInset::validate() as well.
> This will lead e.g. to a LaTeX processing error if you place a mathfrak
> fontinset inside a mathbf, because amssymb doesn't get required then. I
> found out when setting up "noun" validation... somebody look into this
> please.
Fixed.
> BTW Where are lyxposspace, lyxnegspace getting defined?
math_spaceinset.C?
> - case LFUN_EMPH: handleFont(bv, cmd.argument, "mathcal"); break;
> + case LFUN_EMPH: handleFont(bv, cmd.argument, "emph"); break;
I think you'll get angry users here....
> --- src/mathed/math_fontinset.C 2002/08/13 18:21:44 1.15
> +++ src/mathed/math_fontinset.C 2002/09/19 12:52:19
> @@ -76,6 +75,9 @@ void MathFontInset::validate(LaTeXFeatur
> // Fraktur used:
> if (key_->name == "mathfrak" || key_->name == "mathbb")
> features.require("amssymb");
> + // MV BUG: this fails if embedded in other (font-)inset. Why?
> + //if (key_->name == "noun") features.require("noun");
> + features.require("noun");
Because it does not call validate on the content. Just fixed it.
Anyway. Looks good now except for the lyxfunc.C part which I haven't
checked.
Andre'
--
Those who desire to give up Freedom in order to gain Security,
will not have, nor do they deserve, either one. (T. Jefferson)
