From a3d4bf38b0245cdf3551d59ca1a73bceb01abc0f Mon Sep 17 00:00:00 2001 From: Bernhard Reutner-Fischer Date: Sat, 3 Jun 2006 21:40:11 +0000 Subject: [PATCH] - use bb functions instead of handrolling stuff. - add XXX question about supposedly superfluous else clause text data bss dec hex filename 5918 0 0 5918 171e editors/sed.o.orig 5901 0 0 5901 170d editors/sed.o --- editors/sed.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/editors/sed.c b/editors/sed.c index 6585a60bd..1b780d478 100644 --- a/editors/sed.c +++ b/editors/sed.c @@ -481,7 +481,7 @@ static void add_cmd(char *cmdstr) cmdstr = bb_xasprintf("%s\n%s", bbg.add_cmd_line, cmdstr); free(bbg.add_cmd_line); bbg.add_cmd_line = cmdstr; - } else bbg.add_cmd_line=NULL; + } else bbg.add_cmd_line=NULL; /* XXX: erm.. bbg.add_cmd_line was 0 and we set it to *0 here why? */ /* If this line ends with backslash, request next line. */ temp=strlen(cmdstr); @@ -737,8 +737,8 @@ static int puts_maybe_newline(char *s, FILE *file, int missing_newline, int no_n if(!no_newline) fputc('\n',file); if(ferror(file)) { - fprintf(stderr,"Write failed.\n"); - exit(4); /* It's what gnu sed exits with... */ + bb_default_error_retval = 4; /* It's what gnu sed exits with... */ + bb_error_msg_and_die(bb_msg_write_error); } return no_newline; -- 2.25.1