Make sed a\ line continuations work properly for different pattern input modes.
authorRob Landley <rob@landley.net>
Sun, 14 Dec 2014 19:51:28 +0000 (13:51 -0600)
committerRob Landley <rob@landley.net>
Sun, 14 Dec 2014 19:51:28 +0000 (13:51 -0600)
tests/sed.test
toys/pending/sed.c

index 8673885..789a1a5 100644 (file)
@@ -119,6 +119,17 @@ testing "sed D further processing depends on whether line is blank" \
        "sed -e '/one/,/three/{' -e 'i meep' -e'N;2D;}'" \
        "meep\nmeep\ntwo\nthree\n" "" "one\ntwo\nthree\n"
 
+# Different ways of parsing line continuations
+
+testing "" "sed -e '1a\' -e 'huh'" "meep\nhuh\n" "" "meep"
+testing "" "sed -f input" "blah\nboom\n" '1a\\\nboom' 'blah'
+testing "" "sed '1a\
+hello'" "merp\nhello\n" "" "merp"
+#echo meep | sed/sed -e '1a\' -e 'huh'
+#echo blah | sed/sed -f <(echo -e "1a\\\\\nboom")
+#echo merp | sed/sed "1a\\
+#hello"
+
 # -i with $ last line test
 
 exit $FAILCOUNT
index 6c025f0..fd5352a 100644 (file)
@@ -913,10 +913,15 @@ append:
       reg = extend_string((void *)&corwin, line, reg - (char *)corwin, end); 
       line += end;
 
-      // Line continuation?
-      if (class && reg[-1] == '\\') {
-        reg[-1] = 0;
-        corwin->hit++;
+      // Line continuation? (Two slightly different input methods, -e with
+      // embedded newline vs -f line by line. Must parse both correctly.)
+      if (class && line[-1] == '\\') {
+        reg[-2] = 0;
+        if (*line && line[1]) {
+          reg -= 2;
+          line++;
+          goto append;
+        } else corwin->hit++;
       }
 
     // Commands that take no arguments