X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FText2.cpp;h=4a0c1dd3eb27b9b7393d0f4d6e75e6fd77ad2c15;hb=d145c2dc9f27666ca0ed9f8589f99f502d7b2a4d;hp=f34be2317a365945b5b92e22379c83a230a482de;hpb=b306e9a2e15664c630e1b462c3db3caf0e9cc9c9;p=lyx.git diff --git a/src/Text2.cpp b/src/Text2.cpp index f34be2317a..4a0c1dd3eb 100644 --- a/src/Text2.cpp +++ b/src/Text2.cpp @@ -46,10 +46,9 @@ #include "TextMetrics.h" #include "VSpace.h" -#include "insets/InsetEnvironment.h" - #include "mathed/InsetMathHull.h" +#include "support/lassert.h" #include "support/debug.h" #include "support/gettext.h" #include "support/textutils.h" @@ -73,19 +72,19 @@ bool Text::isMainText(Buffer const & buffer) const } -FontInfo Text::getLayoutFont(Buffer const & buffer, pit_type const pit) const +FontInfo Text::layoutFont(Buffer const & buffer, pit_type const pit) const { - LayoutPtr const & layout = pars_[pit].layout(); + Layout const & layout = pars_[pit].layout(); if (!pars_[pit].getDepth()) { - FontInfo lf = layout->resfont; + FontInfo lf = layout.resfont; // In case the default family has been customized - if (layout->font.family() == INHERIT_FAMILY) + if (layout.font.family() == INHERIT_FAMILY) lf.setFamily(buffer.params().getFont().fontInfo().family()); return lf; } - FontInfo font = layout->font; + FontInfo font = layout.font; // Realize with the fonts of lesser depth. //font.realize(outerFont(pit, paragraphs())); font.realize(buffer.params().getFont().fontInfo()); @@ -94,19 +93,19 @@ FontInfo Text::getLayoutFont(Buffer const & buffer, pit_type const pit) const } -FontInfo Text::getLabelFont(Buffer const & buffer, Paragraph const & par) const +FontInfo Text::labelFont(Buffer const & buffer, Paragraph const & par) const { - LayoutPtr const & layout = par.layout(); + Layout const & layout = par.layout(); if (!par.getDepth()) { - FontInfo lf = layout->reslabelfont; + FontInfo lf = layout.reslabelfont; // In case the default family has been customized - if (layout->labelfont.family() == INHERIT_FAMILY) + if (layout.labelfont.family() == INHERIT_FAMILY) lf.setFamily(buffer.params().getFont().fontInfo().family()); return lf; } - FontInfo font = layout->labelfont; + FontInfo font = layout.labelfont; // Realize with the fonts of lesser depth. font.realize(buffer.params().getFont().fontInfo()); @@ -118,15 +117,15 @@ void Text::setCharFont(Buffer const & buffer, pit_type pit, pos_type pos, Font const & fnt, Font const & display_font) { Font font = fnt; - LayoutPtr const & layout = pars_[pit].layout(); + Layout const & layout = pars_[pit].layout(); // Get concrete layout font to reduce against FontInfo layoutfont; if (pos < pars_[pit].beginOfBody()) - layoutfont = layout->labelfont; + layoutfont = layout.labelfont; else - layoutfont = layout->font; + layoutfont = layout.font; // Realize against environment font information if (pars_[pit].getDepth()) { @@ -136,7 +135,7 @@ void Text::setCharFont(Buffer const & buffer, pit_type pit, pars_[tp].getDepth()) { tp = outerHook(tp, paragraphs()); if (tp != pit_type(paragraphs().size())) - layoutfont.realize(pars_[tp].layout()->font); + layoutfont.realize(pars_[tp].layout().font); } } @@ -158,7 +157,7 @@ void Text::setInsetFont(BufferView const & bv, pit_type pit, pos_type pos, Font const & font, bool toggleall) { Inset * const inset = pars_[pit].getInset(pos); - BOOST_ASSERT(inset && inset->noFontChange()); + LASSERT(inset && inset->noFontChange(), /**/); CursorSlice::idx_type endidx = inset->nargs(); for (CursorSlice cs(*inset); cs.idx() != endidx; ++cs.idx()) { @@ -196,17 +195,17 @@ pit_type Text::undoSpan(pit_type pit) void Text::setLayout(Buffer const & buffer, pit_type start, pit_type end, docstring const & layout) { - BOOST_ASSERT(start != end); + LASSERT(start != end, /**/); BufferParams const & bufparams = buffer.params(); - LayoutPtr const & lyxlayout = bufparams.textClass()[layout]; + Layout const & lyxlayout = bufparams.documentClass()[layout]; for (pit_type pit = start; pit != end; ++pit) { Paragraph & par = pars_[pit]; par.applyLayout(lyxlayout); - if (lyxlayout->margintype == MARGIN_MANUAL) + if (lyxlayout.margintype == MARGIN_MANUAL) par.setLabelWidthString(par.translateIfPossible( - lyxlayout->labelstring(), buffer.params())); + lyxlayout.labelstring(), buffer.params())); } } @@ -214,23 +213,7 @@ void Text::setLayout(Buffer const & buffer, pit_type start, pit_type end, // set layout over selection and make a total rebreak of those paragraphs void Text::setLayout(Cursor & cur, docstring const & layout) { - BOOST_ASSERT(this == cur.text()); - // special handling of new environment insets - BufferView & bv = cur.bv(); - BufferParams const & params = bv.buffer().params(); - LayoutPtr const & lyxlayout = params.textClass()[layout]; - if (lyxlayout->is_environment) { - // move everything in a new environment inset - LYXERR(Debug::DEBUG, "setting layout " << to_utf8(layout)); - lyx::dispatch(FuncRequest(LFUN_LINE_BEGIN)); - lyx::dispatch(FuncRequest(LFUN_LINE_END_SELECT)); - lyx::dispatch(FuncRequest(LFUN_CUT)); - Inset * inset = new InsetEnvironment(params, layout); - insertInset(cur, inset); - //inset->edit(cur, true); - //lyx::dispatch(FuncRequest(LFUN_PASTE)); - return; - } + LASSERT(this == cur.text(), /**/); pit_type start = cur.selBegin().pit(); pit_type end = cur.selEnd().pit() + 1; @@ -244,7 +227,7 @@ void Text::setLayout(Cursor & cur, docstring const & layout) static bool changeDepthAllowed(Text::DEPTH_CHANGE type, Paragraph const & par, int max_depth) { - if (par.layout()->labeltype == LABEL_BIBLIO) + if (par.layout().labeltype == LABEL_BIBLIO) return false; int const depth = par.params().depth(); if (type == Text::INC_DEPTH && depth < max_depth) @@ -257,7 +240,7 @@ static bool changeDepthAllowed(Text::DEPTH_CHANGE type, bool Text::changeDepthAllowed(Cursor & cur, DEPTH_CHANGE type) const { - BOOST_ASSERT(this == cur.text()); + LASSERT(this == cur.text(), /**/); // this happens when selecting several cells in tabular (bug 2630) if (cur.selBegin().idx() != cur.selEnd().idx()) return false; @@ -277,7 +260,7 @@ bool Text::changeDepthAllowed(Cursor & cur, DEPTH_CHANGE type) const void Text::changeDepth(Cursor & cur, DEPTH_CHANGE type) { - BOOST_ASSERT(this == cur.text()); + LASSERT(this == cur.text(), /**/); pit_type const beg = cur.selBegin().pit(); pit_type const end = cur.selEnd().pit() + 1; cur.recordUndoSelection(); @@ -302,15 +285,15 @@ void Text::changeDepth(Cursor & cur, DEPTH_CHANGE type) void Text::setFont(Cursor & cur, Font const & font, bool toggleall) { - BOOST_ASSERT(this == cur.text()); + LASSERT(this == cur.text(), /**/); // Set the current_font // Determine basis font FontInfo layoutfont; pit_type pit = cur.pit(); if (cur.pos() < pars_[pit].beginOfBody()) - layoutfont = getLabelFont(cur.buffer(), pars_[pit]); + layoutfont = labelFont(cur.buffer(), pars_[pit]); else - layoutfont = getLayoutFont(cur.buffer(), pit); + layoutfont = layoutFont(cur.buffer(), pit); // Update current font cur.real_current_font.update(font, @@ -360,7 +343,7 @@ void Text::setFont(BufferView const & bv, CursorSlice const & begin, setInsetFont(bv, pit, pos, font, toggleall); } TextMetrics const & tm = bv.textMetrics(this); - Font f = tm.getDisplayFont(pit, pos); + Font f = tm.displayFont(pit, pos); f.update(font, language, toggleall); setCharFont(buffer, pit, pos, f, tm.font_); } @@ -369,24 +352,23 @@ void Text::setFont(BufferView const & bv, CursorSlice const & begin, bool Text::cursorTop(Cursor & cur) { - BOOST_ASSERT(this == cur.text()); + LASSERT(this == cur.text(), /**/); return setCursor(cur, 0, 0); } bool Text::cursorBottom(Cursor & cur) { - BOOST_ASSERT(this == cur.text()); + LASSERT(this == cur.text(), /**/); return setCursor(cur, cur.lastpit(), boost::prior(paragraphs().end())->size()); } void Text::toggleFree(Cursor & cur, Font const & font, bool toggleall) { - BOOST_ASSERT(this == cur.text()); + LASSERT(this == cur.text(), /**/); // If the mask is completely neutral, tell user - if (font.fontInfo() == ignore_font && - (font.language() == 0 || font.language() == ignore_language)) { + if (font.fontInfo() == ignore_font && font.language() == ignore_language) { // Could only happen with user style cur.message(_("No font change defined.")); return; @@ -416,7 +398,7 @@ void Text::toggleFree(Cursor & cur, Font const & font, bool toggleall) docstring Text::getStringToIndex(Cursor const & cur) { - BOOST_ASSERT(this == cur.text()); + LASSERT(this == cur.text(), /**/); if (cur.selection()) return cur.selectionAsString(false); @@ -440,7 +422,7 @@ docstring Text::getStringToIndex(Cursor const & cur) void Text::setParagraphs(Cursor & cur, docstring arg, bool merge) { - BOOST_ASSERT(cur.text()); + LASSERT(cur.text(), /**/); // make sure that the depth behind the selection are restored, too pit_type undopit = undoSpan(cur.selEnd().pit()); recUndo(cur, cur.selBegin().pit(), undopit - 1); @@ -452,7 +434,7 @@ void Text::setParagraphs(Cursor & cur, docstring arg, bool merge) Paragraph & par = pars_[pit]; ParagraphParameters params = par.params(); params.read(argument, merge); - par.params().apply(params, *par.layout()); + par.params().apply(params, par.layout()); } } @@ -462,7 +444,7 @@ void Text::setParagraphs(Cursor & cur, docstring arg, bool merge) //quite so much. void Text::setParagraphs(Cursor & cur, ParagraphParameters const & p) { - BOOST_ASSERT(cur.text()); + LASSERT(cur.text(), /**/); // make sure that the depth behind the selection are restored, too pit_type undopit = undoSpan(cur.selEnd().pit()); recUndo(cur, cur.selBegin().pit(), undopit - 1); @@ -470,8 +452,7 @@ void Text::setParagraphs(Cursor & cur, ParagraphParameters const & p) for (pit_type pit = cur.selBegin().pit(), end = cur.selEnd().pit(); pit <= end; ++pit) { Paragraph & par = pars_[pit]; - Layout const & layout = *(par.layout()); - par.params().apply(p, layout); + par.params().apply(p, par.layout()); } } @@ -479,8 +460,8 @@ void Text::setParagraphs(Cursor & cur, ParagraphParameters const & p) // this really should just insert the inset and not move the cursor. void Text::insertInset(Cursor & cur, Inset * inset) { - BOOST_ASSERT(this == cur.text()); - BOOST_ASSERT(inset); + LASSERT(this == cur.text(), /**/); + LASSERT(inset, /**/); cur.paragraph().insertInset(cur.pos(), inset, cur.current_font, Change(cur.buffer().params().trackChanges ? Change::INSERTED : Change::UNCHANGED)); @@ -526,7 +507,7 @@ bool Text::setCursor(Cursor & cur, pit_type par, pos_type pos, bool setfont, bool boundary) { TextMetrics const & tm = cur.bv().textMetrics(this); - bool const update_needed = !tm.has(par); + bool const update_needed = !tm.contains(par); Cursor old = cur; setCursorIntern(cur, par, pos, setfont, boundary); return cur.bv().checkDepm(cur, old) || update_needed; @@ -535,7 +516,7 @@ bool Text::setCursor(Cursor & cur, pit_type par, pos_type pos, void Text::setCursor(CursorSlice & cur, pit_type par, pos_type pos) { - BOOST_ASSERT(par != int(paragraphs().size())); + LASSERT(par != int(paragraphs().size()), /**/); cur.pit() = par; cur.pos() = pos; @@ -545,14 +526,14 @@ void Text::setCursor(CursorSlice & cur, pit_type par, pos_type pos) // None of these should happen, but we're scaredy-cats if (pos < 0) { lyxerr << "dont like -1" << endl; - BOOST_ASSERT(false); + LASSERT(false, /**/); } if (pos > para.size()) { lyxerr << "dont like 1, pos: " << pos << " size: " << para.size() << " par: " << par << endl; - BOOST_ASSERT(false); + LASSERT(false, /**/); } } @@ -560,7 +541,7 @@ void Text::setCursor(CursorSlice & cur, pit_type par, pos_type pos) void Text::setCursorIntern(Cursor & cur, pit_type par, pos_type pos, bool setfont, bool boundary) { - BOOST_ASSERT(this == cur.text()); + LASSERT(this == cur.text(), /**/); cur.boundary(boundary); setCursor(cur.top(), par, pos); if (setfont) @@ -655,161 +636,27 @@ bool Text::cursorBackward(Cursor & cur) bool Text::cursorVisLeft(Cursor & cur, bool skip_inset) { - pit_type new_pit = cur.pit(); // the paragraph to which we will move - pos_type new_pos; // the position we will move to - bool new_boundary; // will we move to a boundary position? - pos_type left_pos; // position visually left of current cursor - pos_type right_pos; // position visually right of current cursor - bool new_pos_is_RTL; // is new position we're moving to RTL? - - cur.getSurroundingPos(left_pos, right_pos); - - LYXERR(Debug::RTL, left_pos <<"|"<< right_pos << " (pos: "<= cur.pos(), true)) { - LYXERR(Debug::RTL, "entering inset at: " << temp_cur.pos()); - cur = temp_cur; // set the real cursor to new position inside inset! + temp_cur.posVisLeft(skip_inset); + if (temp_cur.depth() > cur.depth()) { + cur = temp_cur; return false; } - - // Are we already at leftmost pos in row? - if (left_pos == -1) { - - Cursor new_cur = cur; - if (!new_cur.posVisToNewRow(true)) { - LYXERR(Debug::RTL, "not moving!"); - return false; - } - - // we actually move the cursor at the end of this function, for now - // just keep track of the new position... - new_pit = new_cur.pit(); - new_pos = new_cur.pos(); - new_boundary = new_cur.boundary(); - - LYXERR(Debug::RTL, "left edge, moving: " << int(new_pit) << "," - << int(new_pos) << "," << (new_boundary ? 1 : 0)); - - } - // normal movement to the left - else { - // Recall, if the cursor is at position 'x', that means *before* - // the character at position 'x'. In RTL, "before" means "to the - // right of", in LTR, "to the left of". So currently our situation - // is this: the position to our left is 'left_pos' (i.e., we're - // currently to the right of 'left_pos'). In order to move to the - // left, it depends whether or not the character at 'left_pos' is RTL. - new_pos_is_RTL = cur.paragraph().getFontSettings( - cur.bv().buffer().params(), left_pos).isVisibleRightToLeft(); - // If the character at 'left_pos' *is* RTL, then in order to move to - // the left of it, we need to be *after* 'left_pos', i.e., move to - // position 'left_pos' + 1. - if (new_pos_is_RTL) { - new_pos = left_pos + 1; - // if the position *after* left_pos is not RTL, set boundary to - // true (we want to be *after* left_pos, not before left_pos + 1!) - new_boundary = !cur.paragraph().getFontSettings( - cur.bv().buffer().params(), new_pos).isVisibleRightToLeft(); - } - // Otherwise (if the character at position 'left_pos' is LTR), then - // moving to the left of it is as easy as setting the new position - // to 'left_pos'. - else { - new_pos = left_pos; - new_boundary = false; - } - - } - - LYXERR(Debug::RTL, "moving to: " << new_pos - << (new_boundary ? " (boundary)" : "")); - - return setCursor(cur, new_pit, new_pos, true, new_boundary); + return setCursor(cur, temp_cur.pit(), temp_cur.pos(), + true, temp_cur.boundary()); } bool Text::cursorVisRight(Cursor & cur, bool skip_inset) { - pit_type new_pit = cur.pit(); // the paragraph to which we will move - pos_type new_pos; // the position we will move to - bool new_boundary; // will we move to a boundary position? - pos_type left_pos; // position visually left of current cursor - pos_type right_pos; // position visually right of current cursor - bool new_pos_is_RTL; // is new position we're moving to RTL? - - cur.getSurroundingPos(left_pos, right_pos); - - LYXERR(Debug::RTL, left_pos <<"|"<< right_pos << " (pos: "<= cur.pos(), false)) { - LYXERR(Debug::RTL, "entering inset at: " << temp_cur.pos()); - cur = temp_cur; // set the real cursor to new position inside inset! + temp_cur.posVisRight(skip_inset); + if (temp_cur.depth() > cur.depth()) { + cur = temp_cur; return false; } - - // Are we already at rightmost pos in row? - if (right_pos == -1) { - - Cursor new_cur = cur; - if (!new_cur.posVisToNewRow(false)) { - LYXERR(Debug::RTL, "not moving!"); - return false; - } - - // we actually move the cursor at the end of this function, for now - // just keep track of the new position... - new_pit = new_cur.pit(); - new_pos = new_cur.pos(); - new_boundary = new_cur.boundary(); - - LYXERR(Debug::RTL, "right edge, moving: " << int(new_pit) << "," - << int(new_pos) << "," << (new_boundary ? 1 : 0)); - - } - // normal movement to the right - else { - // Recall, if the cursor is at position 'x', that means *before* - // the character at position 'x'. In RTL, "before" means "to the - // right of", in LTR, "to the left of". So currently our situation - // is this: the position to our right is 'right_pos' (i.e., we're - // currently to the left of 'right_pos'). In order to move to the - // right, it depends whether or not the character at 'right_pos' is RTL. - new_pos_is_RTL = cur.paragraph().getFontSettings( - cur.bv().buffer().params(), right_pos).isVisibleRightToLeft(); - // If the character at 'right_pos' *is* LTR, then in order to move to - // the right of it, we need to be *after* 'right_pos', i.e., move to - // position 'right_pos' + 1. - if (!new_pos_is_RTL) { - new_pos = right_pos + 1; - // if the position *after* right_pos is RTL, set boundary to - // true (we want to be *after* right_pos, not before right_pos + 1!) - new_boundary = cur.paragraph().getFontSettings( - cur.bv().buffer().params(), new_pos).isVisibleRightToLeft(); - } - // Otherwise (if the character at position 'right_pos' is RTL), then - // moving to the right of it is as easy as setting the new position - // to 'right_pos'. - else { - new_pos = right_pos; - new_boundary = false; - } - - } - - LYXERR(Debug::RTL, "moving to: " << new_pos - << (new_boundary ? " (boundary)" : "")); - - return setCursor(cur, new_pit, new_pos, true, new_boundary); + return setCursor(cur, temp_cur.pit(), temp_cur.pos(), + true, temp_cur.boundary()); } @@ -1021,7 +868,7 @@ bool Text::deleteEmptyParagraphMechanism(Cursor & cur, void Text::deleteEmptyParagraphMechanism(pit_type first, pit_type last, bool trackChanges) { - BOOST_ASSERT(first >= 0 && first <= last && last < (int) pars_.size()); + LASSERT(first >= 0 && first <= last && last < (int) pars_.size(), /**/); for (pit_type pit = first; pit <= last; ++pit) { Paragraph & par = pars_[pit];