X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Ftext3.C;h=9fb01eda4f153259b3307df1a09148b06d5424b3;hb=35204f8f33d7400a5fefeffea533fb4cb4097211;hp=4a494ecd466867590e8a0f0eb46833f58adf02bf;hpb=c9c6ed0b7fbccbbed8b5ef48d05cfefa5acfb99e;p=lyx.git

diff --git a/src/text3.C b/src/text3.C
index 4a494ecd46..9fb01eda4f 100644
--- a/src/text3.C
+++ b/src/text3.C
@@ -29,6 +29,7 @@
 #include "CutAndPaste.h"
 #include "debug.h"
 #include "dispatchresult.h"
+#include "errorlist.h"
 #include "factory.h"
 #include "funcrequest.h"
 #include "gettext.h"
@@ -46,9 +47,6 @@
 #include "vspace.h"
 #include "pariterator.h"
 
-#include "frontends/Dialogs.h"
-#include "frontends/Gui.h"
-#include "frontends/LyXView.h"
 #include "frontends/Clipboard.h"
 #include "frontends/Selection.h"
 
@@ -64,26 +62,24 @@
 #include "support/convert.h"
 #include "support/lyxtime.h"
 
-#include "mathed/math_hullinset.h"
-#include "mathed/math_macrotemplate.h"
+#include "mathed/InsetMathHull.h"
+#include "mathed/MathMacroTemplate.h"
 
 #include <boost/current_function.hpp>
 
 #include <clocale>
 #include <sstream>
 
-using lyx::pos_type;
 
-using lyx::cap::copySelection;
-using lyx::cap::cutSelection;
-using lyx::cap::pasteSelection;
-using lyx::cap::replaceSelection;
+namespace lyx {
 
-using lyx::support::isStrUnsignedInt;
-using lyx::support::token;
+using cap::copySelection;
+using cap::cutSelection;
+using cap::pasteSelection;
+using cap::replaceSelection;
 
-using lyx::frontend::Gui;
-using lyx::frontend::Clipboard;
+using support::isStrUnsignedInt;
+using support::token;
 
 using std::endl;
 using std::string;
@@ -125,7 +121,7 @@ namespace {
 		if (selecting || cur.mark())
 			cur.setSelection();
 		if (!cur.selection())
-			cur.bv().owner()->gui().selection().haveSelection(false);
+			theSelection().haveSelection(false);
 		cur.bv().switchKeyMap();
 	}
 
@@ -140,7 +136,7 @@ namespace {
 	void mathDispatch(LCursor & cur, FuncRequest const & cmd, bool display)
 	{
 		recordUndo(cur);
-		string sel = cur.selectionAsString(false);
+		docstring sel = cur.selectionAsString(false);
 		//lyxerr << "selection is: '" << sel << "'" << endl;
 
 		// It may happen that sel is empty but there is a selection
@@ -148,7 +144,7 @@ namespace {
 
 		if (sel.empty()) {
 			const int old_pos = cur.pos();
-			cur.insert(new MathHullInset("simple"));
+			cur.insert(new InsetMathHull(hullSimple));
 			BOOST_ASSERT(old_pos == cur.pos());
 			cur.nextInset()->edit(cur, true);
 			// don't do that also for LFUN_MATH_MODE
@@ -159,30 +155,31 @@ namespace {
 				cur.dispatch(FuncRequest(LFUN_MATH_DISPLAY));
 			// Avoid an unnecessary undo step if cmd.argument
 			// is empty
-			if (!cmd.argument.empty())
+			if (!cmd.argument().empty())
 				cur.dispatch(FuncRequest(LFUN_MATH_INSERT,
-							 cmd.argument));
+							 cmd.argument()));
 		} else {
 			// create a macro if we see "\\newcommand"
 			// somewhere, and an ordinary formula
 			// otherwise
-			istringstream is(sel);
-			if (sel.find("\\newcommand") == string::npos
-			    && sel.find("\\def") == string::npos)
+			if (sel.find(from_ascii("\\newcommand")) == string::npos
+			    && sel.find(from_ascii("\\def")) == string::npos)
 			{
-				MathHullInset * formula = new MathHullInset;
+				InsetMathHull * formula = new InsetMathHull;
+				istringstream is(to_utf8(sel));
 				LyXLex lex(0, 0);
 				lex.setStream(is);
 				formula->read(cur.buffer(), lex);
-				if (formula->getType() == "none")
+				if (formula->getType() == hullNone)
 					// Don't create pseudo formulas if
 					// delimiters are left out
-					formula->mutate("simple");
+					formula->mutate(hullSimple);
 				cur.insert(formula);
-			} else
-				cur.insert(new MathMacroTemplate(is));
+			} else {
+				cur.insert(new MathMacroTemplate(sel));
+			}
 		}
-		cur.message(N_("Math editor mode"));
+		cur.message(from_utf8(N_("Math editor mode")));
 	}
 
 } // namespace anon
@@ -204,7 +201,7 @@ string const freefont2string()
 bool LyXText::cursorPrevious(LCursor & cur)
 {
 	pos_type cpos = cur.pos();
-	lyx::pit_type cpar = cur.pit();
+	pit_type cpar = cur.pit();
 
 	int x = cur.x_target();
 
@@ -227,7 +224,7 @@ bool LyXText::cursorPrevious(LCursor & cur)
 bool LyXText::cursorNext(LCursor & cur)
 {
 	pos_type cpos = cur.pos();
-	lyx::pit_type cpar = cur.pit();
+	pit_type cpar = cur.pit();
 
 	int x = cur.x_target();
 	bool updated = setCursorFromCoordinates(cur, x, cur.bv().workHeight() - 1);
@@ -250,7 +247,7 @@ namespace {
 
 void specialChar(LCursor & cur, InsetSpecialChar::Kind kind)
 {
-	lyx::cap::replaceSelection(cur);
+	cap::replaceSelection(cur);
 	cur.insert(new InsetSpecialChar(kind));
 	cur.posRight();
 }
@@ -266,7 +263,7 @@ bool doInsertInset(LCursor & cur, LyXText * text,
 	recordUndo(cur);
 	bool gotsel = false;
 	if (cur.selection()) {
-		cur.bv().owner()->dispatch(FuncRequest(LFUN_CUT));
+		lyx::dispatch(FuncRequest(LFUN_CUT));
 		gotsel = true;
 	}
 	text->insertInset(cur, inset);
@@ -274,20 +271,19 @@ bool doInsertInset(LCursor & cur, LyXText * text,
 	if (edit)
 		inset->edit(cur, true);
 
-	if (gotsel && pastesel)
-		cur.bv().owner()->dispatch(FuncRequest(LFUN_PASTE));
+	if (gotsel && pastesel) {
+		lyx::dispatch(FuncRequest(LFUN_PASTE));
+		// reset first par to default
+		if (cur.lastpit() != 0 || cur.lastpos() != 0) {
+			LyXLayout_ptr const layout =
+				cur.buffer().params().getLyXTextClass().defaultLayout();
+			cur.text()->paragraphs().begin()->layout(layout);
+		}
+	}
 	return true;
 }
 
 
-void update(LCursor & cur)
-{
-	//we don't call update(true, false) directly to save a metrics call
-	if (cur.bv().fitCursor())
-		cur.bv().update(Update::Force);
-}
-
-
 } // anon namespace
 
 
@@ -299,9 +295,9 @@ void LyXText::number(LCursor & cur)
 }
 
 
-bool LyXText::isRTL(Paragraph const & par) const
+bool LyXText::isRTL(Buffer const & buffer, Paragraph const & par) const
 {
-	return par.isRightToLeftPar(bv()->buffer()->params());
+	return par.isRightToLeftPar(buffer.params());
 }
 
 
@@ -309,6 +305,11 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 {
 	lyxerr[Debug::ACTION] << "LyXText::dispatch: cmd: " << cmd << endl;
 
+	// FIXME: We use the update flag to indicates wether a singlePar or a
+	// full screen update is needed. We reset it here but shall we restore it
+	// at the end?
+	cur.noUpdate();
+
 	BOOST_ASSERT(cur.text() == this);
 	BufferView * bv = &cur.bv();
 	CursorSlice oldTopSlice = cur.top();
@@ -328,7 +329,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_PARAGRAPH_MOVE_DOWN: {
 		pit_type const pit = cur.pit();
-		recUndo(pit, pit + 1);
+		recUndo(cur, pit, pit + 1);
 		finishUndo();
 		std::swap(pars_[pit], pars_[pit + 1]);
 
@@ -343,7 +344,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_PARAGRAPH_MOVE_UP: {
 		pit_type const pit = cur.pit();
-		recUndo(pit - 1, pit);
+		recUndo(cur, pit - 1, pit);
 		finishUndo();
 		std::swap(pars_[pit], pars_[pit - 1]);
 
@@ -367,7 +368,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		// ensure that we have only one start_of_appendix in this document
 		for (pit_type tmp = 0, end = pars_.size(); tmp != end; ++tmp) {
 			if (pars_[tmp].params().startOfAppendix()) {
-				recUndo(tmp);
+				recUndo(cur, tmp);
 				pars_[tmp].params().startOfAppendix(false);
 				break;
 			}
@@ -399,65 +400,21 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		finishChange(cur, false);
 		break;
 
-	case LFUN_WORD_FORWARD:
-		if (!cur.mark())
-			cur.clearSelection();
-		if (isRTL(cur.paragraph()))
-			needsUpdate = cursorLeftOneWord(cur);
-		else
-			needsUpdate = cursorRightOneWord(cur);
-		finishChange(cur, false);
-		break;
-
-	case LFUN_WORD_BACKWARD:
-		if (!cur.mark())
-			cur.clearSelection();
-		if (isRTL(cur.paragraph()))
-			needsUpdate = cursorRightOneWord(cur);
-		else
-			needsUpdate = cursorLeftOneWord(cur);
-		finishChange(cur, false);
-		break;
-
 	case LFUN_BUFFER_BEGIN:
-		if (cur.depth() == 1) {
-			if (!cur.mark())
-				cur.clearSelection();
-			needsUpdate = cursorTop(cur);
-			finishChange(cur, false);
-		} else {
-			cur.undispatched();
-		}
-		break;
-
 	case LFUN_BUFFER_BEGIN_SELECT:
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_BUFFER_BEGIN_SELECT);
 		if (cur.depth() == 1) {
-			if (!cur.selection())
-				cur.resetAnchor();
-			needsUpdate = cursorTop(cur);
-			finishChange(cur, true);
+			needsUpdate |= cursorTop(cur);
 		} else {
 			cur.undispatched();
 		}
 		break;
 
 	case LFUN_BUFFER_END:
-		if (cur.depth() == 1) {
-			if (!cur.mark())
-				cur.clearSelection();
-			needsUpdate = cursorBottom(cur);
-			finishChange(cur, false);
-		} else {
-			cur.undispatched();
-		}
-		break;
-
 	case LFUN_BUFFER_END_SELECT:
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_BUFFER_END_SELECT);
 		if (cur.depth() == 1) {
-			if (!cur.selection())
-				cur.resetAnchor();
-			needsUpdate = cursorBottom(cur);
-			finishChange(cur, true);
+			needsUpdate |= cursorBottom(cur);
 		} else {
 			cur.undispatched();
 		}
@@ -467,11 +424,11 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 	case LFUN_CHAR_FORWARD_SELECT:
 		//lyxerr << BOOST_CURRENT_FUNCTION
 		//       << " LFUN_CHAR_FORWARD[SEL]:\n" << cur << endl;
-		cur.selHandle(cmd.action == LFUN_CHAR_FORWARD_SELECT);
-		if (isRTL(cur.paragraph()))
-			needsUpdate = cursorLeft(cur);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_CHAR_FORWARD_SELECT);
+		if (isRTL(*cur.bv().buffer(), cur.paragraph()))
+			needsUpdate |= cursorLeft(cur);
 		else
-			needsUpdate = cursorRight(cur);
+			needsUpdate |= cursorRight(cur);
 
 		if (!needsUpdate && oldTopSlice == cur.top()
 				&& cur.boundary() == oldBoundary) {
@@ -483,11 +440,11 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 	case LFUN_CHAR_BACKWARD:
 	case LFUN_CHAR_BACKWARD_SELECT:
 		//lyxerr << "handle LFUN_CHAR_BACKWARD[_SELECT]:\n" << cur << endl;
-		cur.selHandle(cmd.action == LFUN_CHAR_BACKWARD_SELECT);
-		if (isRTL(cur.paragraph()))
-			needsUpdate = cursorRight(cur);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_CHAR_BACKWARD_SELECT);
+		if (isRTL(*cur.bv().buffer(), cur.paragraph()))
+			needsUpdate |= cursorRight(cur);
 		else
-			needsUpdate = cursorLeft(cur);
+			needsUpdate |= cursorLeft(cur);
 
 		if (!needsUpdate && oldTopSlice == cur.top()
 			&& cur.boundary() == oldBoundary) {
@@ -498,11 +455,11 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_UP:
 	case LFUN_UP_SELECT:
-		update(cur);
 		//lyxerr << "handle LFUN_UP[SEL]:\n" << cur << endl;
-		cur.selHandle(cmd.action == LFUN_UP_SELECT);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_UP_SELECT);
+
+		needsUpdate |= cursorUp(cur);
 
-		needsUpdate = cursorUp(cur);
 		if (!needsUpdate && oldTopSlice == cur.top()
 			  && cur.boundary() == oldBoundary) {
 			cur.undispatched();
@@ -512,10 +469,10 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_DOWN:
 	case LFUN_DOWN_SELECT:
-		update(cur);
 		//lyxerr << "handle LFUN_DOWN[SEL]:\n" << cur << endl;
-		cur.selHandle(cmd.action == LFUN_DOWN_SELECT);
-		needsUpdate = cursorDown(cur);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_DOWN_SELECT);
+		needsUpdate |= cursorDown(cur);
+
 		if (!needsUpdate && oldTopSlice == cur.top() &&
 		    cur.boundary() == oldBoundary)
 		{
@@ -525,136 +482,80 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		break;
 
 	case LFUN_PARAGRAPH_UP:
-		if (!cur.mark())
-			cur.clearSelection();
-		needsUpdate = cursorUpParagraph(cur);
-		finishChange(cur, false);
-		break;
-
 	case LFUN_PARAGRAPH_UP_SELECT:
-		if (!cur.selection())
-			cur.resetAnchor();
-		cursorUpParagraph(cur);
-		finishChange(cur, true);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_PARAGRAPH_UP_SELECT);
+		needsUpdate |= cursorUpParagraph(cur);
 		break;
 
 	case LFUN_PARAGRAPH_DOWN:
-		if (!cur.mark())
-			cur.clearSelection();
-		needsUpdate = cursorDownParagraph(cur);
-		finishChange(cur, false);
-		break;
-
 	case LFUN_PARAGRAPH_DOWN_SELECT:
-		if (!cur.selection())
-			cur.resetAnchor();
-		cursorDownParagraph(cur);
-		finishChange(cur, true);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_PARAGRAPH_DOWN_SELECT);
+		needsUpdate |= cursorDownParagraph(cur);
 		break;
 
+	case LFUN_SCREEN_UP:
 	case LFUN_SCREEN_UP_SELECT:
-		update(cur);
-		if (!cur.selection())
-			cur.resetAnchor();
-		needsUpdate = cursorPrevious(cur);
-		finishChange(cur, true);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_SCREEN_UP_SELECT);
+		if (cur.pit() == 0 && cur.textRow().pos() == 0) {
+			cur.undispatched();
+			cmd = FuncRequest(LFUN_FINISHED_UP);
+		} else {
+			needsUpdate |= cursorPrevious(cur);
+		}
 		break;
 
+	case LFUN_SCREEN_DOWN:
 	case LFUN_SCREEN_DOWN_SELECT:
-		update(cur);
-		if (!cur.selection())
-			cur.resetAnchor();
-		needsUpdate = cursorNext(cur);
-		finishChange(cur, true);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_SCREEN_DOWN_SELECT);
+		if (cur.pit() == cur.lastpit()
+			  && cur.textRow().endpos() == cur.lastpos()) {
+			cur.undispatched();
+			cmd = FuncRequest(LFUN_FINISHED_DOWN);
+		} else {
+			needsUpdate |= cursorNext(cur);
+		}
 		break;
 
+	case LFUN_LINE_BEGIN:
 	case LFUN_LINE_BEGIN_SELECT:
-		update(cur);
-		if (!cur.selection())
-			cur.resetAnchor();
-		needsUpdate = cursorHome(cur);
-		finishChange(cur, true);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_LINE_BEGIN_SELECT);
+		needsUpdate |= cursorHome(cur);
 		break;
 
+	case LFUN_LINE_END:
 	case LFUN_LINE_END_SELECT:
-		update(cur);
-		if (!cur.selection())
-			cur.resetAnchor();
-		needsUpdate = cursorEnd(cur);
-		finishChange(cur, true);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_LINE_END_SELECT);
+		needsUpdate |= cursorEnd(cur);
 		break;
 
+	case LFUN_WORD_FORWARD:
 	case LFUN_WORD_FORWARD_SELECT:
-		if (!cur.selection())
-			cur.resetAnchor();
-		if (isRTL(cur.paragraph()))
-			cursorLeftOneWord(cur);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_WORD_FORWARD_SELECT);
+		if (isRTL(*cur.bv().buffer(), cur.paragraph()))
+			needsUpdate |= cursorLeftOneWord(cur);
 		else
-			cursorRightOneWord(cur);
-		finishChange(cur, true);
+			needsUpdate |= cursorRightOneWord(cur);
 		break;
 
+	case LFUN_WORD_BACKWARD:
 	case LFUN_WORD_BACKWARD_SELECT:
-		if (!cur.selection())
-			cur.resetAnchor();
-		if (isRTL(cur.paragraph()))
-			cursorRightOneWord(cur);
+		needsUpdate |= cur.selHandle(cmd.action == LFUN_WORD_BACKWARD_SELECT);
+		if (isRTL(*cur.bv().buffer(), cur.paragraph()))
+			needsUpdate |= cursorRightOneWord(cur);
 		else
-			cursorLeftOneWord(cur);
-		finishChange(cur, true);
+			needsUpdate |= cursorLeftOneWord(cur);
 		break;
 
 	case LFUN_WORD_SELECT: {
-		selectWord(cur, lyx::WHOLE_WORD);
+		selectWord(cur, WHOLE_WORD);
 		finishChange(cur, true);
 		break;
 	}
 
-	case LFUN_SCREEN_UP:
-		update(cur);
-		if (!cur.mark())
-			cur.clearSelection();
-		finishChange(cur, false);
-		if (cur.pit() == 0 && cur.textRow().pos() == 0) {
-			cur.undispatched();
-			cmd = FuncRequest(LFUN_FINISHED_UP);
-		} else {
-			needsUpdate = cursorPrevious(cur);
-		}
-		break;
-
-	case LFUN_SCREEN_DOWN:
-		update(cur);
-		if (!cur.mark())
-			cur.clearSelection();
-		finishChange(cur, false);
-		if (cur.pit() == cur.lastpit()
-			  && cur.textRow().endpos() == cur.lastpos()) {
-			cur.undispatched();
-			cmd = FuncRequest(LFUN_FINISHED_DOWN);
-		} else {
-			needsUpdate = cursorNext(cur);
-		}
-		break;
-
-	case LFUN_LINE_BEGIN:
-		if (!cur.mark())
-			cur.clearSelection();
-		needsUpdate = cursorHome(cur);
-		finishChange(cur, false);
-		break;
-
-	case LFUN_LINE_END:
-		if (!cur.mark())
-			cur.clearSelection();
-		needsUpdate = cursorEnd(cur);
-		finishChange(cur, false);
-		break;
-
 	case LFUN_BREAK_LINE: {
 		// Not allowed by LaTeX (labels or empty par)
 		if (cur.pos() > cur.paragraph().beginOfBody()) {
-			lyx::cap::replaceSelection(cur);
+			cap::replaceSelection(cur);
 			cur.insert(new InsetNewline);
 			cur.posRight();
 			moveCursor(cur, false);
@@ -667,7 +568,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 			if (cur.pos() == cur.paragraph().size())
 				// Par boundary, force full-screen update
 				singleParUpdate = false;
-			needsUpdate = erase(cur);
+			needsUpdate |= erase(cur);
 			cur.resetAnchor();
 			// It is possible to make it a lot faster still
 			// just comment out the line below...
@@ -695,11 +596,11 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_CHAR_DELETE_BACKWARD:
 		if (!cur.selection()) {
-			if (bv->owner()->getIntl().getTransManager().backspace()) {
+			if (bv->getIntl().getTransManager().backspace()) {
 				// Par boundary, full-screen update
 				if (cur.pos() == 0)
 					singleParUpdate = false;
-				needsUpdate = backspace(cur);
+				needsUpdate |= backspace(cur);
 				cur.resetAnchor();
 				// It is possible to make it a lot faster still
 				// just comment out the line below...
@@ -726,14 +627,14 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		break;
 
 	case LFUN_BREAK_PARAGRAPH:
-		lyx::cap::replaceSelection(cur);
+		cap::replaceSelection(cur);
 		breakParagraph(cur, 0);
 		cur.resetAnchor();
 		bv->switchKeyMap();
 		break;
 
 	case LFUN_BREAK_PARAGRAPH_KEEP_LAYOUT:
-		lyx::cap::replaceSelection(cur);
+		cap::replaceSelection(cur);
 		breakParagraph(cur, 1);
 		cur.resetAnchor();
 		bv->switchKeyMap();
@@ -742,9 +643,9 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 	case LFUN_BREAK_PARAGRAPH_SKIP: {
 		// When at the beginning of a paragraph, remove
 		// indentation.  Otherwise, do the same as LFUN_BREAK_PARAGRAPH.
-		lyx::cap::replaceSelection(cur);
+		cap::replaceSelection(cur);
 		if (cur.pos() == 0)
-			cur.paragraph().params().labelWidthString(string());
+			cur.paragraph().params().labelWidthString(docstring());
 		else
 			breakParagraph(cur, 0);
 		cur.resetAnchor();
@@ -759,7 +660,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		if (cur_spacing == Spacing::Other)
 			cur_value = par.params().spacing().getValueAsString();
 
-		istringstream is(cmd.argument);
+		istringstream is(to_utf8(cmd.argument()));
 		string tmp;
 		is >> tmp;
 		Spacing::Space new_spacing = cur_spacing;
@@ -783,8 +684,8 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		} else if (tmp == "default") {
 			new_spacing = Spacing::Default;
 		} else {
-			lyxerr << _("Unknown spacing argument: ")
-			       << cmd.argument << endl;
+			lyxerr << to_utf8(_("Unknown spacing argument: "))
+			       << to_utf8(cmd.argument()) << endl;
 		}
 		if (cur_spacing != new_spacing || cur_value != new_value)
 			par.params().spacing(Spacing(new_spacing, new_value));
@@ -801,31 +702,13 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		break;
 	}
 
-	case LFUN_INSET_SETTINGS:
-		cur.inset().showInsetDialog(bv);
+	case LFUN_INSET_DISSOLVE:
+		needsUpdate |= dissolveInset(cur);
 		break;
 
-	case LFUN_NEXT_INSET_TOGGLE: {
-		InsetBase * inset = cur.nextInset();
-		// this is the real function we want to invoke
-		cmd = FuncRequest(LFUN_INSET_TOGGLE);
-		cur.undispatched();
-		// if there is an inset at cursor, see whether it
-		// wants to toggle.
-		if (inset) {
-			LCursor tmpcur = cur;
-			tmpcur.pushLeft(*inset);
-			inset->dispatch(tmpcur, cmd);
-			if (tmpcur.result().dispatched()) {
-				cur.clearSelection();
-				cur.dispatched();
-			}
-		}
-		// if it did not work, try the underlying inset.
-		if (!cur.result().dispatched())
-			cur.inset().dispatch(cur, cmd);
+	case LFUN_INSET_SETTINGS:
+		cur.inset().showInsetDialog(bv);
 		break;
-	}
 
 	case LFUN_SPACE_INSERT:
 		if (cur.paragraph().layout()->free_spacing)
@@ -870,16 +753,19 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		break;
 
 	case LFUN_CHARS_TRANSPOSE:
-		recordUndo(cur);
+ 		charsTranspose(cur);
 		break;
 
 	case LFUN_PASTE:
 		cur.message(_("Paste"));
-		lyx::cap::replaceSelection(cur);
-		if (isStrUnsignedInt(cmd.argument))
-			pasteSelection(cur, convert<unsigned int>(cmd.argument));
+		cap::replaceSelection(cur);
+		if (isStrUnsignedInt(to_utf8(cmd.argument())))
+			pasteSelection(cur, bv->buffer()->errorList("Paste"),
+			convert<unsigned int>(to_utf8(cmd.argument())));
 		else
-			pasteSelection(cur, 0);
+			pasteSelection(cur, bv->buffer()->errorList("Paste"),
+			0);
+		bv->buffer()->errors("Paste");
 		cur.clearSelection(); // bug 393
 		bv->switchKeyMap();
 		finishUndo();
@@ -896,18 +782,19 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		break;
 
 	case LFUN_SERVER_GET_XY:
-		cur.message(convert<string>(cursorX(cur.top(), cur.boundary())) + ' '
-			  + convert<string>(cursorY(cur.top(), cur.boundary())));
+		cur.message(from_utf8(
+			convert<string>(cursorX(cur.buffer(), cur.top(), cur.boundary()))
+			+ ' ' + convert<string>(cursorY(cur.top(), cur.boundary()))));
 		break;
 
 	case LFUN_SERVER_SET_XY: {
 		int x = 0;
 		int y = 0;
-		istringstream is(cmd.argument);
+		istringstream is(to_utf8(cmd.argument()));
 		is >> x >> y;
 		if (!is)
 			lyxerr << "SETXY: Could not parse coordinates in '"
-			       << cmd.argument << std::endl;
+			       << to_utf8(cmd.argument()) << std::endl;
 		else
 			setCursorFromCoordinates(cur, x, y);
 		break;
@@ -915,36 +802,27 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_SERVER_GET_FONT:
 		if (current_font.shape() == LyXFont::ITALIC_SHAPE)
-			cur.message("E");
+			cur.message(from_ascii("E"));
 		else if (current_font.shape() == LyXFont::SMALLCAPS_SHAPE)
-			cur.message("N");
+			cur.message(from_ascii("N"));
 		else
-			cur.message("0");
+			cur.message(from_ascii("0"));
 		break;
 
 	case LFUN_SERVER_GET_LAYOUT:
-		cur.message(cur.paragraph().layout()->name());
+		cur.message(from_utf8(cur.paragraph().layout()->name()));
 		break;
 
 	case LFUN_LAYOUT: {
-		lyxerr[Debug::INFO] << "LFUN_LAYOUT: (arg) "
-		  << cmd.argument << endl;
-
-		// This is not the good solution to the empty argument
-		// problem, but it will hopefully suffice for 1.2.0.
-		// The correct solution would be to augument the
-		// function list/array with information about what
-		// functions needs arguments and their type.
-		if (cmd.argument.empty()) {
-			cur.errorMessage(_("LyX function 'layout' needs an argument."));
-			break;
-		}
+		string layout = to_ascii(cmd.argument());
+		lyxerr[Debug::INFO] << "LFUN_LAYOUT: (arg) " << layout << endl;
 
 		// Derive layout number from given argument (string)
 		// and current buffer's textclass (number)
 		LyXTextClass const & tclass = bv->buffer()->params().getLyXTextClass();
-		bool hasLayout = tclass.hasLayout(cmd.argument);
-		string layout = cmd.argument;
+		if (layout.empty())
+			layout = tclass.defaultLayoutName();
+		bool hasLayout = tclass.hasLayout(layout);
 
 		// If the entry is obsolete, use the new one instead.
 		if (hasLayout) {
@@ -954,8 +832,8 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		}
 
 		if (!hasLayout) {
-			cur.errorMessage(string(N_("Layout ")) + cmd.argument +
-				N_(" not known"));
+			cur.errorMessage(from_utf8(N_("Layout ")) + cmd.argument() +
+				from_utf8(N_(" not known")));
 			break;
 		}
 
@@ -978,7 +856,8 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		if (change_layout) {
 			current_layout = layout;
 			setLayout(cur, layout);
-			bv->owner()->setLayout(layout);
+			// inform the GUI that the layout has changed.
+			bv->layoutChanged(layout);
 			bv->switchKeyMap();
 		}
 		break;
@@ -986,10 +865,10 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_CLIPBOARD_PASTE: {
 		cur.clearSelection();
-		string const clip = bv->owner()->gui().clipboard().get();
+		docstring const clip = theClipboard().get();
 		if (!clip.empty()) {
 			recordUndo(cur);
-			if (cmd.argument == "paragraph")
+			if (cmd.argument() == "paragraph")
 				insertStringAsParagraphs(cur, clip);
 			else
 				insertStringAsLines(cur, clip);
@@ -999,10 +878,10 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_PRIMARY_SELECTION_PASTE: {
 		cur.clearSelection();
-		string const clip = bv->owner()->gui().selection().get();
+		docstring const clip = theSelection().get();
 		if (!clip.empty()) {
 			recordUndo(cur);
-			if (cmd.argument == "paragraph")
+			if (cmd.argument() == "paragraph")
 				insertStringAsParagraphs(cur, clip);
 			else
 				insertStringAsLines(cur, clip);
@@ -1010,11 +889,26 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		break;
 	}
 
+	case LFUN_UNICODE_INSERT: {
+		if (cmd.argument().empty())
+			break;
+		docstring hexstring = cmd.argument();
+		if (lyx::support::isHex(hexstring)) {
+			char_type c = lyx::support::hexToInt(hexstring);
+			if (c > 32 && c < 0x10ffff) {
+				lyxerr << "Inserting c: " << c << endl;
+				docstring s = docstring(1, c);
+				lyx::dispatch(FuncRequest(LFUN_SELF_INSERT, s));
+			}
+		}
+		break;
+	}
+		
 	case LFUN_QUOTE_INSERT: {
-		lyx::cap::replaceSelection(cur);
+		cap::replaceSelection(cur);
 		Paragraph & par = cur.paragraph();
-		lyx::pos_type pos = cur.pos();
-		lyx::char_type c;
+		pos_type pos = cur.pos();
+		char_type c;
 		if (pos == 0)
 			c = ' ';
 		else if (cur.prevInset() && cur.prevInset()->isSpace())
@@ -1027,7 +921,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		BufferParams const & bufparams = bv->buffer()->params();
 		if (!style->pass_thru
 		    && par.getFontSettings(bufparams, pos).language()->lang() != "hebrew") {
-			string arg = cmd.argument;
+			string arg = to_utf8(cmd.argument());
 			if (arg == "single")
 				cur.insert(new InsetQuotes(c,
 				    bufparams.quotes_language,
@@ -1039,17 +933,17 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 			cur.posRight();
 		}
 		else
-			bv->owner()->dispatch(FuncRequest(LFUN_SELF_INSERT, "\""));
+			lyx::dispatch(FuncRequest(LFUN_SELF_INSERT, "\""));
 		break;
 	}
 
 	case LFUN_DATE_INSERT:
-		if (cmd.argument.empty())
-			bv->owner()->dispatch(FuncRequest(LFUN_SELF_INSERT,
-				lyx::formatted_time(lyx::current_time())));
+		if (cmd.argument().empty())
+			lyx::dispatch(FuncRequest(LFUN_SELF_INSERT,
+				formatted_time(current_time())));
 		else
-			bv->owner()->dispatch(FuncRequest(LFUN_SELF_INSERT,
-				lyx::formatted_time(lyx::current_time(), cmd.argument)));
+			lyx::dispatch(FuncRequest(LFUN_SELF_INSERT,
+				formatted_time(current_time(), to_utf8(cmd.argument()))));
 		break;
 
 	case LFUN_MOUSE_TRIPLE:
@@ -1059,15 +953,15 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 			cursorEnd(cur);
 			cur.setSelection();
 			bv->cursor() = cur;
-			bv->owner()->gui().selection().haveSelection(cur.selection());
+			theSelection().haveSelection(cur.selection());
 		}
 		break;
 
 	case LFUN_MOUSE_DOUBLE:
 		if (cmd.button() == mouse_button::button1) {
-			selectWord(cur, lyx::WHOLE_WORD_STRICT);
+			selectWord(cur, WHOLE_WORD_STRICT);
 			bv->cursor() = cur;
-			bv->owner()->gui().selection().haveSelection(cur.selection());
+			theSelection().haveSelection(cur.selection());
 		}
 		break;
 
@@ -1083,7 +977,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		// we have to check this first
 		bool paste_internally = false;
 		if (cmd.button() == mouse_button::button2 && cur.selection()) {
-			bv->owner()->dispatch(FuncRequest(LFUN_COPY));
+			lyx::dispatch(FuncRequest(LFUN_COPY));
 			paste_internally = true;
 		}
 
@@ -1094,9 +988,14 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		// insert this
 		if (cmd.button() == mouse_button::button2) {
 			if (paste_internally)
-				bv->owner()->dispatch(FuncRequest(LFUN_PASTE));
+				lyx::dispatch(FuncRequest(LFUN_PASTE));
 			else
-				bv->owner()->dispatch(FuncRequest(LFUN_PRIMARY_SELECTION_PASTE, "paragraph"));
+				lyx::dispatch(FuncRequest(LFUN_PRIMARY_SELECTION_PASTE, "paragraph"));
+		}
+
+		if (cmd.button() == mouse_button::button1) {
+			needsUpdate = false;
+			cur.noUpdate();
 		}
 
 		break;
@@ -1148,17 +1047,19 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 			break;
 
 		// finish selection
-		if (cmd.button() == mouse_button::button1)
-			bv->owner()->gui().selection().haveSelection(cur.selection());
+		if (cmd.button() == mouse_button::button1) {
+			if (cur.selection())
+				theSelection().haveSelection(cur.selection());
+			needsUpdate = false;
+			cur.noUpdate();
+		}
 
 		bv->switchKeyMap();
-		bv->owner()->updateMenubar();
-		bv->owner()->updateToolbars();
 		break;
 	}
 
 	case LFUN_SELF_INSERT: {
-		if (cmd.argument.empty())
+		if (cmd.argument().empty())
 			break;
 
 		// Automatically delete the currently selected
@@ -1170,17 +1071,21 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		if (lyxrc.auto_region_delete) {
 			if (cur.selection())
 				cutSelection(cur, false, false);
-			bv->owner()->gui().selection().haveSelection(false);
+			theSelection().haveSelection(false);
 		}
 
 		cur.clearSelection();
 		LyXFont const old_font = real_current_font;
 
-		string::const_iterator cit = cmd.argument.begin();
-		string::const_iterator end = cmd.argument.end();
+		docstring::const_iterator cit = cmd.argument().begin();
+		docstring::const_iterator end = cmd.argument().end();
 		for (; cit != end; ++cit)
-			bv->owner()->getIntl().getTransManager().
+#if 0
+			bv->getIntl().getTransManager().
 				translateAndInsert(*cit, this);
+#else
+			insertChar(bv->cursor(), *cit);
+#endif
 
 		cur.resetAnchor();
 		moveCursor(cur, false);
@@ -1190,27 +1095,27 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 	case LFUN_URL_INSERT: {
 		InsetCommandParams p("url");
 		string const data = InsetCommandMailer::params2string("url", p);
-		bv->owner()->getDialogs().show("url", data, 0);
+		bv->showInsetDialog("url", data, 0);
 		break;
 	}
 
 	case LFUN_HTML_INSERT: {
 		InsetCommandParams p("htmlurl");
 		string const data = InsetCommandMailer::params2string("url", p);
-		bv->owner()->getDialogs().show("url", data, 0);
+		bv->showInsetDialog("url", data, 0);
 		break;
 	}
 
 	case LFUN_LABEL_INSERT: {
+		InsetCommandParams p("label");
 		// Try to generate a valid label
-		string const contents = cmd.argument.empty() ?
-			cur.getPossibleLabel() : cmd.argument;
-
-		InsetCommandParams p("label", contents);
+		p["name"] = (cmd.argument().empty()) ?
+			cur.getPossibleLabel() :
+			cmd.argument();
 		string const data = InsetCommandMailer::params2string("label", p);
 
-		if (cmd.argument.empty()) {
-			bv->owner()->getDialogs().show("label", data, 0);
+		if (cmd.argument().empty()) {
+			bv->showInsetDialog("label", data, 0);
 		} else {
 			FuncRequest fr(LFUN_INSET_INSERT, data);
 			dispatch(cur, fr);
@@ -1245,7 +1150,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		if (doInsertInset(cur, this, cmd, false, true))
 			cur.posRight();
 		else
-			bv->owner()->getDialogs().show("tabularcreate");
+			bv->showDialog("tabularcreate");
 
 		break;
 
@@ -1259,7 +1164,8 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		cur.dispatch(FuncRequest(LFUN_LAYOUT, "Caption"));
 		break;
 
-	case LFUN_INDEX_INSERT: {
+	case LFUN_INDEX_INSERT:
+	case LFUN_NOMENCL_INSERT: {
 		InsetBase * inset = createInset(&cur.bv(), cmd);
 		if (!inset)
 			break;
@@ -1268,11 +1174,17 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		cur.clearSelection();
 		insertInset(cur, inset);
 		inset->edit(cur, true);
+		// Show the dialog for the nomenclature entry, since the
+		// description entry still needs to be filled in.
+		if (cmd.action == LFUN_NOMENCL_INSERT)
+			InsetCommandMailer("nomenclature",
+				*reinterpret_cast<InsetCommand *>(inset)).showDialog(&cur.bv());
 		cur.posRight();
 		break;
 	}
 
 	case LFUN_INDEX_PRINT:
+	case LFUN_NOMENCL_PRINT:
 	case LFUN_TOC_INSERT:
 	case LFUN_HFILL_INSERT:
 	case LFUN_LINE_INSERT:
@@ -1296,7 +1208,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_MATH_IMPORT_SELECTION:
 	case LFUN_MATH_MODE:
-		if (cmd.argument == "on")
+		if (cmd.argument() == "on")
 			// don't pass "on" as argument
 			mathDispatch(cur, FuncRequest(LFUN_MATH_MODE), false);
 		else
@@ -1304,15 +1216,15 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		break;
 
 	case LFUN_MATH_MACRO:
-		if (cmd.argument.empty())
-			cur.errorMessage(N_("Missing argument"));
+		if (cmd.argument().empty())
+			cur.errorMessage(from_utf8(N_("Missing argument")));
 		else {
-			string s = cmd.argument;
+			string s = to_utf8(cmd.argument());
 			string const s1 = token(s, ' ', 1);
 			int const nargs = s1.empty() ? 0 : convert<int>(s1);
 			string const s2 = token(s, ' ', 2);
 			string const type = s2.empty() ? "newcommand" : s2;
-			cur.insert(new MathMacroTemplate(token(s, ' ', 0), nargs, type));
+			cur.insert(new MathMacroTemplate(from_utf8(token(s, ' ', 0)), nargs, from_utf8(type)));
 			//cur.nextInset()->edit(cur, true);
 		}
 		break;
@@ -1329,7 +1241,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 	case LFUN_MATH_MATRIX:
 	case LFUN_MATH_DELIM:
 	case LFUN_MATH_BIGDELIM: {
-		cur.insert(new MathHullInset("simple"));
+		cur.insert(new InsetMathHull(hullSimple));
 		cur.dispatch(FuncRequest(LFUN_CHAR_FORWARD));
 		cur.dispatch(cmd);
 		break;
@@ -1392,13 +1304,13 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 	case LFUN_FONT_SIZE: {
 		LyXFont font(LyXFont::ALL_IGNORE);
-		font.setLyXSize(cmd.argument);
+		font.setLyXSize(to_utf8(cmd.argument()));
 		toggleAndShow(cur, this, font);
 		break;
 	}
 
 	case LFUN_LANGUAGE: {
-		Language const * lang = languages.getLanguage(cmd.argument);
+		Language const * lang = languages.getLanguage(to_utf8(cmd.argument()));
 		if (!lang)
 			break;
 		LyXFont font(LyXFont::ALL_IGNORE);
@@ -1418,7 +1330,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 	case LFUN_FONT_FREE_UPDATE: {
 		LyXFont font;
 		bool toggle;
-		if (bv_funcs::string2font(cmd.argument, font, toggle)) {
+		if (bv_funcs::string2font(to_utf8(cmd.argument()), font, toggle)) {
 			freefont = font;
 			toggleall = toggle;
 			toggleAndShow(cur, this, freefont, toggleall);
@@ -1450,13 +1362,11 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		string data;
 		params2string(cur.paragraph(), data);
 		data = "show\n" + data;
-		bv->owner()->getDialogs().show("paragraph", data);
+		bv->showDialogWithData("paragraph", data);
 		break;
 	}
 
 	case LFUN_PARAGRAPH_UPDATE: {
-		if (!bv->owner()->getDialogs().visible("paragraph"))
-			break;
 		string data;
 		params2string(cur.paragraph(), data);
 
@@ -1464,7 +1374,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		bool const accept = !cur.inset().forceDefaultParagraphs(cur.idx());
 
 		data = "update " + convert<string>(accept) + '\n' + data;
-		bv->owner()->getDialogs().update("paragraph", data);
+		bv->updateDialog("paragraph", data);
 		break;
 	}
 
@@ -1485,15 +1395,16 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 	case LFUN_ACCENT_HUNGARIAN_UMLAUT:
 	case LFUN_ACCENT_CIRCLE:
 	case LFUN_ACCENT_OGONEK:
-		bv->owner()->getLyXFunc().handleKeyFunc(cmd.action);
-		if (!cmd.argument.empty())
-			bv->owner()->getIntl().getTransManager()
-				.translateAndInsert(cmd.argument[0], this);
+		theLyXFunc().handleKeyFunc(cmd.action);
+		if (!cmd.argument().empty())
+			// FIXME: Are all these characters encoded in one byte in utf8?
+			bv->getIntl().getTransManager()
+				.translateAndInsert(cmd.argument()[0], this, cur);
 		break;
 
 	case LFUN_FLOAT_LIST: {
 		LyXTextClass const & tclass = bv->buffer()->params().getLyXTextClass();
-		if (tclass.floats().typeExist(cmd.argument)) {
+		if (tclass.floats().typeExist(to_utf8(cmd.argument()))) {
 			// not quite sure if we want this...
 			recordUndo(cur);
 			cur.clearSelection();
@@ -1505,12 +1416,12 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 			}
 
 			setLayout(cur, tclass.defaultLayoutName());
-			setParagraph(cur, Spacing(), LYX_ALIGN_LAYOUT, string(), 0);
-			insertInset(cur, new InsetFloatList(cmd.argument));
+			setParagraph(cur, Spacing(), LYX_ALIGN_LAYOUT, docstring(), 0);
+			insertInset(cur, new InsetFloatList(to_utf8(cmd.argument())));
 			cur.posRight();
 		} else {
 			lyxerr << "Non-existent float type: "
-			       << cmd.argument << endl;
+			       << to_utf8(cmd.argument()) << endl;
 		}
 		break;
 	}
@@ -1526,17 +1437,17 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 	}
 
 	case LFUN_THESAURUS_ENTRY: {
-		string arg = cmd.argument;
+		docstring arg = cmd.argument();
 		if (arg.empty()) {
 			arg = cur.selectionAsString(false);
 			// FIXME
 			if (arg.size() > 100 || arg.empty()) {
 				// Get word or selection
-				selectWordWhenUnderCursor(cur, lyx::WHOLE_WORD);
+				selectWordWhenUnderCursor(cur, WHOLE_WORD);
 				arg = cur.selectionAsString(false);
 			}
 		}
-		bv->owner()->getDialogs().show("thesaurus", arg);
+		bv->showDialogWithData("thesaurus", to_utf8(arg));
 		break;
 	}
 
@@ -1544,29 +1455,20 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		// Given data, an encoding of the ParagraphParameters
 		// generated in the Paragraph dialog, this function sets
 		// the current paragraph appropriately.
-		istringstream is(cmd.argument);
+		istringstream is(to_utf8(cmd.argument()));
 		LyXLex lex(0, 0);
 		lex.setStream(is);
 		ParagraphParameters params;
 		params.read(lex);
 		setParagraph(cur,
-					 params.spacing(),
-					 params.align(),
-					 params.labelWidthString(),
-					 params.noindent());
+			     params.spacing(),
+			     params.align(),
+			     params.labelWidthString(),
+			     params.noindent());
 		cur.message(_("Paragraph layout set"));
 		break;
 	}
 
-	case LFUN_INSET_DIALOG_SHOW: {
-		InsetBase * inset = cur.nextInset();
-		if (inset) {
-			FuncRequest fr(LFUN_INSET_DIALOG_SHOW);
-			inset->dispatch(cur, fr);
-		}
-		break;
-	}
-
 	case LFUN_ESCAPE:
 		if (cur.selection()) {
 			cur.selection() = false;
@@ -1585,26 +1487,40 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 		break;
 	}
 
+	needsUpdate |= (cur.pos() != cur.lastpos()) && cur.selection();
+
+	// FIXME: The cursor flag is reset two lines below
+	// so we need to check here if some of the LFUN did touch that.
+	// for now only LyXText::erase() and LyXText::backspace() do that.
+	// The plan is to verify all the LFUNs and then to remove this
+	// singleParUpdate boolean altogether.
+	if (cur.result().update() & Update::Force) {
+		singleParUpdate = false;
+		needsUpdate = true;
+	}
+
+	// FIXME: the following code should go in favor of fine grained
+	// update flag treatment.
 	if (singleParUpdate)
 		// Inserting characters does not change par height
 		if (cur.bottom().paragraph().dim().height()
 		    == olddim.height()) {
 			// if so, update _only_ this paragraph
-			cur.bv().update(Update::SinglePar |
-					Update::FitCursor |
-					Update::MultiParSel);
-			cur.noUpdate();
+			cur.updateFlags(Update::SinglePar |
+				Update::FitCursor |
+				Update::MultiParSel);
 			return;
 		} else
 			needsUpdate = true;
+
 	if (!needsUpdate
 	    && &oldTopSlice.inset() == &cur.inset()
 	    && oldTopSlice.idx() == cur.idx()
-	    && !sel
+	    && !sel // sel is a backup of cur.selection() at the biginning of the function.
 	    && !cur.selection())
 		cur.noUpdate();
 	else
-		cur.needsUpdate();
+		cur.updateFlags(Update::Force | Update::FitCursor);
 }
 
 
@@ -1636,41 +1552,43 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 		break;
 
 	case LFUN_DIALOG_SHOW_NEW_INSET:
-		if (cmd.argument == "bibitem")
+		if (cmd.argument() == "bibitem")
 			code = InsetBase::BIBITEM_CODE;
-		else if (cmd.argument == "bibtex")
+		else if (cmd.argument() == "bibtex")
 			code = InsetBase::BIBTEX_CODE;
-		else if (cmd.argument == "box")
+		else if (cmd.argument() == "box")
 			code = InsetBase::BOX_CODE;
-		else if (cmd.argument == "branch")
+		else if (cmd.argument() == "branch")
 			code = InsetBase::BRANCH_CODE;
-		else if (cmd.argument == "citation")
+		else if (cmd.argument() == "citation")
 			code = InsetBase::CITE_CODE;
-		else if (cmd.argument == "ert")
+		else if (cmd.argument() == "ert")
 			code = InsetBase::ERT_CODE;
-		else if (cmd.argument == "external")
+		else if (cmd.argument() == "external")
 			code = InsetBase::EXTERNAL_CODE;
-		else if (cmd.argument == "float")
+		else if (cmd.argument() == "float")
 			code = InsetBase::FLOAT_CODE;
-		else if (cmd.argument == "graphics")
+		else if (cmd.argument() == "graphics")
 			code = InsetBase::GRAPHICS_CODE;
-		else if (cmd.argument == "include")
+		else if (cmd.argument() == "include")
 			code = InsetBase::INCLUDE_CODE;
-		else if (cmd.argument == "index")
+		else if (cmd.argument() == "index")
 			code = InsetBase::INDEX_CODE;
-		else if (cmd.argument == "label")
+		else if (cmd.argument() == "nomenclature")
+			code = InsetBase::NOMENCL_CODE;
+		else if (cmd.argument() == "label")
 			code = InsetBase::LABEL_CODE;
-		else if (cmd.argument == "note")
+		else if (cmd.argument() == "note")
 			code = InsetBase::NOTE_CODE;
-		else if (cmd.argument == "ref")
+		else if (cmd.argument() == "ref")
 			code = InsetBase::REF_CODE;
-		else if (cmd.argument == "toc")
+		else if (cmd.argument() == "toc")
 			code = InsetBase::TOC_CODE;
-		else if (cmd.argument == "url")
+		else if (cmd.argument() == "url")
 			code = InsetBase::URL_CODE;
-		else if (cmd.argument == "vspace")
+		else if (cmd.argument() == "vspace")
 			code = InsetBase::VSPACE_CODE;
-		else if (cmd.argument == "wrap")
+		else if (cmd.argument() == "wrap")
 			code = InsetBase::WRAP_CODE;
 		break;
 
@@ -1740,6 +1658,12 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 	case LFUN_INDEX_PRINT:
 		code = InsetBase::INDEX_PRINT_CODE;
 		break;
+	case LFUN_NOMENCL_INSERT:
+		code = InsetBase::NOMENCL_CODE;
+		break;
+	case LFUN_NOMENCL_PRINT:
+		code = InsetBase::NOMENCL_PRINT_CODE;
+		break;
 	case LFUN_TOC_INSERT:
 		code = InsetBase::TOC_CODE;
 		break;
@@ -1765,25 +1689,6 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 			code = InsetBase::SPACE_CODE;
 		break;
 
-#ifdef WITH_WARNINGS
-#warning This LFUN is not used anymore and should be nuked (JMarc 29/10/2005)
-#endif
-#if 0
-	case LFUN_INSET_DIALOG_SHOW: {
-		InsetBase * inset = cur.nextInset();
-		enable = inset;
-		if (inset) {
-			code = inset->lyxCode();
-			if (!(code == InsetBase::INCLUDE_CODE
-				|| code == InsetBase::BIBTEX_CODE
-				|| code == InsetBase::FLOAT_LIST_CODE
-				|| code == InsetBase::TOC_CODE))
-				enable = false;
-		}
-		break;
-	}
-#endif
-
 	case LFUN_INSET_MODIFY:
 		// We need to disable this, because we may get called for a
 		// tabular cell via
@@ -1822,18 +1727,25 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 		break;
 
 	case LFUN_PASTE:
-		enable = lyx::cap::numberOfSelections() > 0;
+		enable = cap::numberOfSelections() > 0;
 		break;
 
-	case LFUN_PARAGRAPH_MOVE_UP: {
+	case LFUN_PARAGRAPH_MOVE_UP:
 		enable = cur.pit() > 0 && !cur.selection();
 		break;
-	}
 
-	case LFUN_PARAGRAPH_MOVE_DOWN: {
+	case LFUN_PARAGRAPH_MOVE_DOWN:
 		enable = cur.pit() < cur.lastpit() && !cur.selection();
 		break;
-	}
+
+	case LFUN_INSET_DISSOLVE:
+		enable = !isMainText(*cur.bv().buffer()) && cur.inset().nargs() == 1;
+		break;
+
+	case LFUN_CHANGE_ACCEPT:
+	case LFUN_CHANGE_REJECT:
+		enable = true; // FIXME: Change tracking (MG)
+		break;
 
 	case LFUN_WORD_DELETE_FORWARD:
 	case LFUN_WORD_DELETE_BACKWARD:
@@ -1873,7 +1785,6 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 	case LFUN_BREAK_PARAGRAPH_SKIP:
 	case LFUN_PARAGRAPH_SPACING:
 	case LFUN_INSET_INSERT:
-	case LFUN_NEXT_INSET_TOGGLE:
 	case LFUN_WORD_UPCASE:
 	case LFUN_WORD_LOWCASE:
 	case LFUN_WORD_CAPITALIZE:
@@ -1893,7 +1804,6 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 	case LFUN_MATH_IMPORT_SELECTION:
 	case LFUN_MATH_MODE:
 	case LFUN_MATH_MACRO:
-	case LFUN_MATH_INSERT:
 	case LFUN_MATH_MATRIX:
 	case LFUN_MATH_DELIM:
 	case LFUN_MATH_BIGDELIM:
@@ -1924,8 +1834,6 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 	case LFUN_ACCENT_HUNGARIAN_UMLAUT:
 	case LFUN_ACCENT_CIRCLE:
 	case LFUN_ACCENT_OGONEK:
-	case LFUN_CHANGE_ACCEPT:
-	case LFUN_CHANGE_REJECT:
 	case LFUN_THESAURUS_ENTRY:
 	case LFUN_PARAGRAPH_PARAMS_APPLY:
 	case LFUN_ESCAPE:
@@ -1933,6 +1841,7 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 	case LFUN_BUFFER_BEGIN:
 	case LFUN_BUFFER_BEGIN_SELECT:
 	case LFUN_BUFFER_END_SELECT:
+	case LFUN_UNICODE_INSERT:
 		// these are handled in our dispatch()
 		enable = true;
 		break;
@@ -1948,3 +1857,6 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 	flag.enabled(enable);
 	return true;
 }
+
+
+} // namespace lyx