Author: vfr
Date: Sun Jan 2 13:07:33 2011
New Revision: 37063
URL: http://www.lyx.org/trac/changeset/37063
Log:
Fix bug #7200: Crash when replacing newline by InsetQuote.
We have to call cap::replaceSelection before determining whether we can insert
an InsetQuote or a normal quote.
At least we should have updated the par variable after calling
cap::replaceSelection, because the paragraph might have been deleted.
Modified:
lyx-devel/trunk/src/Text3.cpp
Modified: lyx-devel/trunk/src/Text3.cpp
==============================================================================
--- lyx-devel/trunk/src/Text3.cpp Sun Jan 2 12:31:28 2011 (r37062)
+++ lyx-devel/trunk/src/Text3.cpp Sun Jan 2 13:07:33 2011 (r37063)
@@ -1348,33 +1348,37 @@
}
case LFUN_QUOTE_INSERT: {
- Paragraph & par = cur.paragraph();
+ // this avoids a double undo
+ // FIXME: should not be needed, ideally
+ if (!cur.selection())
+ cur.recordUndo();
+ cap::replaceSelection(cur);
+
+ Paragraph const & par = cur.paragraph();
pos_type pos = cur.pos();
- BufferParams const & bufparams = bv->buffer().params();
+
Layout const & style = par.layout();
InsetLayout const & ilayout = cur.inset().getLayout();
- if (!style.pass_thru && !ilayout.isPassThru()
- && par.getFontSettings(bufparams, pos).language()->lang()
!= "hebrew") {
- // this avoids a double undo
- // FIXME: should not be needed, ideally
- if (!cur.selection())
- cur.recordUndo();
- cap::replaceSelection(cur);
- pos = cur.pos();
- char_type c;
- if (pos == 0)
- c = ' ';
- else if (cur.prevInset() && cur.prevInset()->isSpace())
- c = ' ';
- else
+ BufferParams const & bufparams = bv->buffer().params();
+ bool const hebrew =
+ par.getFontSettings(bufparams, pos).language()->lang()
== "hebrew";
+ bool const allow_inset_quote =
+ !(style.pass_thru || ilayout.isPassThru() || hebrew);
+
+ if (allow_inset_quote) {
+ char_type c = ' ';
+ if (pos > 0 && (!cur.prevInset() ||
!cur.prevInset()->isSpace()))
c = par.getChar(pos - 1);
- string arg = to_utf8(cmd.argument());
- cur.insert(new InsetQuotes(cur.buffer(), c, (arg ==
"single")
- ? InsetQuotes::SingleQuotes :
InsetQuotes::DoubleQuotes));
+ string const arg = to_utf8(cmd.argument());
+ InsetQuotes::QuoteTimes const quote_type = (arg ==
"single")
+ ? InsetQuotes::SingleQuotes :
InsetQuotes::DoubleQuotes;
+ cur.insert(new InsetQuotes(cur.buffer(), c,
quote_type));
cur.posForward();
- }
- else
+ } else {
+ // The cursor might have been invalidated by the
replaceSelection.
+ cur.buffer()->changed(true);
lyx::dispatch(FuncRequest(LFUN_SELF_INSERT, "\""));
+ }
break;
}
@@ -2207,7 +2211,6 @@
cur.screenUpdateFlags(Update::SinglePar | Update::FitCursor);
return;
}
-
if (!needsUpdate
&& &oldTopSlice.inset() == &cur.inset()
&& oldTopSlice.idx() == cur.idx()
