Apple-2gs-SW/hush
1
0
Fork 0
mirror of https://github.com/sheumann/hush.git synced 2024-12-26 10:32:02 +00:00
Code Issues Projects Releases Wiki Activity

sed: fix sed -i: unlike without -i, it does not forget ranges

Browse Source 
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2010-08-01 04:14:46 +02:00
parent 138ce54c9c
commit 2e284a40bc
2 changed files with 15 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
editors/sed.c
View File

@ -61,6 +61,10 @@
#include "libbb.h" #include "libbb.h"
#include "xregex.h" #include "xregex.h"
enum {
OPT_in_place = 1 << 0,
};
/* Each sed command turns into one of these structures. */ /* Each sed command turns into one of these structures. */
typedef struct sed_cmd_s { typedef struct sed_cmd_s {
/* Ordered by alignment requirements: currently 36 bytes on x86 */ /* Ordered by alignment requirements: currently 36 bytes on x86 */
@ -938,8 +942,11 @@ static void process_files(void)
if (matched) { if (matched) {
/* once matched, "n,xxx" range is dead, disabling it */ /* once matched, "n,xxx" range is dead, disabling it */
if (sed_cmd->beg_line > 0) if (sed_cmd->beg_line > 0
&& !(option_mask32 & OPT_in_place) /* but not for -i */
) {
sed_cmd->beg_line = -2; sed_cmd->beg_line = -2;
}
sed_cmd->in_match = !( sed_cmd->in_match = !(
/* has the ending line come, or is this a single address command? */ /* has the ending line come, or is this a single address command? */
(sed_cmd->end_line ? (sed_cmd->end_line ?
@ -1270,9 +1277,6 @@ static void add_cmd_block(char *cmdstr)
int sed_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; int sed_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
int sed_main(int argc UNUSED_PARAM, char **argv) int sed_main(int argc UNUSED_PARAM, char **argv)
{ {
enum {
OPT_in_place = 1 << 0,
};
unsigned opt; unsigned opt;
llist_t *opt_e, *opt_f; llist_t *opt_e, *opt_f;
int status = EXIT_SUCCESS; int status = EXIT_SUCCESS;
@ -1292,6 +1296,7 @@ int sed_main(int argc UNUSED_PARAM, char **argv)
opt_e = opt_f = NULL; opt_e = opt_f = NULL;
opt_complementary = "e::f::" /* can occur multiple times */ opt_complementary = "e::f::" /* can occur multiple times */
"nn"; /* count -n */ "nn"; /* count -n */
/* -i must be first, to match OPT_in_place definition */
opt = getopt32(argv, "irne:f:", &opt_e, &opt_f, opt = getopt32(argv, "irne:f:", &opt_e, &opt_f,
&G.be_quiet); /* counter for -n */ &G.be_quiet); /* counter for -n */
//argc -= optind; //argc -= optind;

7
testsuite/sed.tests
View File

@ -270,11 +270,16 @@ testing "sed a cmd ended by double backslash" \
| two \\ | two \\
' '
# fisrt three lines are deleted; 4th line is matched and printed by "2,3" and by "4" ranges # first three lines are deleted; 4th line is matched and printed by "2,3" and by "4" ranges
testing "sed with N skipping lines past ranges on next cmds" \ testing "sed with N skipping lines past ranges on next cmds" \
"sed -n '1{N;N;d};1p;2,3p;3p;4p'" \ "sed -n '1{N;N;d};1p;2,3p;3p;4p'" \
"4\n4\n" "" "1\n2\n3\n4\n" "4\n4\n" "" "1\n2\n3\n4\n"
testing "sed -i with address modifies all files, not only first" \
"cp input input2; sed -i -e '1s/foo/bar/' input input2 && cat input input2; rm input2" \
"bar\nbar\n" "foo\n" "foo\n"
# testing "description" "arguments" "result" "infile" "stdin" # testing "description" "arguments" "result" "infile" "stdin"
exit $FAILCOUNT exit $FAILCOUNT