Change cell numbers so that 0 is the main cell

This leads to code simplification in overset, root and underset.
Further simplification is possible.
This commit is contained in:
Jean-Marc Lasgouttes 2018-11-05 21:29:47 -10:00
parent 9d304c0a1d
commit 097da3a943
8 changed files with 86 additions and 96 deletions

View File

@ -19,6 +19,8 @@
#include "LaTeXFeatures.h" #include "LaTeXFeatures.h"
#include "MetricsInfo.h" #include "MetricsInfo.h"
#include "support/lassert.h"
using namespace std; using namespace std;
namespace lyx { namespace lyx {
@ -32,14 +34,14 @@ Inset * InsetMathOverset::clone() const
void InsetMathOverset::metrics(MetricsInfo & mi, Dimension & dim) const void InsetMathOverset::metrics(MetricsInfo & mi, Dimension & dim) const
{ {
Changer dummy2 = mi.base.changeEnsureMath(); Changer dummy2 = mi.base.changeEnsureMath();
Dimension dim1;
cell(1).metrics(mi, dim1);
Changer dummy = mi.base.changeFrac();
Dimension dim0; Dimension dim0;
cell(0).metrics(mi, dim0); cell(0).metrics(mi, dim0);
dim.wid = max(dim0.width(), dim1.wid) + 4; Changer dummy = mi.base.changeFrac();
dim.asc = dim1.asc + dim0.height() + 4; Dimension dim1;
dim.des = dim1.des; cell(1).metrics(mi, dim1);
dim.wid = max(dim1.width(), dim0.wid) + 4;
dim.asc = dim0.asc + dim1.height() + 4;
dim.des = dim0.des;
} }
@ -47,13 +49,24 @@ void InsetMathOverset::draw(PainterInfo & pi, int x, int y) const
{ {
Changer dummy2 = pi.base.changeEnsureMath(); Changer dummy2 = pi.base.changeEnsureMath();
Dimension const dim = dimension(*pi.base.bv); Dimension const dim = dimension(*pi.base.bv);
Dimension const & dim0 = cell(0).dimension(*pi.base.bv);
Dimension const & dim1 = cell(1).dimension(*pi.base.bv); Dimension const & dim1 = cell(1).dimension(*pi.base.bv);
Dimension const & dim0 = cell(0).dimension(*pi.base.bv);
int m = x + dim.wid / 2; int m = x + dim.wid / 2;
int yo = y - dim1.asc - dim0.des - 1; int yo = y - dim0.asc - dim1.des - 1;
cell(1).draw(pi, m - dim1.wid / 2, y); cell(0).draw(pi, m - dim0.wid / 2, y);
Changer dummy = pi.base.changeFrac(); Changer dummy = pi.base.changeFrac();
cell(0).draw(pi, m - dim0.width() / 2, yo); cell(1).draw(pi, m - dim1.width() / 2, yo);
}
bool InsetMathOverset::idxUpDown(Cursor & cur, bool up) const
{
idx_type target = up; // up ? 1 : 0, since upper cell has idx 1
if (cur.idx() == target)
return false;
cur.idx() = target;
cur.pos() = cur.cell().x2pos(&cur.bv(), cur.x_target());
return true;
} }
@ -62,27 +75,27 @@ void InsetMathOverset::write(WriteStream & os) const
MathEnsurer ensurer(os); MathEnsurer ensurer(os);
if (os.fragile()) if (os.fragile())
os << "\\protect"; os << "\\protect";
os << "\\overset{" << cell(0) << "}{" << cell(1) << '}'; os << "\\overset{" << cell(1) << "}{" << cell(0) << '}';
} }
void InsetMathOverset::normalize(NormalStream & os) const void InsetMathOverset::normalize(NormalStream & os) const
{ {
os << "[overset " << cell(0) << ' ' << cell(1) << ']'; os << "[overset " << cell(1) << ' ' << cell(0) << ']';
} }
void InsetMathOverset::mathmlize(MathStream & ms) const void InsetMathOverset::mathmlize(MathStream & ms) const
{ {
ms << "<mover accent='false'>" << cell(1) << cell(0) << "</mover>"; ms << "<mover accent='false'>" << cell(0) << cell(1) << "</mover>";
} }
void InsetMathOverset::htmlize(HtmlStream & os) const void InsetMathOverset::htmlize(HtmlStream & os) const
{ {
os << MTag("span", "class='overset'") os << MTag("span", "class='overset'")
<< MTag("span", "class='top'") << cell(0) << ETag("span") << MTag("span", "class='top'") << cell(1) << ETag("span")
<< MTag("span") << cell(1) << ETag("span") << MTag("span") << cell(0) << ETag("span")
<< ETag("span"); << ETag("span");
} }

View File

@ -28,9 +28,7 @@ public:
/// ///
void draw(PainterInfo & pi, int x, int y) const; void draw(PainterInfo & pi, int x, int y) const;
/// ///
idx_type firstIdx() const { return 1; } bool idxUpDown(Cursor & cur, bool up) const;
///
idx_type lastIdx() const { return 1; }
/// ///
void write(WriteStream & os) const; void write(WriteStream & os) const;
/// ///

View File

@ -22,6 +22,7 @@
#include "frontends/Painter.h" #include "frontends/Painter.h"
#include "support/lassert.h"
using namespace std; using namespace std;
@ -84,7 +85,7 @@ void mathed_root_metrics(MetricsInfo & mi, MathData const & nucleus,
void InsetMathRoot::metrics(MetricsInfo & mi, Dimension & dim) const void InsetMathRoot::metrics(MetricsInfo & mi, Dimension & dim) const
{ {
mathed_root_metrics(mi, cell(1), &cell(0), dim); mathed_root_metrics(mi, cell(0), &cell(1), dim);
} }
@ -127,26 +128,26 @@ void mathed_draw_root(PainterInfo & pi, int x, int y, MathData const & nucleus,
void InsetMathRoot::draw(PainterInfo & pi, int x, int y) const void InsetMathRoot::draw(PainterInfo & pi, int x, int y) const
{ {
mathed_draw_root(pi, x, y, cell(1), &cell(0), dimension(*pi.base.bv)); mathed_draw_root(pi, x, y, cell(0), &cell(1), dimension(*pi.base.bv));
} }
void InsetMathRoot::write(WriteStream & os) const void InsetMathRoot::write(WriteStream & os) const
{ {
MathEnsurer ensurer(os); MathEnsurer ensurer(os);
os << "\\sqrt[" << cell(0) << "]{" << cell(1) << '}'; os << "\\sqrt[" << cell(1) << "]{" << cell(0) << '}';
} }
void InsetMathRoot::normalize(NormalStream & os) const void InsetMathRoot::normalize(NormalStream & os) const
{ {
os << "[root " << cell(0) << ' ' << cell(1) << ']'; os << "[root " << cell(1) << ' ' << cell(0) << ']';
} }
bool InsetMathRoot::idxUpDown(Cursor & cur, bool up) const bool InsetMathRoot::idxUpDown(Cursor & cur, bool up) const
{ {
Cursor::idx_type const target = up ? 0 : 1; Cursor::idx_type const target = up; //up ? 1 : 0;
if (cur.idx() == target) if (cur.idx() == target)
return false; return false;
cur.idx() = target; cur.idx() = target;
@ -157,34 +158,34 @@ bool InsetMathRoot::idxUpDown(Cursor & cur, bool up) const
void InsetMathRoot::maple(MapleStream & os) const void InsetMathRoot::maple(MapleStream & os) const
{ {
os << '(' << cell(1) << ")^(1/(" << cell(0) <<"))"; os << '(' << cell(0) << ")^(1/(" << cell(1) <<"))";
} }
void InsetMathRoot::mathematica(MathematicaStream & os) const void InsetMathRoot::mathematica(MathematicaStream & os) const
{ {
os << '(' << cell(1) << ")^(1/(" << cell(0) <<"))"; os << '(' << cell(0) << ")^(1/(" << cell(1) <<"))";
} }
void InsetMathRoot::octave(OctaveStream & os) const void InsetMathRoot::octave(OctaveStream & os) const
{ {
os << '(' << cell(1) << ")^(1/(" << cell(0) <<"))"; os << '(' << cell(0) << ")^(1/(" << cell(1) <<"))";
} }
void InsetMathRoot::mathmlize(MathStream & os) const void InsetMathRoot::mathmlize(MathStream & os) const
{ {
os << MTag("mroot") << cell(1) << cell(0) << ETag("mroot"); os << MTag("mroot") << cell(0) << cell(1) << ETag("mroot");
} }
void InsetMathRoot::htmlize(HtmlStream & os) const void InsetMathRoot::htmlize(HtmlStream & os) const
{ {
os << MTag("span", "class='root'") os << MTag("span", "class='root'")
<< MTag("sup") << cell(0) << ETag("sup") << MTag("sup") << cell(1) << ETag("sup")
<< from_ascii("&radic;") << from_ascii("&radic;")
<< MTag("span", "class='rootof'") << cell(1) << ETag("span") << MTag("span", "class='rootof'") << cell(0) << ETag("span")
<< ETag("span"); << ETag("span");
} }

View File

@ -26,14 +26,11 @@ public:
InsetMathRoot(Buffer * buf); InsetMathRoot(Buffer * buf);
/// ///
bool idxUpDown(Cursor & cur, bool up) const; bool idxUpDown(Cursor & cur, bool up) const;
/// ///
void metrics(MetricsInfo & mi, Dimension & dim) const; void metrics(MetricsInfo & mi, Dimension & dim) const;
/// ///
void draw(PainterInfo & pi, int x, int y) const; void draw(PainterInfo & pi, int x, int y) const;
///
idx_type firstIdx() const { return 1; }
///
idx_type lastIdx() const { return 1; }
/// ///
void write(WriteStream & os) const; void write(WriteStream & os) const;

View File

@ -36,15 +36,17 @@ Inset * InsetMathStackrel::clone() const
bool InsetMathStackrel::idxUpDown(Cursor & cur, bool up) const bool InsetMathStackrel::idxUpDown(Cursor & cur, bool up) const
{ {
idx_type const npos = 1234; // impossible number
idx_type target = npos;
if (up) { if (up) {
if (cur.idx() == 0) idx_type const targets[] = { 1, npos, 0 };
return false; target = targets[cur.idx()];
} else { } else {
if (cur.idx() + 1 == nargs()) idx_type const targets[] = { 2, 0, npos };
return false; target = targets[cur.idx()];
} }
InsetMath::idx_type target = up ? cur.idx() - 1 : cur.idx() + 1;
if (cur.idx() == target) if (target == npos || target == nargs())
return false; return false;
cur.idx() = target; cur.idx() = target;
cur.pos() = cell(target).x2pos(&cur.bv(), cur.x_target()); cur.pos() = cell(target).x2pos(&cur.bv(), cur.x_target());
@ -62,21 +64,21 @@ MathClass InsetMathStackrel::mathClass() const
void InsetMathStackrel::metrics(MetricsInfo & mi, Dimension & dim) const void InsetMathStackrel::metrics(MetricsInfo & mi, Dimension & dim) const
{ {
Changer dummy2 = mi.base.changeEnsureMath(); Changer dummy2 = mi.base.changeEnsureMath();
Dimension dim1;
cell(1).metrics(mi, dim1);
Changer dummy = mi.base.changeFrac();
Dimension dim0; Dimension dim0;
cell(0).metrics(mi, dim0); cell(0).metrics(mi, dim0);
Changer dummy = mi.base.changeFrac();
Dimension dim1;
cell(1).metrics(mi, dim1);
if (nargs() > 2) { if (nargs() > 2) {
Dimension dim2; Dimension dim2;
cell(2).metrics(mi, dim2); cell(2).metrics(mi, dim2);
dim.wid = max(max(dim0.width(), dim1.width()), dim2.width()) + 4; dim.wid = max(max(dim1.width(), dim0.width()), dim2.width()) + 4;
dim.asc = dim1.ascent() + dim0.height() + 4; dim.asc = dim0.ascent() + dim1.height() + 4;
dim.des = dim1.descent() + dim2.height() + dim2.descent() + 1; dim.des = dim0.descent() + dim2.height() + dim2.descent() + 1;
} else { } else {
dim.wid = max(dim0.width(), dim1.width()) + 4; dim.wid = max(dim1.width(), dim0.width()) + 4;
dim.asc = dim1.ascent() + dim0.height() + 4; dim.asc = dim0.ascent() + dim1.height() + 4;
dim.des = dim1.descent(); dim.des = dim0.descent();
} }
} }
@ -88,13 +90,13 @@ void InsetMathStackrel::draw(PainterInfo & pi, int x, int y) const
Dimension const & dim0 = cell(0).dimension(*pi.base.bv); Dimension const & dim0 = cell(0).dimension(*pi.base.bv);
Dimension const & dim1 = cell(1).dimension(*pi.base.bv); Dimension const & dim1 = cell(1).dimension(*pi.base.bv);
int m = x + dim.width() / 2; int m = x + dim.width() / 2;
int yo = y - dim1.ascent() - dim0.descent() - 1; int yo = y - dim0.ascent() - dim1.descent() - 1;
cell(1).draw(pi, m - dim1.width() / 2, y); cell(0).draw(pi, m - dim0.width() / 2, y);
Changer dummy = pi.base.changeFrac(); Changer dummy = pi.base.changeFrac();
cell(0).draw(pi, m - dim0.width() / 2, yo); cell(1).draw(pi, m - dim1.width() / 2, yo);
if (nargs() > 2) { if (nargs() > 2) {
Dimension const & dim2 = cell(2).dimension(*pi.base.bv); Dimension const & dim2 = cell(2).dimension(*pi.base.bv);
int y2 = y + dim1.descent() + dim2.ascent() + 1; int y2 = y + dim0.descent() + dim2.ascent() + 1;
cell(2).draw(pi, m - dim2.width() / 2, y2); cell(2).draw(pi, m - dim2.width() / 2, y2);
} }
} }
@ -106,13 +108,13 @@ void InsetMathStackrel::write(WriteStream & os) const
os << "\\stackrel"; os << "\\stackrel";
if (nargs() > 2) if (nargs() > 2)
os << '[' << cell(2) << ']'; os << '[' << cell(2) << ']';
os << '{' << cell(0) << "}{" << cell(1) << '}'; os << '{' << cell(1) << "}{" << cell(0) << '}';
} }
void InsetMathStackrel::normalize(NormalStream & os) const void InsetMathStackrel::normalize(NormalStream & os) const
{ {
os << "[stackrel " << cell(0) << ' ' << cell(1); os << "[stackrel " << cell(1) << ' ' << cell(0);
if (nargs() > 2) if (nargs() > 2)
os << ' ' << cell(2); os << ' ' << cell(2);
os << ']'; os << ']';
@ -122,9 +124,9 @@ void InsetMathStackrel::normalize(NormalStream & os) const
void InsetMathStackrel::mathmlize(MathStream & ms) const void InsetMathStackrel::mathmlize(MathStream & ms) const
{ {
if (nargs() > 2) if (nargs() > 2)
ms << "<munderover>" << cell(1) << cell(2) << cell(0) << "</munderover>"; ms << "<munderover>" << cell(0) << cell(2) << cell(1) << "</munderover>";
else else
ms << "<mover accent='false'>" << cell(1) << cell(0) << "</mover>"; ms << "<mover accent='false'>" << cell(0) << cell(1) << "</mover>";
} }
@ -132,14 +134,14 @@ void InsetMathStackrel::htmlize(HtmlStream & os) const
{ {
if (nargs() > 2) { if (nargs() > 2) {
os << MTag("span", "class='underoverset'") os << MTag("span", "class='underoverset'")
<< MTag("span", "class='top'") << cell(0) << ETag("span") << MTag("span", "class='top'") << cell(1) << ETag("span")
<< MTag("span") << cell(1) << ETag("span") << MTag("span") << cell(0) << ETag("span")
<< MTag("span", "class='bottom'") << cell(2) << ETag("span"); << MTag("span", "class='bottom'") << cell(2) << ETag("span");
} else { } else {
// at the moment, this is exactly the same as overset // at the moment, this is exactly the same as overset
os << MTag("span", "class='overset'") os << MTag("span", "class='overset'")
<< MTag("span", "class='top'") << cell(0) << ETag("span") << MTag("span", "class='top'") << cell(1) << ETag("span")
<< MTag("span") << cell(1) << ETag("span"); << MTag("span") << cell(0) << ETag("span");
} }
os << ETag("span"); os << ETag("span");
} }

View File

@ -30,10 +30,6 @@ public:
/// ///
void draw(PainterInfo & pi, int x, int y) const; void draw(PainterInfo & pi, int x, int y) const;
/// ///
idx_type firstIdx() const { return 1; }
///
idx_type lastIdx() const { return 1; }
///
void write(WriteStream & os) const; void write(WriteStream & os) const;
/// ///
void normalize(NormalStream &) const; void normalize(NormalStream &) const;

View File

@ -33,14 +33,14 @@ Inset * InsetMathUnderset::clone() const
void InsetMathUnderset::metrics(MetricsInfo & mi, Dimension & dim) const void InsetMathUnderset::metrics(MetricsInfo & mi, Dimension & dim) const
{ {
Changer dummy2 = mi.base.changeEnsureMath(); Changer dummy2 = mi.base.changeEnsureMath();
Dimension dim1;
cell(1).metrics(mi, dim1);
Changer dummy = mi.base.changeFrac();
Dimension dim0; Dimension dim0;
cell(0).metrics(mi, dim0); cell(0).metrics(mi, dim0);
dim.wid = max(dim0.width(), dim1.width()) + 4; Changer dummy = mi.base.changeFrac();
dim.asc = dim1.ascent(); Dimension dim1;
dim.des = dim1.descent() + dim0.height() + 4; cell(1).metrics(mi, dim1);
dim.wid = max(dim1.width(), dim0.width()) + 4;
dim.asc = dim0.ascent();
dim.des = dim0.descent() + dim1.height() + 4;
} }
@ -48,24 +48,13 @@ void InsetMathUnderset::draw(PainterInfo & pi, int x, int y) const
{ {
Changer dummy2 = pi.base.changeEnsureMath(); Changer dummy2 = pi.base.changeEnsureMath();
Dimension const dim = dimension(*pi.base.bv); Dimension const dim = dimension(*pi.base.bv);
Dimension const & dim0 = cell(0).dimension(*pi.base.bv);
Dimension const & dim1 = cell(1).dimension(*pi.base.bv); Dimension const & dim1 = cell(1).dimension(*pi.base.bv);
Dimension const & dim0 = cell(0).dimension(*pi.base.bv);
int m = x + dim.wid / 2; int m = x + dim.wid / 2;
int yo = y + dim1.descent() + dim0.ascent() + 1; int yo = y + dim0.descent() + dim1.ascent() + 1;
cell(1).draw(pi, m - dim1.width() / 2, y); cell(0).draw(pi, m - dim0.width() / 2, y);
Changer dummy = pi.base.changeFrac(); Changer dummy = pi.base.changeFrac();
cell(0).draw(pi, m - dim0.width() / 2, yo); cell(1).draw(pi, m - dim1.width() / 2, yo);
}
bool InsetMathUnderset::idxUpDown(Cursor & cur, bool up) const
{
idx_type target = up; // up ? 1 : 0, since upper cell has idx 1
if (cur.idx() == target)
return false;
cur.idx() = target;
cur.pos() = cur.cell().x2pos(&cur.bv(), cur.x_target());
return true;
} }
@ -74,27 +63,27 @@ void InsetMathUnderset::write(WriteStream & os) const
MathEnsurer ensurer(os); MathEnsurer ensurer(os);
if (os.fragile()) if (os.fragile())
os << "\\protect"; os << "\\protect";
os << "\\underset{" << cell(0) << "}{" << cell(1) << '}'; os << "\\underset{" << cell(1) << "}{" << cell(0) << '}';
} }
void InsetMathUnderset::normalize(NormalStream & os) const void InsetMathUnderset::normalize(NormalStream & os) const
{ {
os << "[underset " << cell(0) << ' ' << cell(1) << ']'; os << "[underset " << cell(1) << ' ' << cell(0) << ']';
} }
void InsetMathUnderset::mathmlize(MathStream & ms) const void InsetMathUnderset::mathmlize(MathStream & ms) const
{ {
ms << "<munder accent='false'>" << cell(1) << cell(0) << "</munder>"; ms << "<munder accent='false'>" << cell(0) << cell(1) << "</munder>";
} }
void InsetMathUnderset::htmlize(HtmlStream & os) const void InsetMathUnderset::htmlize(HtmlStream & os) const
{ {
os << MTag("span", "class='underset'") os << MTag("span", "class='underset'")
<< MTag("span") << cell(0) << ETag("span") << MTag("span") << cell(1) << ETag("span")
<< MTag("span", "class='bottom'") << cell(1) << ETag("span") << MTag("span", "class='bottom'") << cell(0) << ETag("span")
<< ETag("span"); << ETag("span");
} }

View File

@ -28,12 +28,6 @@ public:
/// ///
void draw(PainterInfo & pi, int x, int y) const; void draw(PainterInfo & pi, int x, int y) const;
/// ///
idx_type firstIdx() const { return 1; }
///
idx_type lastIdx() const { return 1; }
///
bool idxUpDown(Cursor & cur, bool up) const;
///
void write(WriteStream & ws) const; void write(WriteStream & ws) const;
/// ///
void normalize(NormalStream & ns) const; void normalize(NormalStream & ns) const;