X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FText.cpp;h=932a3929d79bb4da2965b8cad4e1d8d671d9abe5;hb=f08f5821ec5d29232bccb342a5b1ff86600b0716;hp=fbdb2e7eccca0e84cf8b09d7c15731287356af30;hpb=5fd5cf41c0a79919d219c73193de54b57d2b1f68;p=lyx.git diff --git a/src/Text.cpp b/src/Text.cpp index fbdb2e7ecc..932a3929d7 100644 --- a/src/Text.cpp +++ b/src/Text.cpp @@ -30,7 +30,6 @@ #include "Cursor.h" #include "CutAndPaste.h" #include "DispatchResult.h" -#include "Encoding.h" #include "ErrorList.h" #include "FuncRequest.h" #include "factory.h" @@ -46,21 +45,21 @@ #include "ParIterator.h" #include "TextClass.h" #include "TextMetrics.h" -#include "VSpace.h" #include "WordLangTuple.h" #include "WordList.h" #include "insets/InsetText.h" #include "insets/InsetBibitem.h" #include "insets/InsetCaption.h" -#include "insets/InsetLine.h" #include "insets/InsetNewline.h" #include "insets/InsetNewpage.h" -#include "insets/InsetOptArg.h" +#include "insets/InsetArgument.h" +#include "insets/InsetIPAMacro.h" #include "insets/InsetSpace.h" #include "insets/InsetSpecialChar.h" #include "insets/InsetTabular.h" +#include "support/convert.h" #include "support/debug.h" #include "support/docstream.h" #include "support/gettext.h" @@ -70,8 +69,34 @@ #include +#include #include + +// TODO: replace if in Text::readParToken() with compile time switch +#if 0 + +#include "support/metahash.h" + +typedef boost::mpl::string<'\\end','_lay','out'> end_layout; +typedef boost::mpl::string<'\\end','in','set'> end_inset; + +void foo() +{ + std::string token = "\\end_layout"; + + switch (boost::hash_value(token)) { + case lyx::support::hash_string::value: + return; + case lyx::support::hash_string::value: + return; + default: ; + }; + +} +#endif + + using namespace std; using namespace lyx::support; @@ -171,91 +196,100 @@ void mergeParagraph(BufferParams const & bparams, } -pit_type depthHook(pit_type pit, ParagraphList const & pars, depth_type depth) +Text::Text(InsetText * owner, bool use_default_layout) + : owner_(owner), autoBreakRows_(false), undo_counter_(0) +{ + pars_.push_back(Paragraph()); + Paragraph & par = pars_.back(); + par.setInsetOwner(owner); + DocumentClass const & dc = owner->buffer().params().documentClass(); + if (use_default_layout) + par.setDefaultLayout(dc); + else + par.setPlainLayout(dc); +} + + +Text::Text(InsetText * owner, Text const & text) + : owner_(owner), autoBreakRows_(text.autoBreakRows_), undo_counter_(0) +{ + pars_ = text.pars_; + ParagraphList::iterator const end = pars_.end(); + ParagraphList::iterator it = pars_.begin(); + for (; it != end; ++it) + it->setInsetOwner(owner); +} + + +pit_type Text::depthHook(pit_type pit, depth_type depth) const { pit_type newpit = pit; if (newpit != 0) --newpit; - while (newpit != 0 && pars[newpit].getDepth() > depth) + while (newpit != 0 && pars_[newpit].getDepth() > depth) --newpit; - if (pars[newpit].getDepth() > depth) + if (pars_[newpit].getDepth() > depth) return pit; return newpit; } -pit_type outerHook(pit_type par_offset, ParagraphList const & pars) +pit_type Text::outerHook(pit_type par_offset) const { - Paragraph const & par = pars[par_offset]; + Paragraph const & par = pars_[par_offset]; if (par.getDepth() == 0) - return pars.size(); - return depthHook(par_offset, pars, depth_type(par.getDepth() - 1)); + return pars_.size(); + return depthHook(par_offset, depth_type(par.getDepth() - 1)); } -bool isFirstInSequence(pit_type par_offset, ParagraphList const & pars) +bool Text::isFirstInSequence(pit_type par_offset) const { - Paragraph const & par = pars[par_offset]; + Paragraph const & par = pars_[par_offset]; - pit_type dhook_offset = depthHook(par_offset, pars, par.getDepth()); + pit_type dhook_offset = depthHook(par_offset, par.getDepth()); if (dhook_offset == par_offset) return true; - Paragraph const & dhook = pars[dhook_offset]; + Paragraph const & dhook = pars_[dhook_offset]; return dhook.layout() != par.layout() || dhook.getDepth() != par.getDepth(); } -int getEndLabel(pit_type p, ParagraphList const & pars) +int Text::getTocLevel(pit_type par_offset) const { - pit_type pit = p; - depth_type par_depth = pars[p].getDepth(); - while (pit != pit_type(pars.size())) { - Layout const & layout = pars[pit].layout(); - int const endlabeltype = layout.endlabeltype; - - if (endlabeltype != END_LABEL_NO_LABEL) { - if (p + 1 == pit_type(pars.size())) - return endlabeltype; - - depth_type const next_depth = - pars[p + 1].getDepth(); - if (par_depth > next_depth || - (par_depth == next_depth && layout != pars[p + 1].layout())) - return endlabeltype; - break; - } - if (par_depth == 0) - break; - pit = outerHook(pit, pars); - if (pit != pit_type(pars.size())) - par_depth = pars[pit].getDepth(); - } - return END_LABEL_NO_LABEL; + Paragraph const & par = pars_[par_offset]; + + if (par.layout().isEnvironment() && !isFirstInSequence(par_offset)) + return Layout::NOT_IN_TOC; + + return par.layout().toclevel; } -Font const outerFont(pit_type par_offset, ParagraphList const & pars) +Font const Text::outerFont(pit_type par_offset) const { - depth_type par_depth = pars[par_offset].getDepth(); + depth_type par_depth = pars_[par_offset].getDepth(); FontInfo tmpfont = inherit_font; - + depth_type prev_par_depth = 0; // Resolve against environment font information - while (par_offset != pit_type(pars.size()) + while (par_offset != pit_type(pars_.size()) + && par_depth != prev_par_depth && par_depth && !tmpfont.resolved()) { - par_offset = outerHook(par_offset, pars); - if (par_offset != pit_type(pars.size())) { - tmpfont.realize(pars[par_offset].layout().font); - par_depth = pars[par_offset].getDepth(); + prev_par_depth = par_depth; + par_offset = outerHook(par_offset); + if (par_offset != pit_type(pars_.size())) { + tmpfont.realize(pars_[par_offset].layout().font); + par_depth = pars_[par_offset].getDepth(); } } @@ -263,48 +297,71 @@ Font const outerFont(pit_type par_offset, ParagraphList const & pars) } -void acceptChanges(ParagraphList & pars, BufferParams const & bparams) +static void acceptOrRejectChanges(ParagraphList & pars, + BufferParams const & bparams, Text::ChangeOp op) { pit_type pars_size = static_cast(pars.size()); - // first, accept changes within each individual paragraph - // (do not consider end-of-par) + // first, accept or reject changes within each individual + // paragraph (do not consider end-of-par) for (pit_type pit = 0; pit < pars_size; ++pit) { - if (!pars[pit].empty()) // prevent assertion failure - pars[pit].acceptChanges(0, pars[pit].size()); + // prevent assertion failure + if (!pars[pit].empty()) { + if (op == Text::ACCEPT) + pars[pit].acceptChanges(0, pars[pit].size()); + else + pars[pit].rejectChanges(0, pars[pit].size()); + } } - // next, accept imaginary end-of-par characters + // next, accept or reject imaginary end-of-par characters for (pit_type pit = 0; pit < pars_size; ++pit) { pos_type pos = pars[pit].size(); - - if (pars[pit].isInserted(pos)) { - pars[pit].setChange(pos, Change(Change::UNCHANGED)); - } else if (pars[pit].isDeleted(pos)) { - if (pit == pars_size - 1) { - // we cannot remove a par break at the end of the last - // paragraph; instead, we mark it unchanged + if (pars[pit].isChanged(pos)) { + // keep the end-of-par char if it is inserted and accepted + // or when it is deleted and rejected. + if (pars[pit].isInserted(pos) == (op == Text::ACCEPT)) { pars[pit].setChange(pos, Change(Change::UNCHANGED)); } else { - mergeParagraph(bparams, pars, pit); - --pit; - --pars_size; + if (pit == pars_size - 1) { + // we cannot remove a par break at the end of the last + // paragraph; instead, we mark it unchanged + pars[pit].setChange(pos, Change(Change::UNCHANGED)); + } else { + mergeParagraph(bparams, pars, pit); + --pit; + --pars_size; + } } } } } + +void acceptChanges(ParagraphList & pars, BufferParams const & bparams) +{ + acceptOrRejectChanges(pars, bparams, Text::ACCEPT); +} + + +void rejectChanges(ParagraphList & pars, BufferParams const & bparams) +{ + acceptOrRejectChanges(pars, bparams, Text::REJECT); +} + + InsetText const & Text::inset() const { return *owner_; } + void Text::readParToken(Paragraph & par, Lexer & lex, string const & token, Font & font, Change & change, ErrorList & errorList) { - Buffer const & buf = owner_->buffer(); - BufferParams const & bp = buf.params(); + Buffer * buf = const_cast(&owner_->buffer()); + BufferParams const & bp = buf->params(); if (token[0] != '\\') { docstring dstr = lex.getDocString(); @@ -339,7 +396,13 @@ void Text::readParToken(Paragraph & par, Lexer & lex, // of this document. For example, when you apply class article to a beamer document, // all unknown layouts such as frame will be added to document class article so that // these layouts can keep their original names. - tclass.addLayoutIfNeeded(layoutname); + bool const added_one = tclass.addLayoutIfNeeded(layoutname); + if (added_one) { + // Warn the user. + docstring const s = bformat(_("Layout `%1$s' was not found."), layoutname); + errorList.push_back( + ErrorItem(_("Layout Not Found"), s, par.id(), 0, par.size())); + } par.setLayout(bp.documentClass()[layoutname]); @@ -390,10 +453,10 @@ void Text::readParToken(Paragraph & par, Lexer & lex, } } else if (token == "\\numeric") { lex.next(); - font.fontInfo().setNumber(font.setLyXMisc(lex.getString())); + font.fontInfo().setNumber(setLyXMisc(lex.getString())); } else if (token == "\\emph") { lex.next(); - font.fontInfo().setEmph(font.setLyXMisc(lex.getString())); + font.fontInfo().setEmph(setLyXMisc(lex.getString())); } else if (token == "\\bar") { lex.next(); string const tok = lex.getString(); @@ -409,42 +472,46 @@ void Text::readParToken(Paragraph & par, Lexer & lex, "`$$Token'"); } else if (token == "\\strikeout") { lex.next(); - font.fontInfo().setStrikeout(font.setLyXMisc(lex.getString())); + font.fontInfo().setStrikeout(setLyXMisc(lex.getString())); } else if (token == "\\uuline") { lex.next(); - font.fontInfo().setUuline(font.setLyXMisc(lex.getString())); + font.fontInfo().setUuline(setLyXMisc(lex.getString())); } else if (token == "\\uwave") { lex.next(); - font.fontInfo().setUwave(font.setLyXMisc(lex.getString())); + font.fontInfo().setUwave(setLyXMisc(lex.getString())); } else if (token == "\\noun") { lex.next(); - font.fontInfo().setNoun(font.setLyXMisc(lex.getString())); + font.fontInfo().setNoun(setLyXMisc(lex.getString())); } else if (token == "\\color") { lex.next(); setLyXColor(lex.getString(), font.fontInfo()); } else if (token == "\\SpecialChar") { - auto_ptr inset; - inset.reset(new InsetSpecialChar); - inset->read(lex); - par.insertInset(par.size(), inset.release(), - font, change); + auto_ptr inset; + inset.reset(new InsetSpecialChar); + inset->read(lex); + inset->setBuffer(*buf); + par.insertInset(par.size(), inset.release(), font, change); + } else if (token == "\\IPAChar") { + auto_ptr inset; + inset.reset(new InsetIPAChar); + inset->read(lex); + inset->setBuffer(*buf); + par.insertInset(par.size(), inset.release(), font, change); } else if (token == "\\backslash") { par.appendChar('\\', font, change); } else if (token == "\\LyXTable") { - auto_ptr inset(new InsetTabular(const_cast(buf))); + auto_ptr inset(new InsetTabular(buf)); inset->read(lex); par.insertInset(par.size(), inset.release(), font, change); - } else if (token == "\\lyxline") { - par.insertInset(par.size(), new InsetLine, font, change); } else if (token == "\\change_unchanged") { change = Change(Change::UNCHANGED); } else if (token == "\\change_inserted" || token == "\\change_deleted") { lex.eatLine(); istringstream is(lex.getString()); - unsigned int aid; + int aid; time_t ct; is >> aid >> ct; - map const & am = bp.author_map; + BufferParams::AuthorMap const & am = bp.author_map; if (am.find(aid) == am.end()) { errorList.push_back(ErrorItem(_("Change tracking error"), bformat(_("Unknown author index for change: %1$d\n"), aid), @@ -504,6 +571,9 @@ void Text::readParagraph(Paragraph & par, Lexer & lex, // Initialize begin_of_body_ on load; redoParagraph maintains par.setBeginOfBody(); + + // mark paragraph for spell checking on load + // par.requestSpellCheck(); } @@ -511,8 +581,8 @@ class TextCompletionList : public CompletionList { public: /// - TextCompletionList(Cursor const & cur) - : buffer_(cur.buffer()), pos_(0) + TextCompletionList(Cursor const & cur, WordList const * list) + : buffer_(cur.buffer()), pos_(0), list_(list) {} /// virtual ~TextCompletionList() {} @@ -522,12 +592,12 @@ public: /// virtual size_t size() const { - return theWordList().size(); + return list_->size(); } /// virtual docstring const & data(size_t idx) const { - return theWordList().word(idx); + return list_->word(idx); } private: @@ -535,6 +605,8 @@ private: Buffer const * buffer_; /// size_t pos_; + /// + WordList const * list_; }; @@ -561,10 +633,11 @@ double Text::spacing(Paragraph const & par) const * keep_layout == false * - keep current depth and layout when keep_layout == true */ -static void breakParagraph(BufferParams const & bparams, - ParagraphList & pars, pit_type par_offset, pos_type pos, +static void breakParagraph(Text & text, pit_type par_offset, pos_type pos, bool keep_layout) { + BufferParams const & bparams = text.inset().buffer().params(); + ParagraphList & pars = text.paragraphs(); // create a new paragraph, and insert into the list ParagraphList::iterator tmp = pars.insert(boost::next(pars.begin(), par_offset + 1), @@ -578,13 +651,12 @@ static void breakParagraph(BufferParams const & bparams, // end of a paragraph tmp->setPlainOrDefaultLayout(bparams.documentClass()); - // layout stays the same with latex-environments if (keep_layout) { tmp->setLayout(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)]; + Paragraph const & hook = pars[text.outerHook(par_offset)]; tmp->setLayout(hook.layout()); // not sure the line below is useful tmp->setLabelWidthString(par.params().labelWidthString()); @@ -644,7 +716,6 @@ static void breakParagraph(BufferParams const & bparams, par.setPlainOrDefaultLayout(bparams.documentClass()); } - // layout stays the same with latex-environments if (keep_layout) { par.setLayout(tmp->layout()); par.setLabelWidthString(tmp->params().labelWidthString()); @@ -663,14 +734,15 @@ void Text::breakParagraph(Cursor & cur, bool inverse_logic) DocumentClass const & tclass = cur.buffer()->params().documentClass(); Layout const & layout = cpar.layout(); - // this is only allowed, if the current paragraph is not empty - // or caption and if it has not the keepempty flag active - if (cur.lastpos() == 0 && !cpar.allowEmpty() && - layout.labeltype != LABEL_SENSITIVE) + if (cur.lastpos() == 0 && !cpar.allowEmpty()) { + if (changeDepthAllowed(cur, DEC_DEPTH)) + changeDepth(cur, DEC_DEPTH); + else + setLayout(cur, tclass.defaultLayoutName()); return; + } - // a layout change may affect also the following paragraph - recUndo(cur, cur.pit(), undoSpan(cur.pit()) - 1); + cur.recordUndo(); // Always break behind a space // It is better to erase the space (Dekel) @@ -678,9 +750,10 @@ void Text::breakParagraph(Cursor & cur, bool inverse_logic) cpar.eraseChar(cur.pos(), cur.buffer()->params().trackChanges); // What should the layout for the new paragraph be? - bool keep_layout = inverse_logic ? - !layout.isEnvironment() - : layout.isEnvironment(); + bool keep_layout = layout.isEnvironment() + || (layout.isParagraph() && layout.parbreak_is_newline); + if (inverse_logic) + keep_layout = !keep_layout; // We need to remember this before we break the paragraph, because // that invalidates the layout variable @@ -689,8 +762,7 @@ void Text::breakParagraph(Cursor & cur, bool inverse_logic) // we need to set this before we insert the paragraph. bool const isempty = cpar.allowEmpty() && cpar.empty(); - lyx::breakParagraph(cur.buffer()->params(), paragraphs(), cpit, - cur.pos(), keep_layout); + lyx::breakParagraph(*this, cpit, cur.pos(), keep_layout); // After this, neither paragraph contains any rows! @@ -714,10 +786,9 @@ void Text::breakParagraph(Cursor & cur, bool inverse_logic) break; // the character couldn't be deleted physically due to change tracking } - cur.buffer()->updateLabels(); - // A singlePar update is not enough in this case. - cur.updateFlags(Update::Force); + cur.screenUpdateFlags(Update::Force); + cur.forceBufferUpdate(); // This check is necessary. Otherwise the new empty paragraph will // be deleted automatically. And it is more friendly for the user! @@ -729,12 +800,12 @@ void Text::breakParagraph(Cursor & cur, bool inverse_logic) // needed to insert the selection -void Text::insertStringAsLines(DocIterator const & dit, docstring const & str, +void Text::insertStringAsLines(Cursor & cur, docstring const & str, Font const & font) { BufferParams const & bparams = owner_->buffer().params(); - pit_type pit = dit.pit(); - pos_type pos = dit.pos(); + pit_type pit = cur.pit(); + pos_type pos = cur.pos(); // insert the string, don't insert doublespace bool space_inserted = true; @@ -743,8 +814,8 @@ void Text::insertStringAsLines(DocIterator const & dit, docstring const & str, Paragraph & par = pars_[pit]; if (*cit == '\n') { if (autoBreakRows_ && (!par.empty() || par.allowEmpty())) { - lyx::breakParagraph(bparams, pars_, pit, pos, - par.layout().isEnvironment()); + lyx::breakParagraph(*this, pit, pos, + par.layout().isEnvironment()); ++pit; pos = 0; space_inserted = true; @@ -776,12 +847,13 @@ void Text::insertStringAsLines(DocIterator const & dit, docstring const & str, space_inserted = (*cit == ' '); } } + setCursor(cur, pit, pos); } // turn double CR to single CR, others are converted into one // blank. Then insertStringAsLines is called -void Text::insertStringAsParagraphs(DocIterator const & dit, docstring const & str, +void Text::insertStringAsParagraphs(Cursor & cur, docstring const & str, Font const & font) { docstring linestr = str; @@ -803,7 +875,7 @@ void Text::insertStringAsParagraphs(DocIterator const & dit, docstring const & s newline_inserted = false; } } - insertStringAsLines(dit, linestr, font); + insertStringAsLines(cur, linestr, font); } @@ -830,7 +902,7 @@ void Text::insertChar(Cursor & cur, char_type c) static docstring const number_seperators = from_ascii(".,:"); if (cur.current_font.fontInfo().number() == FONT_ON) { - if (!isDigit(c) && !contains(number_operators, c) && + if (!isDigitASCII(c) && !contains(number_operators, c) && !(contains(number_seperators, c) && cur.pos() != 0 && cur.pos() != cur.lastpos() && @@ -838,7 +910,7 @@ void Text::insertChar(Cursor & cur, char_type c) tm.displayFont(pit, cur.pos() - 1).fontInfo().number() == FONT_ON) ) number(cur); // Set current_font.number to OFF - } else if (isDigit(c) && + } else if (isDigitASCII(c) && cur.real_current_font.isVisibleRightToLeft()) { number(cur); // Set current_font.number to ON @@ -915,23 +987,17 @@ void Text::insertChar(Cursor & cur, char_type c) // disable the double-space checking if (!freeSpacing && isLineSeparatorChar(c)) { if (cur.pos() == 0) { - static bool sent_space_message = false; - if (!sent_space_message) { - cur.message(_("You cannot insert a space at the " - "beginning of a paragraph. Please read the Tutorial.")); - sent_space_message = true; - } + cur.message(_( + "You cannot insert a space at the " + "beginning of a paragraph. Please read the Tutorial.")); return; } LASSERT(cur.pos() > 0, /**/); if ((par.isLineSeparator(cur.pos() - 1) || par.isNewline(cur.pos() - 1)) - && !par.isDeleted(cur.pos() - 1)) { - static bool sent_space_message = false; - if (!sent_space_message) { - cur.message(_("You cannot type two spaces this way. " - "Please read the Tutorial.")); - sent_space_message = true; - } + && !par.isDeleted(cur.pos() - 1)) { + cur.message(_( + "You cannot type two spaces this way. " + "Please read the Tutorial.")); return; } } @@ -940,8 +1006,9 @@ void Text::insertChar(Cursor & cur, char_type c) cur.buffer()->params().trackChanges); cur.checkBufferStructure(); -// cur.updateFlags(Update::Force); - bool boundary = tm.isRTLBoundary(cur.pit(), cur.pos() + 1); +// cur.screenUpdateFlags(Update::Force); + bool boundary = cur.boundary() + || tm.isRTLBoundary(cur.pit(), cur.pos() + 1); setCursor(cur, cur.pit(), cur.pos() + 1, false, boundary); charInserted(cur); } @@ -953,12 +1020,11 @@ void Text::charInserted(Cursor & cur) // Here we call finishUndo for every 20 characters inserted. // This is from my experience how emacs does it. (Lgb) - static unsigned int counter; - if (counter < 20) { - ++counter; + if (undo_counter_ < 20) { + ++undo_counter_; } else { cur.finishUndo(); - counter = 0; + undo_counter_ = 0; } // register word if a non-letter was entered @@ -1149,9 +1215,11 @@ void Text::selectWord(Cursor & cur, word_location loc) setCursor(cur, from.pit(), from.pos()); if (to == from) return; - cur.resetAnchor(); + if (!cur.selection()) + cur.resetAnchor(); setCursor(cur, to.pit(), to.pos()); cur.setSelection(); + cur.setWordSelection(true); } @@ -1227,7 +1295,11 @@ void Text::acceptOrRejectChanges(Cursor & cur, ChangeOp op) pos_type left = (pit == begPit ? begPos : 0); pos_type right = (pit == endPit ? endPos : parSize); - + + if (left == right) + // there is no change here + continue; + if (op == ACCEPT) { pars_[pit].acceptChanges(left, right); } else { @@ -1288,8 +1360,8 @@ void Text::acceptOrRejectChanges(Cursor & cur, ChangeOp op) cur.finishUndo(); cur.clearSelection(); setCursorIntern(cur, begPit, begPos); - cur.updateFlags(Update::Force); - cur.buffer()->updateLabels(); + cur.screenUpdateFlags(Update::Force); + cur.forceBufferUpdate(); } @@ -1440,9 +1512,9 @@ bool Text::handleBibitems(Cursor & cur) cur.recordUndo(ATOMIC_UNDO, prevcur.pit()); mergeParagraph(bufparams, cur.text()->paragraphs(), prevcur.pit()); - cur.buffer()->updateLabels(); + cur.forceBufferUpdate(); setCursorIntern(cur, prevcur.pit(), prevcur.pos()); - cur.updateFlags(Update::Force); + cur.screenUpdateFlags(Update::Force); return true; } @@ -1468,7 +1540,7 @@ bool Text::erase(Cursor & cur) cur.top().forwardPos(); if (was_inset) - cur.buffer()->updateLabels(); + cur.forceBufferUpdate(); else cur.checkBufferStructure(); needsUpdate = true; @@ -1544,7 +1616,7 @@ bool Text::backspacePos0(Cursor & cur) } if (needsUpdate) { - cur.buffer()->updateLabels(); + cur.forceBufferUpdate(); setCursorIntern(cur, prevcur.pit(), prevcur.pos()); } @@ -1584,7 +1656,7 @@ bool Text::backspace(Cursor & cur) bool const was_inset = cur.paragraph().isInset(cur.pos()); cur.paragraph().eraseChar(cur.pos(), cur.buffer()->params().trackChanges); if (was_inset) - cur.buffer()->updateLabels(); + cur.forceBufferUpdate(); else cur.checkBufferStructure(); } @@ -1595,7 +1667,7 @@ bool Text::backspace(Cursor & cur) needsUpdate |= handleBibitems(cur); // A singlePar update is not enough in this case. -// cur.updateFlags(Update::Force); +// cur.screenUpdateFlags(Update::Force); setCursor(cur.top(), cur.pit(), cur.pos()); return needsUpdate; @@ -1625,13 +1697,23 @@ bool Text::dissolveInset(Cursor & cur) spit += cur.pit(); Buffer & b = *cur.buffer(); cur.paragraph().eraseChar(cur.pos(), b.params().trackChanges); + if (!plist.empty()) { + // see bug 7319 + // we clear the cache so that we won't get conflicts with labels + // that get pasted into the buffer. we should update this before + // its being empty matters. if not (i.e., if we encounter bugs), + // then this should instead be: + // cur.buffer().updateBuffer(); + // but we'll try the cheaper solution here. + cur.buffer()->clearReferenceCache(); + // ERT paragraphs have the Language latex_language. // This is invalid outside of ERT, so we need to // change it to the buffer language. ParagraphList::iterator it = plist.begin(); ParagraphList::iterator it_end = plist.end(); - for (; it != it_end; it++) + for (; it != it_end; ++it) it->changeLanguage(b.params(), latex_language, b.language()); pasteParagraphList(cur, plist, b.params().documentClassPtr(), @@ -1639,10 +1721,12 @@ bool Text::dissolveInset(Cursor & cur) // restore position cur.pit() = min(cur.lastpit(), spit); cur.pos() = min(cur.lastpos(), spos); - } else - // this is the least that needs to be done (bug 6003) - // in the above case, pasteParagraphList handles this - cur.buffer()->updateLabels(); + } + + cur.forceBufferUpdate(); + + // Ensure the current language is set correctly (bug 6292) + cur.text()->setCursor(cur, cur.pit(), cur.pos()); cur.clearSelection(); cur.resetAnchor(); return true; @@ -1711,8 +1795,6 @@ bool Text::read(Lexer & lex, readParagraph(pars_.back(), lex, errorList); // register the words in the global word list - CursorSlice sl = CursorSlice(*insetPtr); - sl.pit() = pars_.size() - 1; pars_.back().updateWords(); } else if (token == "\\begin_deeper") { ++depth; @@ -1740,6 +1822,7 @@ bool Text::read(Lexer & lex, return res; } + // Returns the current font and depth as a message. docstring Text::currentState(Cursor const & cur) const { @@ -1828,10 +1911,11 @@ docstring Text::getPossibleLabel(Cursor const & cur) const docstring text; docstring par_text = pars_[pit].asString(); - string piece; - // the return string of math matrices might contain linebreaks + + // The return string of math matrices might contain linebreaks par_text = subst(par_text, '\n', '-'); - for (int i = 0; i < lyxrc.label_init_length; ++i) { + int const numwords = 3; + for (int i = 0; i < numwords; ++i) { if (par_text.empty()) break; docstring head; @@ -1841,12 +1925,13 @@ docstring Text::getPossibleLabel(Cursor const & cur) const text += '-'; text += head; } + + // Make sure it isn't too long + unsigned int const max_label_length = 32; + if (text.size() > max_label_length) + text.resize(max_label_length); - // No need for a prefix if the user said so. - if (lyxrc.label_init_length <= 0) - return text; - - // Will contain the label type. + // Will contain the label prefix. docstring name; // For section, subsection, etc... @@ -1858,42 +1943,39 @@ docstring Text::getPossibleLabel(Cursor const & cur) const } } if (layout->latextype != LATEX_PARAGRAPH) - name = from_ascii(layout->latexname()); + name = layout->refprefix; - // for captions, we just take the caption type + // For captions, we just take the caption type Inset * caption_inset = cur.innerInsetOfType(CAPTION_CODE); - if (caption_inset) - name = from_ascii(static_cast(caption_inset)->type()); - - // If none of the above worked, we'll see if we're inside various - // types of insets and take our abbreviation from them. - if (name.empty()) { - InsetCode const codes[] = { - FLOAT_CODE, - WRAP_CODE, - FOOT_CODE - }; - for (unsigned int i = 0; i < (sizeof codes / sizeof codes[0]); ++i) { - Inset * float_inset = cur.innerInsetOfType(codes[i]); - if (float_inset) { - name = float_inset->name(); - break; - } + if (caption_inset) { + string const & ftype = static_cast(caption_inset)->floattype(); + FloatList const & fl = cur.buffer()->params().documentClass().floats(); + if (fl.typeExist(ftype)) { + Floating const & flt = fl.getType(ftype); + name = from_utf8(flt.refPrefix()); } + if (name.empty()) + name = from_utf8(ftype.substr(0,3)); } - // Create a correct prefix for prettyref - if (name == "theorem") - name = from_ascii("thm"); - else if (name == "Foot") - name = from_ascii("fn"); - else if (name == "listing") - name = from_ascii("lst"); + // If none of the above worked, see if the inset knows. + if (name.empty()) { + InsetLayout const & il = cur.inset().getLayout(); + name = il.refprefix(); + } if (!name.empty()) - text = name.substr(0, 3) + ':' + text; + text = name + ':' + text; + + // We need a unique label + docstring label = text; + int i = 1; + while (cur.buffer()->insetLabel(label)) { + label = text + '-' + convert(i); + ++i; + } - return text; + return label; } @@ -1915,6 +1997,18 @@ docstring Text::asString(pit_type beg, pit_type end, int options) const } +void Text::forToc(docstring & os, size_t maxlen, bool shorten) const +{ + if (maxlen == 0) + maxlen = std::numeric_limits::max(); + else + LASSERT(maxlen >= 8, maxlen = TOC_ENTRY_LENGTH); + for (size_t i = 0; i != pars_.size() && os.length() < maxlen; ++i) + pars_[i].forToc(os, maxlen); + if (shorten && os.length() >= maxlen) + os = os.substr(0, maxlen - 3) + from_ascii("..."); +} + void Text::charsTranspose(Cursor & cur) { @@ -2011,7 +2105,8 @@ bool Text::completionSupported(Cursor const & cur) const CompletionList const * Text::createCompletionList(Cursor const & cur) const { - return new TextCompletionList(cur); + WordList const * list = theWordList(*cur.getFont().language()); + return new TextCompletionList(cur, list); } @@ -2020,8 +2115,8 @@ bool Text::insertCompletion(Cursor & cur, docstring const & s, bool /*finished*/ LASSERT(cur.bv().cursor() == cur, /**/); cur.insert(s); cur.bv().cursor() = cur; - if (!(cur.disp_.update() & Update::Force)) - cur.updateFlags(cur.disp_.update() | Update::SinglePar); + if (!(cur.result().screenUpdate() & Update::Force)) + cur.screenUpdateFlags(cur.result().screenUpdate() | Update::SinglePar); return true; }