From: Chris Allegretta <chrisa@asty.org>
Date: Sat, 2 Aug 2008 22:31:01 +0000 (+0000)
Subject: - Add very basic undo for text replace.
X-Git-Tag: v2.1.3~4
X-Git-Url: https://git.wh0rd.org/?a=commitdiff_plain;h=3c1131a5b37bc81a22e8275625016f2269aff993;p=nano.git

- Add very basic undo for text replace.
- Fix lots of warnings whe using -Wall
- Fix some comments



git-svn-id: svn://svn.savannah.gnu.org/nano/trunk/nano@4290 35c25a1d-7b9e-4130-9fde-d3aeb78583b8
---

diff --git a/src/search.c b/src/search.c
index cee7ac1f..35d3d135 100644
--- a/src/search.c
+++ b/src/search.c
@@ -788,6 +788,9 @@ ssize_t do_replace_loop(
 	    char *copy;
 	    size_t length_change;
 
+#ifndef NANO_TINY
+	    update_undo(REPLACE, openfile);
+#endif
 	    if (i == 2)
 		replaceall = TRUE;
 
diff --git a/src/text.c b/src/text.c
index 861014b8..7bd3a7db 100644
--- a/src/text.c
+++ b/src/text.c
@@ -368,7 +368,7 @@ void do_unindent(void)
 void do_undo(void)
 {
     undo *u = openfile->current_undo;
-    filestruct *f = openfile->current, *t, *t2;
+    filestruct *f = openfile->current, *t;
     int len = 0;
     char *undidmsg, *data;
 
@@ -458,6 +458,12 @@ void do_undo(void)
 	free_filestruct(cutbuffer);
 	cutbuffer = NULL;
 	break;
+    case REPLACE:
+	undidmsg = _("text replace");
+	data = u->strdata;
+	u->strdata = f->data;
+	f->data = data;
+	break;
     default:
 	undidmsg = _("wtf?");
 	break;
@@ -472,8 +478,8 @@ void do_undo(void)
 void do_redo(void)
 {
     undo *u = openfile->undotop;
-    filestruct *f = openfile->current, *t, *t2;
-    int len = 0, i, i2;
+    filestruct *f = openfile->current, *t;
+    int len = 0, i;
     char *undidmsg, *data;
 
     for (; u != NULL && u->next != openfile->current_undo; u = u->next)
@@ -588,6 +594,12 @@ void do_redo(void)
         openfile->mark_begin_x = 0;
 	edit_refresh();
 	break;
+    case REPLACE:
+	undidmsg = _("text replace");
+	data = u->strdata;
+	u->strdata = f->data;
+	f->data = data;
+	break;
     default:
 	undidmsg = _("wtf?");
 	break;
@@ -822,6 +834,7 @@ void add_undo(undo_type current_action, openfilestruct *fs)
 	u->begin = fs->current_x;
 	break;
     case SPLIT:
+    case REPLACE:
 	data = mallocstrcpy(NULL, fs->current->data);
         u->begin = fs->current_x;
 	u->strdata = data;
@@ -835,6 +848,11 @@ void add_undo(undo_type current_action, openfilestruct *fs)
 	}
 	u->to_end = (current_action == CUTTOEND);
 	break;
+    case UNCUT:
+	break;
+    case OTHER:
+	statusbar(_("OOPS. Tried to add unknown thing to undo struct, I'd save your work"));
+	break;
     }
 
 #ifdef DEBUG
@@ -858,8 +876,12 @@ void update_undo(undo_type action, openfilestruct *fs)
 
 
 #ifdef DEBUG
-        fprintf(stderr, "action = %d, fs->last_action = %d,  openfile->current->lineno = %d, fs->current_undo->lineno = %d\n", 
-		action, fs->last_action, openfile->current->lineno,  fs->current_undo->lineno);
+        fprintf(stderr, "action = %d, fs->last_action = %d,  openfile->current->lineno = %d",
+		action, fs->last_action, openfile->current->lineno);
+	if (fs->current_undo)
+	    fprintf(stderr, "fs->current_undo->lineno = %d\n",  fs->current_undo->lineno);
+	else
+	    fprintf(stderr, "\n");
 #endif
 
     /* Change to an add if we're not using the same undo struct
@@ -938,10 +960,13 @@ void update_undo(undo_type action, openfilestruct *fs)
 	u->cutbottom = cutbottom;
 	u->linescut++;
 	break;
+    case REPLACE:
+	add_undo(action, fs);
+	break;
     case SPLIT:
     case UNSPLIT:
-	/* We don't really ever update an enter key press, treat it as a new */
-//	add_undo(action, fs);
+	/* These cases are handled by the earlier check for a new line and action */
+    case OTHER:
 	break;
     }