On Friday 27 July 2001 18:31, Lars Gullik Bjønnes wrote:
> Michael Schmitt <[EMAIL PROTECTED]> writes:
> 
> | Hi,
> | 
> | below find a list of all compiler warnings reported by g++ 2.95.3.
> 
> And these are the only real warnings.
> 
> | insetgraphics.C:264: warning: initialization to `int' from `float'
> | insetref.C:37: warning: unused parameter `class BufferView * bv'
> | ControlCharacter.C:193: warning: enumeration value `LATEX_TOGGLE' not 
handled in switch
> | FormMathsPanel.C:244: warning: enumeration value `MM_DOTS' not handled in 
switch

Am I correct to say that LaTeX is no longer a font property and that I can 
therefore remove LATEX_TOGGLE from the helper enum in the frontends?

namespace character {
        ///
        enum FONT_STATE {
                ///
                IGNORE,
                ///
                EMPH_TOGGLE,
                ///
                UNDERBAR_TOGGLE,
                ///
                NOUN_TOGGLE,
                ///
                LATEX_TOGGLE,
                ///
                INHERIT
        };
}

It is used to fill the Misc choice item in the Character dialog (Latex mode), 
but isn't actually acted upon, hence the warning posted by Michael, above:

void ControlCharacter::setBar(character::FONT_STATE val)
{
        switch (val) {
        case character::IGNORE:
                font_->setEmph(LyXFont::IGNORE);
                font_->setUnderbar(LyXFont::IGNORE);
                font_->setNoun(LyXFont::IGNORE);
                break;

        case character::EMPH_TOGGLE:
                font_->setEmph(LyXFont::TOGGLE);
                break;

        case character::UNDERBAR_TOGGLE:
                font_->setUnderbar(LyXFont::TOGGLE);
                break;

        case character::NOUN_TOGGLE:
                font_->setNoun(LyXFont::TOGGLE);
                break;

        case character::INHERIT:
                font_->setEmph(LyXFont::INHERIT);
                font_->setUnderbar(LyXFont::INHERIT);
                font_->setNoun(LyXFont::INHERIT);
                break;
        }
}

Angus

Reply via email to