Michael Gerz wrote:
> Which patches require more testing? Jürgen, how about your patches? I
> somehow lost track of things.

The tabular undo patch (to fix bugs 1937 and 1986) needs testing. I have once 
again attached an updated version, since insettabular has seen several 
changes recently. 

Martin's tabular metrics patch (1765) needs testing, too.

Regards,
Jürgen
Index: insets/insettabular.C
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/insets/insettabular.C,v
retrieving revision 1.486
diff -u -r1.486 insettabular.C
--- insets/insettabular.C	8 Sep 2005 15:29:40 -0000	1.486
+++ insets/insettabular.C	10 Sep 2005 12:20:19 -0000
@@ -631,7 +631,7 @@
 	case LFUN_CUT:
 		if (tablemode(cur)) {
 			if (copySelection(cur)) {
-				recordUndo(cur, Undo::DELETE);
+				recordUndoInset(cur, Undo::DELETE);
 				cutSelection(cur);
 			}
 		}
@@ -641,9 +641,10 @@
 
 	case LFUN_BACKSPACE:
 	case LFUN_DELETE:
-		recordUndo(cur, Undo::DELETE);
-		if (tablemode(cur))
+		if (tablemode(cur)) {
+			recordUndoInset(cur, Undo::DELETE);
 			cutSelection(cur);
+		}
 		else
 			cell(cur.idx())->dispatch(cur, cmd);
 		break;
@@ -730,7 +731,7 @@
 
 	case LFUN_PASTE:
 		if (hasPasteBuffer() && tabularStackDirty()) {
-			recordUndo(cur, Undo::INSERT);
+			recordUndoInset(cur, Undo::INSERT);
 			pasteSelection(cur);
 			break;
 		}
@@ -1403,7 +1404,7 @@
 		break;
 	}
 
-	recordUndo(cur, Undo::ATOMIC);
+	recordUndoInset(cur, Undo::ATOMIC);
 
 	getSelection(cur, sel_row_start, sel_row_end, sel_col_start, sel_col_end);
 	row_type const row = tabular.row_of_cell(cur.idx());

Reply via email to