bugfix: pattern matching an empty substring in ngx.re.gmatch did not match at the end of the subject string.

agentzh · agentzh · commit 4b9100c1dd9c · 2012-09-28T00:07:47.000-07:00
diff --git a/src/ngx_http_lua_regex.c b/src/ngx_http_lua_regex.c
@@ -948,7 +948,7 @@ ngx_http_lua_ngx_re_gmatch_iterator(lua_State *L)
         offset++;
     }
 
-    if (offset >= (ssize_t) subj.len) {
+    if (offset > (ssize_t) subj.len) {
         offset = -1;
 
         if (!(ctx->flags & NGX_LUA_RE_COMPILE_ONCE)) {
diff --git a/t/035-gmatch.t b/t/035-gmatch.t
@@ -509,4 +509,5 @@ matched: []
 matched: []
 matched: []
 matched: []
+matched: []
 

Original file line number	Diff line number	Diff line change
`@@ -948,7 +948,7 @@ ngx_http_lua_ngx_re_gmatch_iterator(lua_State *L)`
`948`	`948`	`offset++;`
`949`	`949`	`}`
`950`	`950`
`951`		`- if (offset >= (ssize_t) subj.len) {`
	`951`	`+ if (offset > (ssize_t) subj.len) {`
`952`	`952`	`offset = -1;`
`953`	`953`
`954`	`954`	`if (!(ctx->flags & NGX_LUA_RE_COMPILE_ONCE)) {`