2 files changed, 46 insertions, 46 deletions

diff --git a/editors/sed.c b/editors/sed.c
index e18e48ab5..2c64ad500 100644
--- a/editors/sed.c
+++ b/editors/sed.c
@@ -395,7 +395,9 @@ static int parse_subst_cmd(sed_cmd_t *sed_cmd, const char *substr)
         /* process the flags */
 
         sed_cmd->which_match = 1;
+        dbg("s flags:'%s'", substr + idx + 1);
         while (substr[++idx]) {
+                dbg("s flag:'%c'", substr[idx]);
                 /* Parse match number */
                 if (isdigit(substr[idx])) {
                         if (match[0] != '^') {
@@ -403,7 +405,7 @@ static int parse_subst_cmd(sed_cmd_t *sed_cmd, const char *substr)
                                 const char *pos = substr + idx;
 /* FIXME: error check? */
                                 sed_cmd->which_match = (unsigned)strtol(substr+idx, (char**) &pos, 10);
-                                idx = pos - substr;
+                                idx = pos - substr - 1;
                         }
                         continue;
                 }
@@ -443,6 +445,7 @@ static int parse_subst_cmd(sed_cmd_t *sed_cmd, const char *substr)
                 case '}':
                         goto out;
                 default:
+                        dbg("s bad flags:'%s'", substr + idx);
                         bb_error_msg_and_die("bad option in substitution expression");
                 }
         }
@@ -1519,12 +1522,16 @@ int sed_main(int argc UNUSED_PARAM, char **argv)
 
                         /* -i: process each FILE separately: */
 
+                        if (stat(*argv, &statbuf) != 0) {
+                                bb_simple_perror_msg(*argv);
+                                G.exitcode = EXIT_FAILURE;
+                                G.current_input_file++;
+                                continue;
+                        }
                         G.outname = xasprintf("%sXXXXXX", *argv);
                         nonstdoutfd = xmkstemp(G.outname);
                         G.nonstdout = xfdopen_for_write(nonstdoutfd);
-
                         /* Set permissions/owner of output file */
-                        stat(*argv, &statbuf);
                         /* chmod'ing AFTER chown would preserve suid/sgid bits,
                          * but GNU sed 4.2.1 does not preserve them either */
                         fchmod(nonstdoutfd, statbuf.st_mode);
diff --git a/editors/vi.c b/editors/vi.c
index a6505e0bf..0c6906c6b 100644
--- a/editors/vi.c
+++ b/editors/vi.c
@@ -566,8 +566,7 @@ static void redraw(int);	// force a full screen refresh
 static char* format_line(char* /*, int*/);
 static void refresh(int);       // update the terminal from screen[]
 
-static void Indicate_Error(void);       // use flash or beep to indicate error
-#define indicate_error(c) Indicate_Error()
+static void indicate_error(void);       // use flash or beep to indicate error
 static void Hit_Return(void);
 
 #if ENABLE_FEATURE_VI_SEARCH
@@ -1840,11 +1839,11 @@ static char *bound_dot(char *p) // make sure  text[0] <= P < "end"
 {
         if (p >= end && end > text) {
                 p = end - 1;
-                indicate_error('1');
+                indicate_error();
         }
         if (p < text) {
                 p = text;
-                indicate_error('2');
+                indicate_error();
         }
         return p;
 }
@@ -2023,16 +2022,9 @@ static char *char_insert(char *p, char c, int undo) // insert the char c at 'p'
                         p = text_hole_delete(p, p, ALLOW_UNDO_QUEUED);  // shrink buffer 1 char
                 }
         } else {
-#if ENABLE_FEATURE_VI_SETOPTS
                 // insert a char into text[]
-                char *sp;               // "save p"
-#endif
-
                 if (c == 13)
                         c = '\n';       // translate \r to \n
-#if ENABLE_FEATURE_VI_SETOPTS
-                sp = p;                 // remember addr of insert
-#endif
 #if ENABLE_FEATURE_VI_UNDO
 # if ENABLE_FEATURE_VI_UNDO_QUEUE
                 if (c == '\n')
@@ -2056,8 +2048,8 @@ static char *char_insert(char *p, char c, int undo) // insert the char c at 'p'
 #endif /* ENABLE_FEATURE_VI_UNDO */
                 p += 1 + stupid_insert(p, c);   // insert the char
 #if ENABLE_FEATURE_VI_SETOPTS
-                if (showmatch && strchr(")]}", *sp) != NULL) {
-                        showmatching(sp);
+                if (showmatch && strchr(")]}", c) != NULL) {
+                        showmatching(p - 1);
                 }
                 if (autoindent && c == '\n') {  // auto indent the new line
                         char *q;
@@ -2217,34 +2209,32 @@ static char *skip_thing(char *p, int linecnt, int dir, int type)
 }
 
 // find matching char of pair  ()  []  {}
+// will crash if c is not one of these
 static char *find_pair(char *p, const char c)
 {
-        char match, *q;
+        const char *braces = "()[]{}";
+        char match;
         int dir, level;
 
-        match = ')';
+        dir = strchr(braces, c) - braces;
+        dir ^= 1;
+        match = braces[dir];
+        dir = ((dir & 1) << 1) - 1; /* 1 for ([{, -1 for )\} */
+
+        // look for match, count levels of pairs  (( ))
         level = 1;
-        dir = 1;                        // assume forward
-        switch (c) {
-        case '(': match = ')'; break;
-        case '[': match = ']'; break;
-        case '{': match = '}'; break;
-        case ')': match = '('; dir = -1; break;
-        case ']': match = '['; dir = -1; break;
-        case '}': match = '{'; dir = -1; break;
-        }
-        for (q = p + dir; text <= q && q < end; q += dir) {
-                // look for match, count levels of pairs  (( ))
-                if (*q == c)
+        for (;;) {
+                p += dir;
+                if (p < text || p >= end)
+                        return NULL;
+                if (*p == c)
                         level++;        // increase pair levels
-                if (*q == match)
+                if (*p == match) {
                         level--;        // reduce pair level
-                if (level == 0)
-                        break;          // found matching pair
+                        if (level == 0)
+                                return p; // found matching pair
+                }
         }
-        if (level != 0)
-                q = NULL;               // indicate no match
-        return q;
 }
 
 #if ENABLE_FEATURE_VI_SETOPTS
@@ -2256,7 +2246,7 @@ static void showmatching(char *p)
         // we found half of a pair
         q = find_pair(p, *p);   // get loc of matching char
         if (q == NULL) {
-                indicate_error('3');    // no matching char
+                indicate_error();       // no matching char
         } else {
                 // "q" now points to matching pair
                 save_dot = dot; // remember where we are
@@ -2823,6 +2813,9 @@ static int mysleep(int hund)	// sleep for 'hund' 1/100 seconds or stdin ready
 #else
         struct pollfd pfd[1];
 
+        if (hund != 0)
+                fflush_all();
+
         pfd[0].fd = STDIN_FILENO;
         pfd[0].events = POLLIN;
         return safe_poll(pfd, 1, hund*10) > 0;
@@ -3106,7 +3099,7 @@ static void flash(int h)
         redraw(TRUE);
 }
 
-static void Indicate_Error(void)
+static void indicate_error(void)
 {
 #if ENABLE_FEATURE_VI_CRASHME
         if (crashme > 0)
@@ -3649,7 +3642,7 @@ static void do_cmd(int c)
                 break;
         case 27:                        // esc
                 if (cmd_mode == 0)
-                        indicate_error(c);
+                        indicate_error();
                 cmd_mode = 0;   // stop insrting
                 undo_queue_commit();
                 end_cmd_q();
@@ -3668,7 +3661,7 @@ static void do_cmd(int c)
                 if ((unsigned)c1 <= 25) { // a-z?
                         YDreg = c1;
                 } else {
-                        indicate_error(c);
+                        indicate_error();
                 }
                 break;
         case '\'':                      // '- goto a specific mark
@@ -3686,7 +3679,7 @@ static void do_cmd(int c)
                         dot_begin();    // go to B-o-l
                         dot_skip_over_ws();
                 } else {
-                        indicate_error(c);
+                        indicate_error();
                 }
                 break;
         case 'm':                       // m- Mark a line
@@ -3699,7 +3692,7 @@ static void do_cmd(int c)
                         // remember the line
                         mark[c1] = dot;
                 } else {
-                        indicate_error(c);
+                        indicate_error();
                 }
                 break;
         case 'P':                       // P- Put register before
@@ -3760,7 +3753,7 @@ static void do_cmd(int c)
                                 // we found half of a pair
                                 p = find_pair(q, *q);
                                 if (p == NULL) {
-                                        indicate_error(c);
+                                        indicate_error();
                                 } else {
                                         dot = p;
                                 }
@@ -3768,7 +3761,7 @@ static void do_cmd(int c)
                         }
                 }
                 if (*q == '\n')
-                        indicate_error(c);
+                        indicate_error();
                 break;
         case 'f':                       // f- forward to a user specified char
                 last_forward_char = get_one_char();     // get the search char
@@ -4101,7 +4094,7 @@ static void do_cmd(int c)
                 // ZZ means to save file (if necessary), then exit
                 c1 = get_one_char();
                 if (c1 != 'Z') {
-                        indicate_error(c);
+                        indicate_error();
                         break;
                 }
                 if (modified_count) {
@@ -4185,7 +4178,7 @@ static void do_cmd(int c)
                         // could not recognize object
                         c = c1 = 27;    // error-
                         ml = 0;
-                        indicate_error(c);
+                        indicate_error();
                 }
                 if (ml && whole) {
                         if (c == 'c') {