]> git.lyx.org Git - lyx.git/blobdiff - src/text2.C
Alfredo's second patch
[lyx.git] / src / text2.C
index 32c8f0c1b268cdad2624985a8de430d1c3d80f15..c7babeb609ecf9f262e0211f224fd09dc28293e4 100644 (file)
@@ -40,6 +40,7 @@
 #include "support/lstrings.h"
 
 #include "support/BoostFormat.h"
+#include <boost/tuple/tuple.hpp>
 
 using std::vector;
 using std::copy;
@@ -79,7 +80,6 @@ void LyXText::init(BufferView * bview, bool reinit)
                rowlist_.clear();
                need_break_row = rows().end();
                width = height = 0;
-               copylayouttype.erase();
                top_y(0);
                clearPaint();
        } else if (!rowlist_.empty())
@@ -230,11 +230,12 @@ void LyXText::setCharFont(Buffer const * buf, ParagraphList::iterator pit,
 
        // Realize against environment font information
        if (pit->getDepth()) {
-#warning FIXME I think I hate this outerHood stuff.
-               Paragraph * tp = &*pit;
-               while (!layoutfont.resolved() && tp && tp->getDepth()) {
-                       tp = outerHook(tp);
-                       if (tp)
+               ParagraphList::iterator tp = pit;
+               while (!layoutfont.resolved() &&
+                      tp != ownerParagraphs().end() &&
+                      tp->getDepth()) {
+                       tp = outerHook(tp, ownerParagraphs());
+                       if (tp != ownerParagraphs().end())
                                layoutfont.realize(tp->layout()->font);
                }
        }
@@ -388,7 +389,7 @@ LyXText::setLayout(LyXCursor & cur, LyXCursor & sstart_cur,
                ++endpit;
        }
 
-       setUndo(bv(), Undo::EDIT, &*sstart_cur.par(), &*undoendpit);
+       setUndo(bv(), Undo::EDIT, sstart_cur.par(), undoendpit);
 
        // ok we have a selection. This is always between sstart_cur
        // and sel_end cursor
@@ -462,7 +463,7 @@ bool LyXText::changeDepth(bv_funcs::DEPTH_CHANGE type, bool test_only)
        ParagraphList::iterator pastend = boost::next(end);
 
        if (!test_only)
-               setUndo(bv(), Undo::EDIT, &(*start), &(*pastend));
+               setUndo(bv(), Undo::EDIT, start, pastend);
 
        bool changed = false;
 
@@ -565,7 +566,7 @@ void LyXText::setFont(LyXFont const & font, bool toggleall)
        // and sel_end cursor
 
        setUndo(bv(), Undo::EDIT,
-               &*selection.start.par(), &*boost::next(selection.end.par()));
+               selection.start.par(), boost::next(selection.end.par()));
        freezeUndo();
        cursor = selection.start;
        while (cursor.par() != selection.end.par() ||
@@ -645,6 +646,7 @@ void LyXText::redoParagraphs(LyXCursor const & cur,
        int y = cur.y() - tmprit->baseline();
 
        ParagraphList::iterator first_phys_pit;
+       RowList::iterator prevrit;
        if (tmprit == rows().begin()) {
                // A trick/hack for UNDO.
                // This is needed because in an UNDO/REDO we could have
@@ -652,8 +654,7 @@ void LyXText::redoParagraphs(LyXCursor const & cur,
                // the row is NOT my really first par anymore.
                // Got it Lars ;) (Jug 20011206)
                first_phys_pit = ownerParagraphs().begin();
-#warning FIXME
-               // In here prevrit could be set to rows().end(). (Lgb)
+               prevrit = rows().end();
        } else {
                first_phys_pit = tmprit->par();
                while (tmprit != rows().begin()
@@ -662,18 +663,7 @@ void LyXText::redoParagraphs(LyXCursor const & cur,
                        --tmprit;
                        y -= tmprit->height();
                }
-#warning FIXME
-               // Is it possible to put the prevrit setting in here? (Lgb)
-       }
-
-       RowList::iterator prevrit;
-       bool good_prevrit = false;
-#warning FIXME
-       // It seems to mee that good_prevrit is not needed if we let
-       // a bad prevrit have the value rows().end() (Lgb)
-       if (tmprit != rows().begin()) {
                prevrit = boost::prior(tmprit);
-               good_prevrit = true;
        }
 
        // remove it
@@ -684,24 +674,18 @@ void LyXText::redoParagraphs(LyXCursor const & cur,
 
        // Reinsert the paragraphs.
        ParagraphList::iterator tmppit = first_phys_pit;
-#warning FIXME
-       // See if this loop can be rewritten as a while loop instead.
-       // That should also make the code a bit easier to read. (Lgb)
-       do {
-               if (tmppit != ownerParagraphs().end()) {
-                       insertParagraph(tmppit, tmprit);
-                       while (tmprit != rows().end()
-                              && tmprit->par() == tmppit) {
-                               ++tmprit;
-                       }
-                       ++tmppit;
-               }
-       } while (tmppit != ownerParagraphs().end() && tmppit != endpit);
 
-#warning FIXME
-       // If the above changes are done, then we can compare prevrit
-       // with rows().end() here. (Lgb)
-       if (good_prevrit) {
+       while (tmppit != ownerParagraphs().end()) {
+               insertParagraph(tmppit, tmprit);
+               while (tmprit != rows().end()
+                      && tmprit->par() == tmppit) {
+                       ++tmprit;
+               }
+               ++tmppit;
+               if (tmppit == endpit)
+                       break;
+       }
+       if (prevrit != rows().end()) {
                setHeightOfRow(prevrit);
                const_cast<LyXText *>(this)->postPaint(y - prevrit->height());
        } else {
@@ -984,7 +968,7 @@ void LyXText::setParagraph(bool line_top, bool line_bottom,
                ++endpit;
        }
 
-       setUndo(bv(), Undo::EDIT, &*selection.start.par(), &*undoendpit);
+       setUndo(bv(), Undo::EDIT, selection.start.par(), undoendpit);
 
 
        ParagraphList::iterator tmppit = selection.end.par();
@@ -1073,7 +1057,8 @@ void LyXText::setCounter(Buffer const * buf, ParagraphList::iterator pit)
        if (pit != ownerParagraphs().begin()
            && boost::prior(pit)->getDepth() > pit->getDepth()
            && layout->labeltype != LABEL_BIBLIO) {
-               pit->enumdepth = depthHook(&*pit, pit->getDepth())->enumdepth;
+               pit->enumdepth = depthHook(pit, ownerParagraphs(),
+                                          pit->getDepth())->enumdepth;
        }
 
        if (!pit->params().labelString().empty()) {
@@ -1102,9 +1087,9 @@ void LyXText::setCounter(Buffer const * buf, ParagraphList::iterator pit)
 
                        // Is there a label? Useful for Chapter layout
                        if (!pit->params().appendix()) {
-                               s << layout->labelstring();
+                               s << buf->B_(layout->labelstring());
                        } else {
-                               s << layout->labelstring_appendix();
+                               s << buf->B_(layout->labelstring_appendix());
                        }
 
                        // Use of an integer is here less than elegant. For now.
@@ -1119,7 +1104,8 @@ void LyXText::setCounter(Buffer const * buf, ParagraphList::iterator pit)
                                        langtype = "latin";
                        }
 
-                       s << textclass.counters()
+                       s << " "
+                         << textclass.counters()
                                .numberLabel(layout->latexname(),
                                             numbertype, langtype, head);
 
@@ -1166,7 +1152,7 @@ void LyXText::setCounter(Buffer const * buf, ParagraphList::iterator pit)
                }
                // In biblio should't be following counters but...
        } else {
-               string s = layout->labelstring();
+               string s = buf->B_(layout->labelstring());
 
                // the caption hack:
                if (layout->labeltype == LABEL_SENSITIVE) {
@@ -1194,14 +1180,14 @@ void LyXText::setCounter(Buffer const * buf, ParagraphList::iterator pit)
 
                                // Doesn't work... yet.
 #if USE_BOOST_FORMAT
-                               s = boost::io::str(boost::format(_("%1$s #:")) % fl.name());
+                               s = boost::io::str(boost::format(_("%1$s #:")) % buf->B_(fl.name()));
                                // s << boost::format(_("%1$s %1$d:")
                                //        % fl.name()
                                //        % buf->counters().value(fl.name());
 #else
                                ostringstream o;
                                //o << fl.name() << ' ' << buf->counters().value(fl.name()) << ":";
-                               o << fl.name() << " #:";
+                               o << buf->B_(fl.name()) << " #:";
                                s = STRCONV(o.str());
 #endif
                        } else {
@@ -1269,8 +1255,8 @@ void LyXText::insertInset(Inset * inset)
 {
        if (!cursor.par()->insetAllowed(inset->lyxCode()))
                return;
-       setUndo(bv(), Undo::FINISH, &*cursor.par(),
-               &*boost::next(cursor.par()));
+       setUndo(bv(), Undo::FINISH, cursor.par(),
+               boost::next(cursor.par()));
        freezeUndo();
        cursor.par()->insertInset(cursor.pos(), inset);
        // Just to rebreak and refresh correctly.
@@ -1288,20 +1274,6 @@ void LyXText::insertInset(Inset * inset)
 }
 
 
-void LyXText::copyEnvironmentType()
-{
-       copylayouttype = cursor.par()->layout()->name();
-}
-
-
-void LyXText::pasteEnvironmentType()
-{
-       // do nothing if there has been no previous copyEnvironmentType()
-       if (!copylayouttype.empty())
-               setLayout(copylayouttype);
-}
-
-
 void LyXText::cutSelection(bool doclear, bool realcut)
 {
        // Stuff what we got on the clipboard. Even if there is no selection.
@@ -1336,43 +1308,36 @@ void LyXText::cutSelection(bool doclear, bool realcut)
                ++endpit;
        }
 
-       setUndo(bv(), Undo::DELETE, &*selection.start.par(), &*undoendpit);
+       setUndo(bv(), Undo::DELETE, selection.start.par(), undoendpit);
 
-       // there are two cases: cut only within one paragraph or
-       // more than one paragraph
-       if (selection.start.par() == selection.end.par()) {
-               // only within one paragraph
-               endpit = selection.end.par();
-               int pos = selection.end.pos();
-               CutAndPaste::cutSelection(&*selection.start.par(), &*endpit,
-                                         selection.start.pos(), pos,
-                                         bv()->buffer()->params.textclass,
-                                         doclear, realcut);
-               selection.end.pos(pos);
-       } else {
-               endpit = selection.end.par();
-               int pos = selection.end.pos();
-               CutAndPaste::cutSelection(&*selection.start.par(), &*endpit,
-                                         selection.start.pos(), pos,
-                                         bv()->buffer()->params.textclass,
-                                         doclear, realcut);
-               cursor.par(endpit);
-               selection.end.par(endpit);
-               selection.end.pos(pos);
-               cursor.pos(selection.end.pos());
-       }
-       ++endpit;
 
+       endpit = selection.end.par();
+       int endpos = selection.end.pos();
+
+       boost::tie(endpit, endpos) = realcut ?
+               CutAndPaste::cutSelection(ownerParagraphs(),
+                                         selection.start.par(), endpit,
+                                         selection.start.pos(), endpos,
+                                         bv()->buffer()->params.textclass,
+                                         doclear)
+               : CutAndPaste::eraseSelection(ownerParagraphs(),
+                                             selection.start.par(), endpit,
+                                             selection.start.pos(), endpos,
+                                             doclear);
        // sometimes necessary
        if (doclear)
                selection.start.par()->stripLeadingSpaces();
 
-       redoParagraphs(selection.start, endpit);
-
+       redoParagraphs(selection.start, boost::next(endpit));
+#warning FIXME latent bug
+       // endpit will be invalidated on redoParagraphs once ParagraphList
+       // becomes a std::list? There are maybe other places on which this
+       // can happend? (Ab)
        // cutSelection can invalidate the cursor so we need to set
        // it anew. (Lgb)
        // we prefer the end for when tracking changes
-       cursor = selection.end;
+       cursor.pos(endpos);
+       cursor.par(endpit);
 
        // need a valid cursor. (Lgb)
        clearSelection();
@@ -1402,8 +1367,8 @@ void LyXText::copySelection()
                   || selection.start.pos() < selection.end.pos()))
                selection.start.pos(selection.start.pos() + 1);
 
-       CutAndPaste::copySelection(&*selection.start.par(),
-                                  &*selection.end.par(),
+       CutAndPaste::copySelection(selection.start.par(),
+                                  selection.end.par(),
                                   selection.start.pos(), selection.end.pos(),
                                   bv()->buffer()->params.textclass);
 }
@@ -1416,23 +1381,23 @@ void LyXText::pasteSelection()
                return;
 
        setUndo(bv(), Undo::INSERT,
-               &*cursor.par(), &*boost::next(cursor.par()));
+               cursor.par(), boost::next(cursor.par()));
 
-       Paragraph * endpar;
-       ParagraphList::iterator actpit = cursor.par();
-       int pos = cursor.pos();
+       ParagraphList::iterator endpit;
+       PitPosPair ppp;
 
-       Paragraph * actpar = &*actpit;
-       CutAndPaste::pasteSelection(&actpar, &endpar, pos,
-                                   bv()->buffer()->params.textclass);
-
-       redoParagraphs(cursor, endpar);
+       boost::tie(ppp, endpit) = 
+               CutAndPaste::pasteSelection(ownerParagraphs(), 
+                                           cursor.par(), cursor.pos(), 
+                                           bv()->buffer()->params.textclass);
+       
+       redoParagraphs(cursor, endpit);
 
        setCursor(cursor.par(), cursor.pos());
        clearSelection();
 
        selection.cursor = cursor;
-       setCursor(actpit, pos);
+       setCursor(ppp.first, ppp.second);
        setSelection();
        updateCounters();
 }
@@ -1492,8 +1457,7 @@ void LyXText::insertStringAsLines(string const & str)
        // only to be sure, should not be neccessary
        clearSelection();
 
-       Paragraph * par = &*pit;
-       bv()->buffer()->insertStringAsLines(par, pos, current_font, str);
+       bv()->buffer()->insertStringAsLines(pit, pos, current_font, str);
 
        redoParagraphs(cursor, endpit);
        setCursor(cursor.par(), cursor.pos());
@@ -1662,15 +1626,16 @@ void LyXText::setCursor(LyXCursor & cur, ParagraphList::iterator pit,
        // same paragraph and there is a previous row then put the cursor on
        // the end of the previous row
        cur.iy(y + row->baseline());
-       Inset * ins;
-       if (row != beg && pos &&
-               boost::prior(row)->par() == row->par() &&
+       if (row != beg &&
+           pos &&
+           boost::prior(row)->par() == row->par() &&
            pos < pit->size() &&
-               pit->getChar(pos) == Paragraph::META_INSET &&
-               (ins = pit->getInset(pos)) && (ins->needFullRow() || ins->display()))
-       {
-               --row;
-               y -= row->height();
+           pit->getChar(pos) == Paragraph::META_INSET) {
+               Inset * ins = pit->getInset(pos);
+               if (ins && (ins->needFullRow() || ins->display())) {
+                       --row;
+                       y -= row->height();
+               }
        }
 
        cur.row(row);
@@ -2271,7 +2236,7 @@ bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
                                ++endpit;
                        }
 
-                       setUndo(bv(), Undo::DELETE, &*old_cursor.par(), &*endpit);
+                       setUndo(bv(), Undo::DELETE, old_cursor.par(), endpit);
                        cursor = tmpcursor;
 
                        // delete old row
@@ -2302,7 +2267,7 @@ bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
                                ++endpit;
                        }
 
-                       setUndo(bv(), Undo::DELETE, &*old_cursor.par(), &*endpit);
+                       setUndo(bv(), Undo::DELETE, old_cursor.par(), endpit);
                        cursor = tmpcursor;
 
                        // delete old row