X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Fparagraph_funcs.C;h=95c7c1552633e5d0cd4a93812187742c9499eea2;hb=35204f8f33d7400a5fefeffea533fb4cb4097211;hp=20f9bcb8191f28dc135d050512320206ff827b7a;hpb=6ae4dc23f47b751abcb418094b1bdf31331d239c;p=lyx.git diff --git a/src/paragraph_funcs.C b/src/paragraph_funcs.C index 20f9bcb819..95c7c15526 100644 --- a/src/paragraph_funcs.C +++ b/src/paragraph_funcs.C @@ -12,99 +12,63 @@ #include "paragraph_funcs.h" -#include "buffer.h" #include "bufferparams.h" - -#include "debug.h" -#include "encoding.h" -#include "gettext.h" -#include "language.h" #include "lyxtext.h" -#include "outputparams.h" #include "paragraph_pimpl.h" -#include "pariterator.h" -#include "sgml.h" -#include "texrow.h" -#include "vspace.h" - -#include "support/filetools.h" -#include "support/lstrings.h" -#include "support/lyxlib.h" - -#include -#include - -using lyx::pos_type; -using lyx::par_type; - -using lyx::support::ascii_lowercase; -using lyx::support::bformat; -using lyx::support::compare_ascii_no_case; -using lyx::support::compare_no_case; -using lyx::support::contains; -using lyx::support::split; -using lyx::support::subst; - -using std::auto_ptr; -using std::endl; -using std::string; -using std::vector; -using std::istringstream; -using std::ostream; -using std::pair; -namespace { +namespace lyx { + +using std::string; + -bool moveItem(Paragraph & from, Paragraph & to, - BufferParams const & params, pos_type i, pos_type j) +static bool moveItem(Paragraph & fromPar, pos_type fromPos, + Paragraph & toPar, pos_type toPos, BufferParams const & params) { - Paragraph::value_type const tmpchar = from.getChar(i); - LyXFont tmpfont = from.getFontSettings(params, i); + Paragraph::value_type const tmpChar = fromPar.getChar(fromPos); + LyXFont const tmpFont = fromPar.getFontSettings(params, fromPos); + Change const tmpChange = fromPar.lookupChange(fromPos); - if (tmpchar == Paragraph::META_INSET) { - InsetBase * tmpinset = 0; - if (from.getInset(i)) { + if (tmpChar == Paragraph::META_INSET) { + InsetBase * tmpInset = 0; + if (fromPar.getInset(fromPos)) { // the inset is not in a paragraph anymore - tmpinset = from.insetlist.release(i); - from.insetlist.erase(i); + tmpInset = fromPar.insetlist.release(fromPos); + fromPar.insetlist.erase(fromPos); } - if (!to.insetAllowed(tmpinset->lyxCode())) { - delete tmpinset; + if (!toPar.insetAllowed(tmpInset->lyxCode())) { + delete tmpInset; return false; } - if (tmpinset) - to.insertInset(j, tmpinset, tmpfont); + if (tmpInset) + toPar.insertInset(toPos, tmpInset, tmpFont, tmpChange); } else { - if (!to.checkInsertChar(tmpfont)) - return false; - to.insertChar(j, tmpchar, tmpfont); + toPar.insertChar(toPos, tmpChar, tmpFont, tmpChange); } return true; } -} - void breakParagraph(BufferParams const & bparams, - ParagraphList & pars, par_type par_offset, pos_type pos, int flag) + ParagraphList & pars, pit_type par_offset, pos_type pos, int flag) { // create a new paragraph, and insert into the list ParagraphList::iterator tmp = - pars.insert(pars.begin() + par_offset + 1, Paragraph()); + pars.insert(boost::next(pars.begin(), par_offset + 1), + Paragraph()); Paragraph & par = pars[par_offset]; + // we will invalidate the row cache + par.rows().clear(); + // without doing that we get a crash when typing at the // end of a paragraph tmp->layout(bparams.getLyXTextClass().defaultLayout()); // remember to set the inset_owner tmp->setInsetOwner(par.inInset()); - if (bparams.tracking_changes) - tmp->trackChanges(); - // this is an idea for a more userfriendly layout handling, I will // see what the users say @@ -112,6 +76,13 @@ void breakParagraph(BufferParams const & bparams, if (flag) { tmp->layout(par.layout()); tmp->setLabelWidthString(par.params().labelWidthString()); + tmp->params().depth(par.params().depth()); + } else if (par.params().depth() > 0) { + Paragraph const & hook = pars[outerHook(par_offset, pars)]; + tmp->layout(hook.layout()); + // not sure the line below is useful + tmp->setLabelWidthString(par.params().labelWidthString()); + tmp->params().depth(hook.params().depth()); } bool const isempty = (par.allowEmpty() && par.empty()); @@ -135,22 +106,21 @@ void breakParagraph(BufferParams const & bparams, pos_type pos_end = par.size() - 1; for (pos_type i = pos, j = pos; i <= pos_end; ++i) { - Change::Type change = par.lookupChange(i); - if (moveItem(par, *tmp, bparams, i, j - pos)) { - tmp->setChange(j - pos, change); + if (moveItem(par, i, *tmp, j - pos, bparams)) { ++j; } } for (pos_type i = pos_end; i >= pos; --i) - par.eraseIntern(i); + // FIXME: change tracking (MG) + par.eraseChar(i, false); // erase without change tracking } if (pos) { // Make sure that we keep the language when // breaking paragrpah. if (tmp->empty()) { - LyXFont changed = tmp->getFirstFontSettings(); + LyXFont changed = tmp->getFirstFontSettings(bparams); LyXFont old = par.getFontSettings(bparams, par.size()); changed.setLanguage(old.language()); tmp->setFont(0, changed); @@ -159,9 +129,10 @@ void breakParagraph(BufferParams const & bparams, return; } - par.params().clear(); - - par.layout(bparams.getLyXTextClass().defaultLayout()); + if (!isempty) { + par.params().clear(); + par.layout(bparams.getLyXTextClass().defaultLayout()); + } // layout stays the same with latex-environments if (flag) { @@ -171,21 +142,22 @@ void breakParagraph(BufferParams const & bparams, } // subtle, but needed to get empty pars working right - if (bparams.tracking_changes) { - if (!par.size()) { - par.cleanChanges(); - } else if (!tmp->size()) { - tmp->cleanChanges(); - } + if (bparams.trackChanges) { + // FIXME: Change tracking (MG) + // if (!par.size()) + // set 'par' text to INSERTED in CT mode; clear CT info otherwise + // else if (!tmp->size()) + // set 'tmp' text to INSERTED in CT mode; clear CT info otherwise } } void breakParagraphConservative(BufferParams const & bparams, - ParagraphList & pars, par_type par_offset, pos_type pos) + ParagraphList & pars, pit_type par_offset, pos_type pos) { // create a new paragraph - Paragraph & tmp = *pars.insert(pars.begin() + par_offset + 1, Paragraph()); + Paragraph & tmp = *pars.insert(boost::next(pars.begin(), par_offset + 1), + Paragraph()); Paragraph & par = pars[par_offset]; tmp.makeSameLayout(par); @@ -197,18 +169,30 @@ void breakParagraphConservative(BufferParams const & bparams, // paragraph pos_type pos_end = par.size() - 1; - for (pos_type i = pos, j = pos; i <= pos_end; ++i) - if (moveItem(par, tmp, bparams, i, j - pos)) + for (pos_type i = pos, j = pos; i <= pos_end; ++i) { + if (moveItem(par, i, tmp, j - pos, bparams)) { ++j; - - for (pos_type k = pos_end; k >= pos; --k) - par.erase(k); + } + } + // Move over end-of-par change attr + // FIXME: change tracking (MG) + tmp.setChange(tmp.size(), Change(par.lookupChange(par.size()).type)); + + // If tracking changes, set all the text that is to be + // erased to Change::INSERTED. + for (pos_type k = pos_end; k >= pos; --k) { + if (bparams.trackChanges) + // FIXME: Change tracking (MG) + par.setChange(k, Change(Change::INSERTED)); + // FIXME: change tracking (MG) + par.eraseChar(k, false); + } } } void mergeParagraph(BufferParams const & bparams, - ParagraphList & pars, par_type par_offset) + ParagraphList & pars, pit_type par_offset) { Paragraph & next = pars[par_offset + 1]; Paragraph & par = pars[par_offset]; @@ -216,19 +200,31 @@ void mergeParagraph(BufferParams const & bparams, pos_type pos_end = next.size() - 1; pos_type pos_insert = par.size(); + // The imaginary end-of-paragraph character (at par.size()) has to be + // marked as unmodified. Otherwise, its change is adopted by the first + // character of the next paragraph. + + // FIXME: change tracking (MG) + par.setChange(par.size(), Change(Change::UNCHANGED)); + + Change change = next.lookupChange(next.size()); // ok, now copy the paragraph - for (pos_type i = 0, j = 0; i <= pos_end; ++i) - if (moveItem(next, par, bparams, i, pos_insert + j)) + for (pos_type i = 0, j = 0; i <= pos_end; ++i) { + if (moveItem(next, i, par, pos_insert + j, bparams)) { ++j; + } + } + // Move the change of the end-of-paragraph character + // FIXME: change tracking (MG) + par.setChange(par.size(), change); - pars.erase(pars.begin() + par_offset + 1); + pars.erase(boost::next(pars.begin(), par_offset + 1)); } -par_type depthHook(par_type pit, - ParagraphList const & pars, Paragraph::depth_type depth) +pit_type depthHook(pit_type pit, ParagraphList const & pars, depth_type depth) { - par_type newpit = pit; + pit_type newpit = pit; if (newpit != 0) --newpit; @@ -243,21 +239,21 @@ par_type depthHook(par_type pit, } -par_type outerHook(par_type par_offset, ParagraphList const & pars) +pit_type outerHook(pit_type par_offset, ParagraphList const & pars) { Paragraph const & par = pars[par_offset]; if (par.getDepth() == 0) return pars.size(); - return depthHook(par_offset, pars, Paragraph::depth_type(par.getDepth() - 1)); + return depthHook(par_offset, pars, depth_type(par.getDepth() - 1)); } -bool isFirstInSequence(par_type par_offset, ParagraphList const & pars) +bool isFirstInSequence(pit_type par_offset, ParagraphList const & pars) { Paragraph const & par = pars[par_offset]; - par_type dhook_offset = depthHook(par_offset, pars, par.getDepth()); + pit_type dhook_offset = depthHook(par_offset, pars, par.getDepth()); Paragraph const & dhook = pars[dhook_offset]; @@ -267,19 +263,19 @@ bool isFirstInSequence(par_type par_offset, ParagraphList const & pars) } -int getEndLabel(par_type p, ParagraphList const & pars) +int getEndLabel(pit_type p, ParagraphList const & pars) { - par_type pit = p; - Paragraph::depth_type par_depth = pars[p].getDepth(); - while (pit != par_type(pars.size())) { + pit_type pit = p; + depth_type par_depth = pars[p].getDepth(); + while (pit != pit_type(pars.size())) { LyXLayout_ptr const & layout = pars[pit].layout(); int const endlabeltype = layout->endlabeltype; if (endlabeltype != END_LABEL_NO_LABEL) { - if (p + 1 == par_type(pars.size())) + if (p + 1 == pit_type(pars.size())) return endlabeltype; - Paragraph::depth_type const next_depth = + depth_type const next_depth = pars[p + 1].getDepth(); if (par_depth > next_depth || (par_depth == next_depth && layout != pars[p + 1].layout())) @@ -289,24 +285,24 @@ int getEndLabel(par_type p, ParagraphList const & pars) if (par_depth == 0) break; pit = outerHook(pit, pars); - if (pit != par_type(pars.size())) + if (pit != pit_type(pars.size())) par_depth = pars[pit].getDepth(); } return END_LABEL_NO_LABEL; } -LyXFont const outerFont(par_type par_offset, ParagraphList const & pars) +LyXFont const outerFont(pit_type par_offset, ParagraphList const & pars) { - Paragraph::depth_type par_depth = pars[par_offset].getDepth(); + depth_type par_depth = pars[par_offset].getDepth(); LyXFont tmpfont(LyXFont::ALL_INHERIT); // Resolve against environment font information - while (par_offset != par_type(pars.size()) + while (par_offset != pit_type(pars.size()) && par_depth && !tmpfont.resolved()) { par_offset = outerHook(par_offset, pars); - if (par_offset != par_type(pars.size())) { + if (par_offset != pit_type(pars.size())) { tmpfont.realize(pars[par_offset].layout()->font); par_depth = pars[par_offset].getDepth(); } @@ -316,57 +312,19 @@ LyXFont const outerFont(par_type par_offset, ParagraphList const & pars) } -par_type outerPar(Buffer const & buf, InsetBase const * inset) -{ - ParIterator pit = const_cast(buf).par_iterator_begin(); - ParIterator end = const_cast(buf).par_iterator_end(); - for ( ; pit != end; ++pit) { - LyXText * text; - // the second '=' below is intentional - for (int i = 0; (text = inset->getText(i)); ++i) - if (&text->paragraphs() == &pit.plist()) - return pit.outerPar(); - - InsetList::const_iterator ii = pit->insetlist.begin(); - InsetList::const_iterator iend = pit->insetlist.end(); - for ( ; ii != iend; ++ii) - if (ii->inset == inset) - return pit.outerPar(); - } - lyxerr << "outerPar: should not happen" << endl; - BOOST_ASSERT(false); - return buf.paragraphs().size(); // shut up compiler -} - - -/// return the range of pars [beg, end[ owning the range of y [ystart, yend] -void getParsInRange(ParagraphList & pars, int ystart, int yend, - par_type & beg, par_type & end) -{ - BOOST_ASSERT(!pars.empty()); - par_type const endpar = pars.size(); - par_type const begpar = 0; - - for (beg = endpar - 1; beg != begpar && pars[beg].y > ystart; --beg) - ; - - for (end = beg ; end != endpar && pars[end].y <= yend; ++end) - ; -} - - /// return the number of InsetOptArg in a paragraph int numberOfOptArgs(Paragraph const & par) { int num = 0; - + InsetList::const_iterator it = par.insetlist.begin(); InsetList::const_iterator end = par.insetlist.end(); for (; it != end ; ++it) { - if (it->inset->lyxCode() == InsetBase::OPTARG_CODE) + if (it->inset->lyxCode() == InsetBase::OPTARG_CODE) ++num; } return num; } +} // namespace lyx