Fix #600. Add regression test

jqlang · Nov 13, 2014 · 30e0082 · 30e0082
1 parent 89791a0
commit 30e0082
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/builtin.c b/builtin.c
@@ -1057,9 +1057,7 @@ static const char* const jq_builtins[] = {
        //  # create the \"capture\" object:
   "    | reduce ( $r | .captures | .[] | select(.name != null) | { (.name) : .string } ) as $pair"
   "        ({}; . + $pair)"
-  "    | if . == {} then $in | .[0:$r.offset]+s+.[$r.offset+$r.length:]"
-  "      else (. | s)"
-  "      end"
+  "    | $in[0:$r.offset] + s + $in[$r.offset+$r.length:]"
   "    end ;",
   //
   // repeated substitution of re (which may contain named captures)

diff --git a/tests/all.test b/tests/all.test
@@ -859,6 +859,10 @@ capture("(?<a>[a-z]+)-(?<n>[0-9]+)")
 ["a,b, c, d, e,f", ", a,b, c, d, e,f, "]
 ["a,b:c, d, e,f",":a,b, c, d, e,f, "]
 
+sub("^(?<head>.)"; "Head=\(.head) Tail=")
+"abcdef"
+"Head=a Tail=bcdef"
+
 [.[] | gsub(", "; ":")]
 ["a,b, c, d, e,f",", a,b, c, d, e,f, "]
 ["a,b:c:d:e,f",":a,b:c:d:e,f:"]