From 2f8f71b20dbce1328a5175bb75ae15564f7e9489 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Sun, 10 Dec 2006 02:09:12 +0000 Subject: sed: style fixes --- editors/sed.c | 666 +++++++++++++++++++++++++++++----------------------------- 1 file changed, 338 insertions(+), 328 deletions(-) diff --git a/editors/sed.c b/editors/sed.c index 94368aa..877d1c4 100644 --- a/editors/sed.c +++ b/editors/sed.c @@ -91,15 +91,14 @@ typedef struct sed_cmd_s { static const char *const semicolon_whitespace = "; \n\r\t\v"; -struct sed_globals -{ +struct sed_globals { /* options */ - int be_quiet, in_place, regex_type; + int be_quiet, regex_type; FILE *nonstdout; char *outname, *hold_space; /* List of input files */ - int input_file_count,current_input_file; + int input_file_count, current_input_file; FILE **input_file_list; regmatch_t regmatch[10]; @@ -123,7 +122,7 @@ struct sed_globals void sed_free_and_close_stuff(void); #if ENABLE_FEATURE_CLEAN_UP -void sed_free_and_close_stuff(void) +static void sed_free_and_close_stuff(void) { sed_cmd_t *sed_cmd = bbg.sed_cmd_head.next; @@ -187,9 +186,9 @@ static void parse_escapes(char *dest, char *string, int len, char from, char to) static char *copy_parsing_escapes(char *string, int len) { - char *dest = xmalloc(len+1); + char *dest = xmalloc(len + 1); - parse_escapes(dest,string,len,'n','\n'); + parse_escapes(dest, string, len, 'n', '\n'); return dest; } @@ -209,7 +208,7 @@ static int index_of_next_unescaped_regexp_delim(int delimiter, char *str) if (delimiter < 0) { bracket--; - delimiter *= -1; + delimiter = -delimiter; } for (; (ch = str[idx]); idx++) { @@ -228,7 +227,7 @@ static int index_of_next_unescaped_regexp_delim(int delimiter, char *str) } /* if we make it to here, we've hit the end of the string */ - bb_error_msg_and_die("unmatched '%c'",delimiter); + bb_error_msg_and_die("unmatched '%c'", delimiter); } /* @@ -279,7 +278,7 @@ static int get_address(char *my_str, int *linenum, regex_t ** regex) delimiter = '/'; if (*my_str == '\\') delimiter = *++pos; next = index_of_next_unescaped_regexp_delim(delimiter, ++pos); - temp = copy_parsing_escapes(pos,next); + temp = copy_parsing_escapes(pos, next); *regex = xmalloc(sizeof(regex_t)); xregcomp(*regex, temp, bbg.regex_type|REG_NEWLINE); free(temp); @@ -336,9 +335,10 @@ static int parse_subst_cmd(sed_cmd_t *sed_cmd, char *substr) if (isdigit(substr[idx])) { if (match[0] != '^') { /* Match 0 treated as all, multiple matches we take the last one. */ - char *pos = substr+idx; - sed_cmd->which_match = (unsigned short)strtol(substr+idx,&pos,10); - idx = pos-substr; + char *pos = substr + idx; + /* FIXME: error check? */ + sed_cmd->which_match = (unsigned short)strtol(substr+idx, &pos, 10); + idx = pos - substr; } continue; } @@ -358,7 +358,7 @@ static int parse_subst_cmd(sed_cmd_t *sed_cmd, char *substr) case 'w': { char *temp; - idx += parse_file_cmd(sed_cmd,substr+idx,&temp); + idx += parse_file_cmd(sed_cmd, substr+idx, &temp); break; } @@ -413,7 +413,7 @@ static char *parse_cmd_args(sed_cmd_t *sed_cmd, char *cmdstr) break; } sed_cmd->string = xstrdup(cmdstr); - parse_escapes(sed_cmd->string,sed_cmd->string,strlen(cmdstr),0,0); + parse_escapes(sed_cmd->string, sed_cmd->string, strlen(cmdstr), 0, 0); cmdstr += strlen(cmdstr); /* handle file cmds: (r)ead */ } else if (strchr("rw", sed_cmd->cmd)) { @@ -421,7 +421,7 @@ static char *parse_cmd_args(sed_cmd_t *sed_cmd, char *cmdstr) bb_error_msg_and_die("command only uses one address"); cmdstr += parse_file_cmd(sed_cmd, cmdstr, &sed_cmd->string); if (sed_cmd->cmd == 'w') - sed_cmd->file = xfopen(sed_cmd->string,"w"); + sed_cmd->file = xfopen(sed_cmd->string, "w"); /* handle branch commands */ } else if (strchr(":btT", sed_cmd->cmd)) { int length; @@ -440,8 +440,8 @@ static char *parse_cmd_args(sed_cmd_t *sed_cmd, char *cmdstr) cmdstr += parse_regex_delim(cmdstr, &match, &replace)+1; /* \n already parsed, but \delimiter needs unescaping. */ - parse_escapes(match,match,strlen(match),i,i); - parse_escapes(replace,replace,strlen(replace),i,i); + parse_escapes(match, match, strlen(match), i, i); + parse_escapes(replace, replace, strlen(replace), i, i); sed_cmd->string = xzalloc((strlen(match) + 1) * 2); for (i = 0; match[i] && replace[i]; i++) { @@ -641,7 +641,7 @@ static int do_subst_command(sed_cmd_t *sed_cmd, char **line) /* If we aren't interested in this match, output old line to end of match and continue */ - if (sed_cmd->which_match && sed_cmd->which_match!=match_count) { + if (sed_cmd->which_match && sed_cmd->which_match != match_count) { for (i = 0; i < bbg.regmatch[0].rm_eo; i++) pipe_putc(*oldline++); continue; @@ -698,7 +698,7 @@ static void flush_append(void) /* Output appended lines. */ while ((data = (char *)llist_pop(&bbg.append_head))) { - fprintf(bbg.nonstdout,"%s\n",data); + fprintf(bbg.nonstdout, "%s\n", data); free(data); } } @@ -801,325 +801,327 @@ static void process_files(void) next_line = get_next_line(&next_last_char); /* go through every line in each file */ - for (;;) { - sed_cmd_t *sed_cmd; - int substituted = 0; - - /* Advance to next line. Stop if out of lines. */ - pattern_space = next_line; - if (!pattern_space) break; - last_char = next_last_char; - - /* Read one line in advance so we can act on the last line, - * the '$' address */ - next_line = get_next_line(&next_last_char); - linenum++; -restart: - /* for every line, go through all the commands */ - for (sed_cmd = bbg.sed_cmd_head.next; sed_cmd; sed_cmd = sed_cmd->next) { - int old_matched, matched; - - old_matched = sed_cmd->in_match; - - /* Determine if this command matches this line: */ - - /* Are we continuing a previous multi-line match? */ - sed_cmd->in_match = sed_cmd->in_match - /* Or is no range necessary? */ - || (!sed_cmd->beg_line && !sed_cmd->end_line - && !sed_cmd->beg_match && !sed_cmd->end_match) - /* Or did we match the start of a numerical range? */ - || (sed_cmd->beg_line > 0 && (sed_cmd->beg_line == linenum)) - /* Or does this line match our begin address regex? */ - || (sed_cmd->beg_match && - !regexec(sed_cmd->beg_match, pattern_space, 0, NULL, 0)) - /* Or did we match last line of input? */ - || (sed_cmd->beg_line == -1 && next_line == NULL); - - /* Snapshot the value */ - - matched = sed_cmd->in_match; - - /* Is this line the end of the current match? */ - - if (matched) { - sed_cmd->in_match = !( - /* has the ending line come, or is this a single address command? */ - (sed_cmd->end_line ? - sed_cmd->end_line==-1 ? - !next_line - : sed_cmd->end_line<=linenum - : !sed_cmd->end_match) - /* or does this line matches our last address regex */ - || (sed_cmd->end_match && old_matched && (regexec(sed_cmd->end_match, pattern_space, 0, NULL, 0) == 0)) - ); - } +again: - /* Skip blocks of commands we didn't match. */ - if (sed_cmd->cmd == '{') { - if (sed_cmd->invert ? matched : !matched) - while (sed_cmd && sed_cmd->cmd != '}') - sed_cmd = sed_cmd->next; - if (!sed_cmd) bb_error_msg_and_die("unterminated {"); - continue; - } + sed_cmd_t *sed_cmd; + int substituted = 0; - /* Okay, so did this line match? */ - if (sed_cmd->invert ? !matched : matched) { - /* Update last used regex in case a blank substitute BRE is found */ - if (sed_cmd->beg_match) { - bbg.previous_regex_ptr = sed_cmd->beg_match; - } + /* Advance to next line. Stop if out of lines. */ + pattern_space = next_line; + if (!pattern_space) return; + last_char = next_last_char; - /* actual sedding */ - switch (sed_cmd->cmd) { + /* Read one line in advance so we can act on the last line, + * the '$' address */ + next_line = get_next_line(&next_last_char); + linenum++; +restart: + /* for every line, go through all the commands */ + for (sed_cmd = bbg.sed_cmd_head.next; sed_cmd; sed_cmd = sed_cmd->next) { + int old_matched, matched; + + old_matched = sed_cmd->in_match; + + /* Determine if this command matches this line: */ + + /* Are we continuing a previous multi-line match? */ + sed_cmd->in_match = sed_cmd->in_match + /* Or is no range necessary? */ + || (!sed_cmd->beg_line && !sed_cmd->end_line + && !sed_cmd->beg_match && !sed_cmd->end_match) + /* Or did we match the start of a numerical range? */ + || (sed_cmd->beg_line > 0 && (sed_cmd->beg_line == linenum)) + /* Or does this line match our begin address regex? */ + || (sed_cmd->beg_match && + !regexec(sed_cmd->beg_match, pattern_space, 0, NULL, 0)) + /* Or did we match last line of input? */ + || (sed_cmd->beg_line == -1 && next_line == NULL); + + /* Snapshot the value */ + + matched = sed_cmd->in_match; + + /* Is this line the end of the current match? */ + + if (matched) { + sed_cmd->in_match = !( + /* has the ending line come, or is this a single address command? */ + (sed_cmd->end_line ? + sed_cmd->end_line == -1 ? + !next_line + : (sed_cmd->end_line <= linenum) + : !sed_cmd->end_match + ) + /* or does this line matches our last address regex */ + || (sed_cmd->end_match && old_matched + && (regexec(sed_cmd->end_match, + pattern_space, 0, NULL, 0) == 0)) + ); + } - /* Print line number */ - case '=': - fprintf(bbg.nonstdout, "%d\n", linenum); - break; + /* Skip blocks of commands we didn't match. */ + if (sed_cmd->cmd == '{') { + if (sed_cmd->invert ? matched : !matched) + while (sed_cmd && sed_cmd->cmd != '}') + sed_cmd = sed_cmd->next; + if (!sed_cmd) bb_error_msg_and_die("unterminated {"); + continue; + } - /* Write the current pattern space up to the first newline */ - case 'P': - { - char *tmp = strchr(pattern_space, '\n'); + /* Okay, so did this line match? */ + if (sed_cmd->invert ? !matched : matched) { + /* Update last used regex in case a blank substitute BRE is found */ + if (sed_cmd->beg_match) { + bbg.previous_regex_ptr = sed_cmd->beg_match; + } - if (tmp) { - *tmp = '\0'; - sed_puts(pattern_space,1); - *tmp = '\n'; - break; - } - /* Fall Through */ - } + /* actual sedding */ + switch (sed_cmd->cmd) { - /* Write the current pattern space to output */ - case 'p': - sed_puts(pattern_space, last_char); + /* Print line number */ + case '=': + fprintf(bbg.nonstdout, "%d\n", linenum); + break; + + /* Write the current pattern space up to the first newline */ + case 'P': + { + char *tmp = strchr(pattern_space, '\n'); + + if (tmp) { + *tmp = '\0'; + sed_puts(pattern_space, 1); + *tmp = '\n'; break; - /* Delete up through first newline */ - case 'D': - { - char *tmp = strchr(pattern_space,'\n'); - - if (tmp) { - tmp = xstrdup(tmp+1); - free(pattern_space); - pattern_space = tmp; - goto restart; - } } - /* discard this line. */ - case 'd': - goto discard_line; + /* Fall Through */ + } - /* Substitute with regex */ - case 's': - if (!do_subst_command(sed_cmd, &pattern_space)) - break; - substituted |= 1; - - /* handle p option */ - if (sed_cmd->sub_p) - sed_puts(pattern_space, last_char); - /* handle w option */ - if (sed_cmd->file) - sed_cmd->last_char = puts_maybe_newline( - pattern_space, sed_cmd->file, - sed_cmd->last_char, last_char); - break; + /* Write the current pattern space to output */ + case 'p': + sed_puts(pattern_space, last_char); + break; + /* Delete up through first newline */ + case 'D': + { + char *tmp = strchr(pattern_space, '\n'); - /* Append line to linked list to be printed later */ - case 'a': - append(sed_cmd->string); - break; + if (tmp) { + tmp = xstrdup(tmp+1); + free(pattern_space); + pattern_space = tmp; + goto restart; + } + } + /* discard this line. */ + case 'd': + goto discard_line; - /* Insert text before this line */ - case 'i': - sed_puts(sed_cmd->string,1); + /* Substitute with regex */ + case 's': + if (!do_subst_command(sed_cmd, &pattern_space)) break; + substituted |= 1; - /* Cut and paste text (replace) */ - case 'c': - /* Only triggers on last line of a matching range. */ - if (!sed_cmd->in_match) - sed_puts(sed_cmd->string,0); - goto discard_line; - - /* Read file, append contents to output */ - case 'r': - { - FILE *rfile; + /* handle p option */ + if (sed_cmd->sub_p) + sed_puts(pattern_space, last_char); + /* handle w option */ + if (sed_cmd->file) + sed_cmd->last_char = puts_maybe_newline( + pattern_space, sed_cmd->file, + sed_cmd->last_char, last_char); + break; - rfile = fopen(sed_cmd->string, "r"); - if (rfile) { - char *line; + /* Append line to linked list to be printed later */ + case 'a': + append(sed_cmd->string); + break; - while ((line = xmalloc_getline(rfile)) - != NULL) - append(line); - xprint_and_close_file(rfile); - } + /* Insert text before this line */ + case 'i': + sed_puts(sed_cmd->string, 1); + break; - break; + /* Cut and paste text (replace) */ + case 'c': + /* Only triggers on last line of a matching range. */ + if (!sed_cmd->in_match) + sed_puts(sed_cmd->string, 0); + goto discard_line; + + /* Read file, append contents to output */ + case 'r': + { + FILE *rfile; + + rfile = fopen(sed_cmd->string, "r"); + if (rfile) { + char *line; + + while ((line = xmalloc_getline(rfile)) + != NULL) + append(line); + xprint_and_close_file(rfile); } - /* Write pattern space to file. */ - case 'w': - sed_cmd->last_char = puts_maybe_newline( - pattern_space,sed_cmd->file, - sed_cmd->last_char, last_char); - break; + break; + } - /* Read next line from input */ - case 'n': - if (!bbg.be_quiet) - sed_puts(pattern_space, last_char); - if (next_line) { - free(pattern_space); - pattern_space = next_line; - last_char = next_last_char; - next_line = get_next_line(&next_last_char); - linenum++; - break; - } - /* fall through */ + /* Write pattern space to file. */ + case 'w': + sed_cmd->last_char = puts_maybe_newline( + pattern_space, sed_cmd->file, + sed_cmd->last_char, last_char); + break; - /* Quit. End of script, end of input. */ - case 'q': - /* Exit the outer while loop */ + /* Read next line from input */ + case 'n': + if (!bbg.be_quiet) + sed_puts(pattern_space, last_char); + if (next_line) { + free(pattern_space); + pattern_space = next_line; + last_char = next_last_char; + next_line = get_next_line(&next_last_char); + linenum++; + break; + } + /* fall through */ + + /* Quit. End of script, end of input. */ + case 'q': + /* Exit the outer while loop */ + free(next_line); + next_line = NULL; + goto discard_commands; + + /* Append the next line to the current line */ + case 'N': + { + int len; + /* If no next line, jump to end of script and exit. */ + if (next_line == NULL) { + /* Jump to end of script and exit */ free(next_line); next_line = NULL; - goto discard_commands; - - /* Append the next line to the current line */ - case 'N': - { - /* If no next line, jump to end of script and exit. */ - if (next_line == NULL) { - /* Jump to end of script and exit */ - free(next_line); - next_line = NULL; - goto discard_line; - /* append next_line, read new next_line. */ - } else { - int len = strlen(pattern_space); - - pattern_space = realloc(pattern_space, len + strlen(next_line) + 2); - pattern_space[len] = '\n'; - strcpy(pattern_space + len+1, next_line); - last_char = next_last_char; - next_line = get_next_line(&next_last_char); - linenum++; - } - break; + goto discard_line; + /* append next_line, read new next_line. */ } + len = strlen(pattern_space); + pattern_space = realloc(pattern_space, len + strlen(next_line) + 2); + pattern_space[len] = '\n'; + strcpy(pattern_space + len+1, next_line); + last_char = next_last_char; + next_line = get_next_line(&next_last_char); + linenum++; + break; + } - /* Test/branch if substitution occurred */ - case 't': - if (!substituted) break; - substituted = 0; - /* Fall through */ - /* Test/branch if substitution didn't occur */ - case 'T': - if (substituted) break; - /* Fall through */ - /* Branch to label */ - case 'b': - if (!sed_cmd->string) goto discard_commands; - else sed_cmd = branch_to(sed_cmd->string); - break; - /* Transliterate characters */ - case 'y': - { - int i; - - for (i = 0; pattern_space[i]; i++) { - int j; - - for (j = 0; sed_cmd->string[j]; j += 2) { - if (pattern_space[i] == sed_cmd->string[j]) { - pattern_space[i] = sed_cmd->string[j + 1]; - break; - } + /* Test/branch if substitution occurred */ + case 't': + if (!substituted) break; + substituted = 0; + /* Fall through */ + /* Test/branch if substitution didn't occur */ + case 'T': + if (substituted) break; + /* Fall through */ + /* Branch to label */ + case 'b': + if (!sed_cmd->string) goto discard_commands; + else sed_cmd = branch_to(sed_cmd->string); + break; + /* Transliterate characters */ + case 'y': + { + int i, j; + + for (i = 0; pattern_space[i]; i++) { + for (j = 0; sed_cmd->string[j]; j += 2) { + if (pattern_space[i] == sed_cmd->string[j]) { + pattern_space[i] = sed_cmd->string[j + 1]; + break; } } - - break; } - case 'g': /* Replace pattern space with hold space */ - free(pattern_space); - pattern_space = xstrdup(bbg.hold_space ? bbg.hold_space : ""); - break; - case 'G': /* Append newline and hold space to pattern space */ - { - int pattern_space_size = 2; - int hold_space_size = 0; - - if (pattern_space) - pattern_space_size += strlen(pattern_space); - if (bbg.hold_space) - hold_space_size = strlen(bbg.hold_space); - pattern_space = xrealloc(pattern_space, - pattern_space_size + hold_space_size); - if (pattern_space_size == 2) - pattern_space[0] = 0; - strcat(pattern_space, "\n"); - if (bbg.hold_space) - strcat(pattern_space, bbg.hold_space); - last_char = '\n'; - break; - } - case 'h': /* Replace hold space with pattern space */ - free(bbg.hold_space); - bbg.hold_space = xstrdup(pattern_space); - break; - case 'H': /* Append newline and pattern space to hold space */ - { - int hold_space_size = 2; - int pattern_space_size = 0; - - if (bbg.hold_space) - hold_space_size += strlen(bbg.hold_space); - if (pattern_space) - pattern_space_size = strlen(pattern_space); - bbg.hold_space = xrealloc(bbg.hold_space, - hold_space_size + pattern_space_size); - - if (hold_space_size == 2) - *bbg.hold_space = 0; - strcat(bbg.hold_space, "\n"); - if (pattern_space) - strcat(bbg.hold_space, pattern_space); + break; + } + case 'g': /* Replace pattern space with hold space */ + free(pattern_space); + pattern_space = xstrdup(bbg.hold_space ? bbg.hold_space : ""); + break; + case 'G': /* Append newline and hold space to pattern space */ + { + int pattern_space_size = 2; + int hold_space_size = 0; + + if (pattern_space) + pattern_space_size += strlen(pattern_space); + if (bbg.hold_space) + hold_space_size = strlen(bbg.hold_space); + pattern_space = xrealloc(pattern_space, + pattern_space_size + hold_space_size); + if (pattern_space_size == 2) + pattern_space[0] = 0; + strcat(pattern_space, "\n"); + if (bbg.hold_space) + strcat(pattern_space, bbg.hold_space); + last_char = '\n'; - break; - } - case 'x': /* Exchange hold and pattern space */ - { - char *tmp = pattern_space; - pattern_space = bbg.hold_space ? : xzalloc(1); - last_char = '\n'; - bbg.hold_space = tmp; - break; - } - } + break; + } + case 'h': /* Replace hold space with pattern space */ + free(bbg.hold_space); + bbg.hold_space = xstrdup(pattern_space); + break; + case 'H': /* Append newline and pattern space to hold space */ + { + int hold_space_size = 2; + int pattern_space_size = 0; + + if (bbg.hold_space) + hold_space_size += strlen(bbg.hold_space); + if (pattern_space) + pattern_space_size = strlen(pattern_space); + bbg.hold_space = xrealloc(bbg.hold_space, + hold_space_size + pattern_space_size); + + if (hold_space_size == 2) + *bbg.hold_space = 0; + strcat(bbg.hold_space, "\n"); + if (pattern_space) + strcat(bbg.hold_space, pattern_space); + + break; + } + case 'x': /* Exchange hold and pattern space */ + { + char *tmp = pattern_space; + pattern_space = bbg.hold_space ? : xzalloc(1); + last_char = '\n'; + bbg.hold_space = tmp; + break; + } } } + } - /* - * exit point from sedding... - */ + /* + * exit point from sedding... + */ discard_commands: - /* we will print the line unless we were told to be quiet ('-n') - or if the line was suppressed (ala 'd'elete) */ - if (!bbg.be_quiet) sed_puts(pattern_space, last_char); + /* we will print the line unless we were told to be quiet ('-n') + or if the line was suppressed (ala 'd'elete) */ + if (!bbg.be_quiet) sed_puts(pattern_space, last_char); - /* Delete and such jump here. */ + /* Delete and such jump here. */ discard_line: - flush_append(); - free(pattern_space); - } + flush_append(); + free(pattern_space); + + goto again; } /* It is possible to have a command line argument with embedded @@ -1131,7 +1133,7 @@ static void add_cmd_block(char *cmdstr) char *temp = xstrdup(cmdstr), *temp2 = temp; while (go) { - int len = strcspn(temp2,"\n"); + int len = strcspn(temp2, "\n"); if (!temp2[len]) go = 0; else temp2[len] = 0; add_cmd(temp2); @@ -1165,31 +1167,37 @@ static void add_files_link(llist_t *opt_f) int sed_main(int argc, char **argv) { + enum { + OPT_in_place = 1 << 0, + }; unsigned opt; llist_t *opt_e, *opt_f; int status = EXIT_SUCCESS; - bbg.sed_cmd_tail=&bbg.sed_cmd_head; + bbg.sed_cmd_tail = &bbg.sed_cmd_head; /* destroy command strings on exit */ if (ENABLE_FEATURE_CLEAN_UP) atexit(sed_free_and_close_stuff); /* Lie to autoconf when it starts asking stupid questions. */ - if (argc==2 && !strcmp(argv[1],"--version")) { - printf("This is not GNU sed version 4.0\n"); - exit(0); + if (argc == 2 && !strcmp(argv[1], "--version")) { + puts("This is not GNU sed version 4.0"); + return 0; } /* do normal option parsing */ opt_e = opt_f = NULL; - opt_complementary = "e::f::"; /* can occur multiple times */ - opt = getopt32(argc, argv, "irne:f:", &opt_e, &opt_f); - if (opt & 0x1) { // -i - bbg.in_place++; + opt_complementary = "e::f::" /* can occur multiple times */ + "nn"; /* count -n */ + opt = getopt32(argc, argv, "irne:f:", &opt_e, &opt_f, + &bbg.be_quiet); /* counter for -n */ + argc -= optind; + argv += optind; + if (opt & OPT_in_place) { // -i atexit(cleanup_outname); } - if (opt & 0x2) bbg.regex_type|=REG_EXTENDED; // -r - if (opt & 0x4) bbg.be_quiet++; // -n + if (opt & 0x2) bbg.regex_type |= REG_EXTENDED; // -r + //if (opt & 0x4) bbg.be_quiet++; // -n if (opt & 0x8) { // -e /* getopt32 reverses order of arguments, handle it */ add_cmds_link(opt_e); @@ -1198,12 +1206,12 @@ int sed_main(int argc, char **argv) /* getopt32 reverses order of arguments, handle it */ add_files_link(opt_f); } - /* if we didn't get a pattern from -e or -f, use argv[optind] */ + /* if we didn't get a pattern from -e or -f, use argv[0] */ if (!(opt & 0x18)) { - if (argv[optind] == NULL) + if (!argc) bb_show_usage(); - else - add_cmd_block(argv[optind++]); + add_cmd_block(*argv++); + argc--; } /* Flush any unfinished commands. */ add_cmd(""); @@ -1211,11 +1219,11 @@ int sed_main(int argc, char **argv) /* By default, we write to stdout */ bbg.nonstdout = stdout; - /* argv[(optind)..(argc-1)] should be names of file to process. If no + /* argv[0..(argc-1)] should be names of file to process. If no * files were specified or '-' was specified, take input from stdin. * Otherwise, we process all the files specified. */ - if (argv[optind] == NULL) { - if (bbg.in_place) + if (argv[0] == NULL) { + if (opt & OPT_in_place) bb_error_msg_and_die(bb_msg_requires_arg, "-i"); add_input_file(stdin); process_files(); @@ -1223,11 +1231,13 @@ int sed_main(int argc, char **argv) int i; FILE *file; - for (i = optind; i < argc; i++) { + for (i = 0; i < argc; i++) { struct stat statbuf; int nonstdoutfd; - if (!strcmp(argv[i], "-") && !bbg.in_place) { + if (argv[i][0] == '-' && !argv[i][1] + && !(opt & OPT_in_place) + ) { add_input_file(stdin); process_files(); continue; @@ -1237,7 +1247,7 @@ int sed_main(int argc, char **argv) status = EXIT_FAILURE; continue; } - if (!bbg.in_place) { + if (!(opt & OPT_in_place)) { add_input_file(file); continue; } @@ -1246,12 +1256,12 @@ int sed_main(int argc, char **argv) nonstdoutfd = mkstemp(bbg.outname); if (-1 == nonstdoutfd) bb_error_msg_and_die("no temp file"); - bbg.nonstdout = fdopen(nonstdoutfd,"w"); + bbg.nonstdout = fdopen(nonstdoutfd, "w"); /* Set permissions of output file */ - fstat(fileno(file),&statbuf); - fchmod(nonstdoutfd,statbuf.st_mode); + fstat(fileno(file), &statbuf); + fchmod(nonstdoutfd, statbuf.st_mode); add_input_file(file); process_files(); fclose(bbg.nonstdout); @@ -1259,7 +1269,7 @@ int sed_main(int argc, char **argv) bbg.nonstdout = stdout; /* unlink(argv[i]); */ // FIXME: error check / message? - rename(bbg.outname,argv[i]); + rename(bbg.outname, argv[i]); free(bbg.outname); bbg.outname = 0; } -- cgit v1.1