SCons: msvc does not need this /TP option any more after we rename .C => .cpp. Also...

[lyx.git] / src / CutAndPaste.cpp

diff --git a/src/CutAndPaste.cpp b/src/CutAndPaste.cpp
index f66153f7393b329495c284c5afd979553ee5a3e7..a6339581ec0f80d0cf2e48c0e89cfd707942e84b 100644 (file)
--- a/src/CutAndPaste.cpp
+++ b/src/CutAndPaste.cpp
@@ -18,6 +18,7 @@
 #include "Buffer.h"
 #include "buffer_funcs.h"
 #include "BufferParams.h"
+#include "BufferView.h"
 #include "Cursor.h"
 #include "debug.h"
 #include "ErrorList.h"
@@ -413,6 +414,8 @@ docstring grabAndEraseSelection(Cursor & cur)
 void switchBetweenClasses(textclass_type c1, textclass_type c2,
        InsetText & in, ErrorList & errorlist)
 {
+       errorlist.clear();
+
        BOOST_ASSERT(!in.paragraphs().empty());
        if (c1 == c2)
                return;
@@ -423,7 +426,7 @@ void switchBetweenClasses(textclass_type c1, textclass_type c2,
        // layouts
        ParIterator end = par_iterator_end(in);
        for (ParIterator it = par_iterator_begin(in); it != end; ++it) {
-               string const name = it->layout()->name();
+               docstring const name = it->layout()->name();
                bool hasLayout = tclass2.hasLayout(name);
 
                if (hasLayout)
@@ -435,7 +438,7 @@ void switchBetweenClasses(textclass_type c1, textclass_type c2,
                        docstring const s = bformat(
                                                 _("Layout had to be changed from\n%1$s to %2$s\n"
                                                "because of class conversion from\n%3$s to %4$s"),
-                        from_utf8(name), from_utf8(it->layout()->name()),
+                        name, it->layout()->name(),
                         from_utf8(tclass1.name()), from_utf8(tclass2.name()));
                        // To warn the user that something had to be done.
                        errorlist.push_back(ErrorItem(_("Changed Layout"), s,
@@ -450,7 +453,7 @@ void switchBetweenClasses(textclass_type c1, textclass_type c2,
                if (it->lyxCode() == Inset::CHARSTYLE_CODE) {
                        InsetCharStyle & inset =
                                static_cast<InsetCharStyle &>(*it);
-                       string const name = inset.params().type;
+                       string const name = inset.params().name;
                        CharStyles::iterator const found_cs =
                                tclass2.charstyle(name);
                        if (found_cs == tclass2.charstyles().end()) {
@@ -524,6 +527,8 @@ void cutSelection(Cursor & cur, bool doclear, bool realcut)
                Text * text = cur.text();
                BOOST_ASSERT(text);
 
+               saveSelection(cur);
+
                // make sure that the depth behind the selection are restored, too
                recordUndoSelection(cur);
                pit_type begpit = cur.selBegin().pit();
@@ -565,7 +570,6 @@ void cutSelection(Cursor & cur, bool doclear, bool realcut)
                // need a valid cursor. (Lgb)
                cur.clearSelection();
                updateLabels(cur.buffer());
-               theSelection().haveSelection(false);
 
                // tell tabular that a recent copy happened
                dirtyTabularStack(false);
@@ -672,22 +676,17 @@ void copySelection(Cursor & cur, docstring const & plaintext)
 
 void saveSelection(Cursor & cur)
 {
-       LYXERR(Debug::ACTION) << BOOST_CURRENT_FUNCTION << ": `"
-              << to_utf8(cur.selectionAsString(true)) << "'."
-              << endl;
-
-       // FIXME: The two lines below would allow middle-mouse 
-       // pasting that preserves the LyX formatting when the selection
-       // is internal. They would also allow to use the feature on
-       // Windows and Mac. In the future, we may want to optionally enable
-       // this feature via a rc setting.
-       /*
-       if (cur.selection())
+       // This function is called, not when a selection is formed, but when
+       // a selection is cleared. Therefore, multiple keyboard selection
+       // will not repeatively trigger this function (bug 3877).
+       if (cur.selection() 
+           && cur.selBegin() == cur.bv().cursor().selBegin()
+           && cur.selEnd() == cur.bv().cursor().selEnd()) {
+               LYXERR(Debug::ACTION) << BOOST_CURRENT_FUNCTION << ": `"
+                          << to_utf8(cur.selectionAsString(true)) << "'."
+                          << endl;
                copySelectionToStack(cur, selectionBuffer);
-       */
-
-       // tell X whether we now have a valid selection
-       theSelection().haveSelection(cur.selection());
+       }
 }
 
 
@@ -744,7 +743,6 @@ void pasteFromStack(Cursor & cur, ErrorList & errorList, size_t sel_index)
        pasteParagraphList(cur, theCuts[sel_index].first,
                           theCuts[sel_index].second, errorList);
        cur.setSelection();
-       saveSelection(cur);
 }
 
 
@@ -793,7 +791,6 @@ void pasteSelection(Cursor & cur, ErrorList & errorList)
        recordUndo(cur);
        pasteParagraphList(cur, selectionBuffer[0].first,
                           selectionBuffer[0].second, errorList);
-       cur.setSelection();
 }
 
 
@@ -823,7 +820,6 @@ void replaceSelectionWithString(Cursor & cur, docstring const & str, bool backwa
                cur.setSelection(selbeg, -int(str.length()));
        } else
                cur.setSelection(selbeg, str.length());
-       saveSelection(cur);
 }
 
 
@@ -840,6 +836,7 @@ void eraseSelection(Cursor & cur)
        CursorSlice const & i1 = cur.selBegin();
        CursorSlice const & i2 = cur.selEnd();
        if (i1.inset().asInsetMath()) {
+               saveSelection(cur);
                cur.top() = i1;
                if (i1.idx() == i2.idx()) {
                        i1.cell().erase(i1.pos(), i2.pos());
@@ -860,7 +857,6 @@ void eraseSelection(Cursor & cur)
                }
                // need a valid cursor. (Lgb)
                cur.clearSelection();
-               theSelection().haveSelection(false);
        } else {
                lyxerr << "can't erase this selection 1" << endl;
        }