Fix comments

The FIXME is not needed, this is how StyleChanger and FracChanger work:
In the constructor, they change the state of the FontInfo, and in the
destructor the state is set back. Therefore, all code that needs the changed
state, needs to be executed while the objects do still exist.

diff --git a/src/mathed/InsetMathFrac.cpp b/src/mathed/InsetMathFrac.cpp
index 6a175fad31ac26e0568c24684ef557917ee5a5c9..467735558cd2d087e35e25e5e8463a8a8dd9e931 100644 (file)
--- a/src/mathed/InsetMathFrac.cpp
+++ b/src/mathed/InsetMathFrac.cpp
@@ -553,8 +553,8 @@ void InsetMathBinom::metrics(MetricsInfo & mi, Dimension & dim) const
 {
        Dimension dim0, dim1;
 
-       // FIXME: for an unknown reason the cells must be set directly
-       // after the StyleChanger and cannot be set after the if case
+       // The cells must be set while the RAII objects (StyleChanger,
+       // FracChanger) do still exist and cannot be set after the if case.
        if (kind_ == DBINOM) {
                StyleChanger dummy(mi.base, LM_ST_DISPLAY);
                cell(0).metrics(mi, dim0);
@@ -587,8 +587,8 @@ void InsetMathBinom::draw(PainterInfo & pi, int x, int y) const
                kind_ == BRACK ? from_ascii("]") : from_ascii(")");
 
        int m = x + dim.width() / 2;
-       // FIXME: for an unknown reason the cells must be drawn directly
-       // after the StyleChanger and cannot be drawn after the if case
+       // The cells must be drawn while the RAII objects (StyleChanger,
+       // FracChanger) do still exist and cannot be drawn after the if case.
        if (kind_ == DBINOM) {
                StyleChanger dummy(pi.base, LM_ST_DISPLAY);
                cell(0).draw(pi, m - dim0.wid / 2, y - dim0.des - 3 - 5);