← Commit list

6a0abcc02

Denys Vlasenko
Denys Vlasenko
2011-05-03
FAILED Duration: 284s

Subject

sed: support \r in s command

Binary

No binary (build failed)

Architecture & Compiler Flags

Architecture: x86_64-linux-gnu Compiler: gcc (Ubuntu 11.4.0-1ubuntu1~22.04.3) 11.4.0

—

Build Log

(log yok)

Commit Diff GitHub →

editors/sed.c+2 -1
@@ -218,8 +218,9 @@ static char *copy_parsing_escapes(const char *string, int len)
char *dest = xmalloc(len + 1);
parse_escapes(dest, string, len, 'n', '\n');
- /* GNU sed also recognizes \t */
+ /* GNU sed also recognizes \t and \r */
parse_escapes(dest, dest, strlen(dest), 't', '\t');
+ parse_escapes(dest, dest, strlen(dest), 'r', '\r');
return dest;
}
testsuite/sed.tests+3 -0
@@ -287,6 +287,9 @@ 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" ""
+testing "sed understands \r" \
+ "sed 's/r/\r/'" \
+ "\rrr\n" "" "rrr\n"
# testing "description" "arguments" "result" "infile" "stdin"