Martin Vermeer wrote:
> > could somebody please (try to) confirm this crash in branch? Assuming
> > it's new, I think it should be fixed for 1.5.3.
> >
> > http://bugzilla.lyx.org/show_bug.cgi?id=4323
>
> Those are actually two different crashes (haven't confirmed).
>
> The second one was probably exposed by my off-by-one error fix. What
> happens is that an empty column is added at current cursor location, but
> the cursor position within the cell is not reset to 0, so it remains
> suspended in mid-air at a non-existent pos. Before my fix, the empty
> column was inserted one column to the right ;-/
And here's a patch for the other one (delete column while there is a
selection). Simply clear the selection before deleting the column.
Does this make sense?
> I'll have a look.
Yes, please.
Jürgen
Index: src/mathed/InsetMathGrid.cpp
===================================================================
--- src/mathed/InsetMathGrid.cpp (Revision 21423)
+++ src/mathed/InsetMathGrid.cpp (Arbeitskopie)
@@ -1162,6 +1162,7 @@
for (int i = 0, n = extractInt(is); i < n; ++i)
addRow(cur.row());
else if (s == "delete-row") {
+ cur.clearSelection(); // bug 4323
for (int i = 0, n = extractInt(is); i < n; ++i) {
delRow(cur.row());
if (cur.idx() >= nargs())
