re2 1.18.3 → 1.19.1

package/README.md

@@ -58,6 +58,7 @@ Supported properties:
 * [`re2.unicode`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/unicode)
   * `RE2` engine always works in the Unicode mode. See details below.
 * [`re2.sticky`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/sticky)
+* *Since 1.19.0*: [`re2.hasIndices`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/hasIndices)
 * [`re2.source`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/source)
 * [`re2.flags`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/flags)
 
@@ -352,6 +353,8 @@ console.log('re2_res : ' + re2_res); // prints: re2_res : abc,a,b,c
 
 ## Release history
 
+- 1.19.1 *Bugfix: indices for the `d` flag when `lastIndex` is non zero. Bugfix: the match result. Thx, [teebu](https://github.com/teebu).*
+- 1.19.0 *Added `hasIndices` AKA the `d` flag. Thx, [teebu](https://github.com/teebu).*
 - 1.18.3 *Fixed bug with non-matched groups. Thx, [Dan Setterquist](https://github.com/dset).*
 - 1.18.2 *Reference to the binary module by its full name.*
 - 1.18.1 *Support for Node 16, 18, 20 + Darwin arm64 precompiled binaries.*

package/lib/exec.cc

@@ -90,8 +90,8 @@ NAN_METHOD(WrappedRE2::Exec)
 				if (re2->hasIndices) {
 					auto pair = Nan::New<v8::Array>();
 					auto offset = data - str.data;
-					Nan::Set(pair, 0, Nan::New<v8::Integer>(static_cast<int>(offset)));
-					Nan::Set(pair, 1, Nan::New<v8::Integer>(static_cast<int>(offset + item.size())));
+					Nan::Set(pair, 0, Nan::New<v8::Integer>(indexOffset + static_cast<int>(offset)));
+					Nan::Set(pair, 1, Nan::New<v8::Integer>(indexOffset + static_cast<int>(offset + item.size())));
 					Nan::Set(indices, i, pair);
 				}
 			}
@@ -118,8 +118,8 @@ NAN_METHOD(WrappedRE2::Exec)
 					auto pair = Nan::New<v8::Array>();
 					auto offset = getUtf16Length(str.data + lastIndex, data);
 					auto length = getUtf16Length(data, data + item.size());
-					Nan::Set(pair, 0, Nan::New<v8::Integer>(static_cast<int>(offset)));
-					Nan::Set(pair, 1, Nan::New<v8::Integer>(static_cast<int>(offset + length)));
+					Nan::Set(pair, 0, Nan::New<v8::Integer>(indexOffset + static_cast<int>(offset)));
+					Nan::Set(pair, 1, Nan::New<v8::Integer>(indexOffset + static_cast<int>(offset + length)));
 					Nan::Set(indices, i, pair);
 				}
 			}
@@ -159,7 +159,19 @@ NAN_METHOD(WrappedRE2::Exec)
 		Nan::Set(result, Nan::New("groups").ToLocalChecked(), groups);
 
 		if (re2->hasIndices) {
-			Nan::Set(indices, Nan::New("groups").ToLocalChecked(), groups);
+			auto indexGroups = Nan::New<v8::Object>();
+			Nan::SetPrototype(indexGroups, Nan::Null());
+
+			for (auto group : groupNames)
+			{
+				auto value = Nan::Get(indices, group.first);
+				if (!value.IsEmpty())
+				{
+					Nan::Set(indexGroups, Nan::New(group.second).ToLocalChecked(), value.ToLocalChecked());
+				}
+			}
+
+			Nan::Set(indices, Nan::New("groups").ToLocalChecked(), indexGroups);
 		}
 	}
 	else

package/lib/match.cc

@@ -82,7 +82,7 @@ NAN_METHOD(WrappedRE2::Match)
 
 	// form a result
 
-	auto result = Nan::New<v8::Array>();
+	auto result = Nan::New<v8::Array>(), indices = Nan::New<v8::Array>();
 
 	if (a.isBuffer)
 	{
@@ -93,6 +93,23 @@ NAN_METHOD(WrappedRE2::Match)
 			if (data)
 			{
 				Nan::Set(result, i, Nan::CopyBuffer(data, item.size()).ToLocalChecked());
+				if (!re2->global && re2->hasIndices)
+				{
+					auto pair = Nan::New<v8::Array>();
+					auto offset = getUtf16Length(a.data + lastIndex, data);
+					auto length = getUtf16Length(data, data + item.size());
+					Nan::Set(pair, 0, Nan::New<v8::Integer>(static_cast<int>(offset)));
+					Nan::Set(pair, 1, Nan::New<v8::Integer>(static_cast<int>(offset + length)));
+					Nan::Set(indices, i, pair);
+				}
+			}
+			else
+			{
+				Nan::Set(result, i, Nan::Undefined());
+				if (!re2->global && re2->hasIndices)
+				{
+					Nan::Set(indices, i, Nan::Undefined());
+				}
 			}
 		}
 		if (!re2->global)
@@ -110,6 +127,23 @@ NAN_METHOD(WrappedRE2::Match)
 			if (data)
 			{
 				Nan::Set(result, i, Nan::New(data, item.size()).ToLocalChecked());
+				if (!re2->global && re2->hasIndices)
+				{
+					auto pair = Nan::New<v8::Array>();
+					auto offset = getUtf16Length(a.data + lastIndex, data);
+					auto length = getUtf16Length(data, data + item.size());
+					Nan::Set(pair, 0, Nan::New<v8::Integer>(static_cast<int>(offset)));
+					Nan::Set(pair, 1, Nan::New<v8::Integer>(static_cast<int>(offset + length)));
+					Nan::Set(indices, i, pair);
+				}
+			}
+			else
+			{
+				Nan::Set(result, i, Nan::Undefined());
+				if (!re2->global && re2->hasIndices)
+				{
+					Nan::Set(indices, i, Nan::Undefined());
+				}
 			}
 		}
 		if (!re2->global)
@@ -146,10 +180,35 @@ NAN_METHOD(WrappedRE2::Match)
 			}
 
 			Nan::Set(result, Nan::New("groups").ToLocalChecked(), groups);
+
+			if (re2->hasIndices)
+			{
+				auto indexGroups = Nan::New<v8::Object>();
+				Nan::SetPrototype(indexGroups, Nan::Null());
+
+				for (auto group : groupNames)
+				{
+					auto value = Nan::Get(indices, group.first);
+					if (!value.IsEmpty())
+					{
+						Nan::Set(indexGroups, Nan::New(group.second).ToLocalChecked(), value.ToLocalChecked());
+					}
+				}
+
+				Nan::Set(indices, Nan::New("groups").ToLocalChecked(), indexGroups);
+			}
 		}
 		else
 		{
 			Nan::Set(result, Nan::New("groups").ToLocalChecked(), Nan::Undefined());
+			if (re2->hasIndices)
+			{
+				Nan::Set(indices, Nan::New("groups").ToLocalChecked(), Nan::Undefined());
+			}
+		}
+		if (re2->hasIndices)
+		{
+			Nan::Set(result, Nan::New("indices").ToLocalChecked(), indices);
 		}
 	}
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "re2",
-  "version": "1.18.3",
+  "version": "1.19.1",
   "description": "Bindings for RE2: fast, safe alternative to backtracking regular expression engines.",
   "homepage": "https://github.com/uhop/node-re2",
   "bugs": "https://github.com/uhop/node-re2/issues",
@@ -13,12 +13,12 @@
   "dependencies": {
     "install-artifact-from-github": "^1.3.3",
     "nan": "^2.17.0",
-    "node-gyp": "^9.3.1"
+    "node-gyp": "^9.4.0"
   },
   "devDependencies": {
-    "@types/node": "^20.2.3",
+    "@types/node": "^20.3.1",
     "heya-unit": "^0.3.0",
-    "typescript": "^5.0.4"
+    "typescript": "^5.1.3"
   },
   "scripts": {
     "test": "node tests/tests.js",

package/tests/test_exec.js

@@ -21,9 +21,13 @@ unit.add(module, [
 
     var result = re.exec('The Quick Brown Fox Jumps Over The Lazy Dog');
 
-    eval(t.TEST("t.unify(result, ['Quick Brown Fox Jumps', 'Brown', 'Jumps'])"));
+    eval(
+      t.TEST("t.unify(result, ['Quick Brown Fox Jumps', 'Brown', 'Jumps'])")
+    );
     eval(t.TEST('result.index === 4'));
-    eval(t.TEST("result.input === 'The Quick Brown Fox Jumps Over The Lazy Dog'"));
+    eval(
+      t.TEST("result.input === 'The Quick Brown Fox Jumps Over The Lazy Dog'")
+    );
     eval(t.TEST('re.lastIndex === 25'));
   },
   function test_execSucc(t) {
@@ -148,12 +152,20 @@ unit.add(module, [
 
     var result = re.exec('Каждый Охотник Желает Знать Где Сидит Фазан');
 
-    eval(t.TEST("t.unify(result, ['Охотник Желает Знать Где', 'Желает', 'Где'])"));
+    eval(
+      t.TEST("t.unify(result, ['Охотник Желает Знать Где', 'Желает', 'Где'])")
+    );
     eval(t.TEST('result.index === 7'));
-    eval(t.TEST("result.input === 'Каждый Охотник Желает Знать Где Сидит Фазан'"));
+    eval(
+      t.TEST("result.input === 'Каждый Охотник Желает Знать Где Сидит Фазан'")
+    );
     eval(t.TEST('re.lastIndex === 31'));
 
-    eval(t.TEST("result.input.substr(result.index) === 'Охотник Желает Знать Где Сидит Фазан'"));
+    eval(
+      t.TEST(
+        "result.input.substr(result.index) === 'Охотник Желает Знать Где Сидит Фазан'"
+      )
+    );
     eval(t.TEST("result.input.substr(re.lastIndex) === ' Сидит Фазан'"));
   },
   function test_execUnicodeSubsequent(t) {
@@ -248,11 +260,21 @@ unit.add(module, [
 
     eval(t.TEST('result.index === 13'));
     eval(t.TEST('result.input instanceof Buffer'));
-    eval(t.TEST("result.input.toString() === 'Каждый Охотник Желает Знать Где Сидит Фазан'"));
+    eval(
+      t.TEST(
+        "result.input.toString() === 'Каждый Охотник Желает Знать Где Сидит Фазан'"
+      )
+    );
     eval(t.TEST('re.lastIndex === 58'));
 
-    eval(t.TEST("result.input.toString('utf8', result.index) === 'Охотник Желает Знать Где Сидит Фазан'"));
-    eval(t.TEST("result.input.toString('utf8', re.lastIndex) === ' Сидит Фазан'"));
+    eval(
+      t.TEST(
+        "result.input.toString('utf8', result.index) === 'Охотник Желает Знать Где Сидит Фазан'"
+      )
+    );
+    eval(
+      t.TEST("result.input.toString('utf8', re.lastIndex) === ' Сидит Фазан'")
+    );
   },
 
   // Sticky tests
@@ -317,7 +339,9 @@ xy2 (at start of line)
     eval(t.TEST("result[0] === 'xy'"));
     eval(t.TEST('result.index > 3'));
     eval(t.TEST('result.index < pattern.length - 4'));
-    eval(t.TEST('result[0] === pattern.substr(result.index, result[0].length)'));
+    eval(
+      t.TEST('result[0] === pattern.substr(result.index, result[0].length)')
+    );
   },
 
   // dotAll tests
@@ -332,5 +356,85 @@ xy2 (at start of line)
     eval(t.TEST("new RE2('a.c', 's').test('abc')"));
     eval(t.TEST("new RE2(/a.c/s).test('a c')"));
     eval(t.TEST("new RE2(/a.c/s).test('a\\nc')"));
+  },
+
+  // hasIndices tests
+
+  function test_execHasIndices(t) {
+    'use strict';
+
+    eval(t.TEST("!new RE2('1').hasIndices"));
+    eval(t.TEST('!new RE2(/1/).hasIndices'));
+
+    var re = new RE2('(aa)(?<b>b)?(?<c>ccc)', 'd');
+
+    eval(t.TEST('re.hasIndices'));
+
+    var result = re.exec('1aabccc2');
+
+    eval(t.TEST('result.length === 4'));
+    eval(t.TEST("result.input === '1aabccc2'"));
+    eval(t.TEST('result.index === 1'));
+    eval(t.TEST('Object.keys(result.groups).length === 2'));
+    eval(t.TEST("result.groups.b === 'b'"));
+    eval(t.TEST("result.groups.c === 'ccc'"));
+    eval(t.TEST("result[0] === 'aabccc'"));
+    eval(t.TEST("result[1] === 'aa'"));
+    eval(t.TEST("result[2] === 'b'"));
+    eval(t.TEST("result[3] === 'ccc'"));
+    eval(t.TEST('result.indices.length === 4'));
+    eval(t.TEST('t.unify(result.indices, [[1, 7], [1, 3], [3, 4], [4, 7]])'));
+    eval(t.TEST('Object.keys(result.indices.groups).length === 2'));
+    eval(t.TEST('t.unify(result.indices.groups.b, [3, 4])'));
+    eval(t.TEST('t.unify(result.indices.groups.c, [4, 7])'));
+
+    result = re.exec('1aaccc2');
+
+    eval(t.TEST('result.length === 4'));
+    eval(t.TEST("result.input === '1aaccc2'"));
+    eval(t.TEST('result.index === 1'));
+    eval(t.TEST('Object.keys(result.groups).length === 2'));
+    eval(t.TEST('result.groups.b === undefined'));
+    eval(t.TEST("result.groups.c === 'ccc'"));
+    eval(t.TEST("result[0] === 'aaccc'"));
+    eval(t.TEST("result[1] === 'aa'"));
+    eval(t.TEST('result[2] === undefined'));
+    eval(t.TEST("result[3] === 'ccc'"));
+    eval(t.TEST('result.indices.length === 4'));
+    eval(
+      t.TEST('t.unify(result.indices, [[1, 6], [1, 3], undefined, [3, 6]])')
+    );
+    eval(t.TEST('Object.keys(result.indices.groups).length === 2'));
+    eval(t.TEST('t.unify(result.indices.groups.b, undefined)'));
+    eval(t.TEST('t.unify(result.indices.groups.c, [3, 6])'));
+
+    try {
+      re = new RE2(new RegExp('1', 'd'));
+      eval(t.TEST('re.hasIndices'));
+    } catch (e) {
+      // squelch
+    }
+  },
+
+  function test_hasIndexLastIndex(t) {
+    'use strict';
+
+    const re2 = new RE2('a', 'dg');
+
+    eval(t.TEST('re2.lastIndex === 0'));
+
+    let result = re2.exec('abca');
+    eval(t.TEST('re2.lastIndex === 1'));
+    eval(t.TEST('result.index === 0'));
+    eval(t.TEST('t.unify(result.indices, [[0, 1]])'));
+
+    result = re2.exec('abca');
+    eval(t.TEST('re2.lastIndex === 4'));
+    eval(t.TEST('result.index === 3'));
+    eval(t.TEST('t.unify(result.indices, [[3, 4]])'));
+
+    result = re2.exec('abca');
+    eval(t.TEST('re2.lastIndex === 0'));
+    eval(t.TEST('result === null'));
   }
 ]);

package/tests/test_match.js

@@ -1,137 +1,172 @@
-"use strict";
-
-
-var unit = require("heya-unit");
-var RE2  = require("../re2");
+'use strict';
 
+var unit = require('heya-unit');
+var RE2 = require('../re2');
 
 // tests
 
 unit.add(module, [
+  // These tests are copied from MDN:
+  // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/match
+
+  function test_match(t) {
+    'use strict';
+
+    var str = 'For more information, see Chapter 3.4.5.1';
+
+    var re = new RE2(/(chapter \d+(\.\d)*)/i);
+    var result = re.match(str);
+
+    eval(t.TEST('result.input === str'));
+    eval(t.TEST('result.index === 26'));
+    eval(t.TEST('result.length === 3'));
+    eval(t.TEST("result[0] === 'Chapter 3.4.5.1'"));
+    eval(t.TEST("result[1] === 'Chapter 3.4.5.1'"));
+    eval(t.TEST("result[2] === '.1'"));
+  },
+  function test_matchGlobal(t) {
+    'use strict';
+
+    var re = new RE2(/[A-E]/gi);
+    var result = re.match(
+      'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz'
+    );
+
+    eval(
+      t.TEST(
+        "t.unify(result, ['A', 'B', 'C', 'D', 'E', 'a', 'b', 'c', 'd', 'e'])"
+      )
+    );
+  },
+  function test_matchFail(t) {
+    'use strict';
 
-	// These tests are copied from MDN:
-	// https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/match
+    var re = new RE2('(a+)?(b+)?');
+    var result = re.match('aaabb');
 
-	function test_match(t) {
-		"use strict";
+    eval(t.TEST("result[1] === 'aaa'"));
+    eval(t.TEST("result[2] === 'bb'"));
 
-		var str = "For more information, see Chapter 3.4.5.1";
+    result = re.match('aaacbb');
 
-		var re = new RE2(/(chapter \d+(\.\d)*)/i);
-		var result = re.match(str);
+    eval(t.TEST("result[1] === 'aaa'"));
+    eval(t.TEST('result[2] === undefined'));
+  },
+  function test_matchInvalid(t) {
+    'use strict';
 
-		eval(t.TEST("result.input === str"));
-		eval(t.TEST("result.index === 26"));
-		eval(t.TEST("result.length === 3"));
-		eval(t.TEST("result[0] === 'Chapter 3.4.5.1'"));
-		eval(t.TEST("result[1] === 'Chapter 3.4.5.1'"));
-		eval(t.TEST("result[2] === '.1'"));
-	},
-	function test_matchGlobal(t) {
-		"use strict";
+    var re = RE2('');
 
-		var re = new RE2(/[A-E]/gi);
-		var result = re.match("ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz");
+    try {
+      re.match({
+        toString() {
+          throw 'corner';
+        }
+      });
+      t.test(false); // shouldn't be here
+    } catch (e) {
+      eval(t.TEST("e === 'corner'"));
+    }
+  },
 
-		eval(t.TEST("t.unify(result, ['A', 'B', 'C', 'D', 'E', 'a', 'b', 'c', 'd', 'e'])"));
-	},
-	function test_matchFail(t) {
-		"use strict";
+  // Unicode tests
 
-		var re = new RE2("(a+)?(b+)?");
-		var result = re.match("aaabb");
+  function test_matchUnicode(t) {
+    'use strict';
 
-		eval(t.TEST("result[1] === 'aaa'"));
-		eval(t.TEST("result[2] === 'bb'"));
+    var str = 'Это ГЛАВА 3.4.5.1';
 
-		result = re.match("aaacbb");
+    var re = new RE2(/(глава \d+(\.\d)*)/i);
+    var result = re.match(str);
 
-		eval(t.TEST("result[1] === 'aaa'"));
-		eval(t.TEST("result[2] === undefined"));
-	},
-	function test_matchInvalid(t) {
-		"use strict";
+    eval(t.TEST('result.input === str'));
+    eval(t.TEST('result.index === 4'));
+    eval(t.TEST('result.length === 3'));
+    eval(t.TEST("result[0] === 'ГЛАВА 3.4.5.1'"));
+    eval(t.TEST("result[1] === 'ГЛАВА 3.4.5.1'"));
+    eval(t.TEST("result[2] === '.1'"));
+  },
 
-		var re = RE2('');
+  // Buffer tests
 
-		try {
-			re.match({ toString() { throw "corner"; } });
-			t.test(false); // shouldn't be here
-		} catch(e) {
-			eval(t.TEST("e === 'corner'"));
-		}
-	},
+  function test_matchBuffer(t) {
+    'use strict';
 
-	// Unicode tests
+    var buf = new Buffer('Это ГЛАВА 3.4.5.1');
 
-	function test_matchUnicode(t) {
-		"use strict";
+    var re = new RE2(/(глава \d+(\.\d)*)/i);
+    var result = re.match(buf);
 
-		var str = "Это ГЛАВА 3.4.5.1";
+    eval(t.TEST('result.input instanceof Buffer'));
+    eval(t.TEST('result.length === 3'));
+    eval(t.TEST('result[0] instanceof Buffer'));
+    eval(t.TEST('result[1] instanceof Buffer'));
+    eval(t.TEST('result[2] instanceof Buffer'));
 
-		var re = new RE2(/(глава \d+(\.\d)*)/i);
-		var result = re.match(str);
+    eval(t.TEST('result.input === buf'));
+    eval(t.TEST('result.index === 7'));
+    eval(
+      t.TEST("result.input.toString('utf8', result.index) === 'ГЛАВА 3.4.5.1'")
+    );
+    eval(t.TEST("result[0].toString() === 'ГЛАВА 3.4.5.1'"));
+    eval(t.TEST("result[1].toString() === 'ГЛАВА 3.4.5.1'"));
+    eval(t.TEST("result[2].toString() === '.1'"));
+  },
 
-		eval(t.TEST("result.input === str"));
-		eval(t.TEST("result.index === 4"));
-		eval(t.TEST("result.length === 3"));
-		eval(t.TEST("result[0] === 'ГЛАВА 3.4.5.1'"));
-		eval(t.TEST("result[1] === 'ГЛАВА 3.4.5.1'"));
-		eval(t.TEST("result[2] === '.1'"));
-	},
+  // Sticky tests
 
-	// Buffer tests
+  function test_matchSticky(t) {
+    'use strict';
 
-	function test_matchBuffer(t) {
-		"use strict";
+    var re = new RE2('\\s+', 'y');
 
-		var buf = new Buffer("Это ГЛАВА 3.4.5.1");
+    eval(t.TEST("re.match('Hello world, how are you?') === null"));
 
-		var re = new RE2(/(глава \d+(\.\d)*)/i);
-		var result = re.match(buf);
+    re.lastIndex = 5;
 
-		eval(t.TEST("result.input instanceof Buffer"));
-		eval(t.TEST("result.length === 3"));
-		eval(t.TEST("result[0] instanceof Buffer"));
-		eval(t.TEST("result[1] instanceof Buffer"));
-		eval(t.TEST("result[2] instanceof Buffer"));
+    var result = re.match('Hello world, how are you?');
 
-		eval(t.TEST("result.input === buf"));
-		eval(t.TEST("result.index === 7"));
-		eval(t.TEST("result.input.toString('utf8', result.index) === 'ГЛАВА 3.4.5.1'"));
-		eval(t.TEST("result[0].toString() === 'ГЛАВА 3.4.5.1'"));
-		eval(t.TEST("result[1].toString() === 'ГЛАВА 3.4.5.1'"));
-		eval(t.TEST("result[2].toString() === '.1'"));
-	},
+    eval(t.TEST("t.unify(result, [' '])"));
+    eval(t.TEST('result.index === 5'));
+    eval(t.TEST('re.lastIndex === 6'));
 
-	// Sticky tests
+    var re2 = new RE2('\\s+', 'gy');
 
-	function test_matchSticky(t) {
-		"use strict";
+    eval(t.TEST("re2.match('Hello world, how are you?') === null"));
 
-		var re = new RE2("\\s+", "y");
+    re2.lastIndex = 5;
 
-		eval(t.TEST("re.match('Hello world, how are you?') === null"));
+    eval(t.TEST("re2.match('Hello world, how are you?') === null"));
 
-		re.lastIndex = 5;
+    var re3 = new RE2(/[A-E]/giy);
+    var result3 = re3.match(
+      'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz'
+    );
 
-		var result = re.match("Hello world, how are you?");
+    eval(t.TEST("t.unify(result3, ['A', 'B', 'C', 'D', 'E'])"));
+  },
 
-		eval(t.TEST("t.unify(result, [' '])"));
-		eval(t.TEST("result.index === 5"));
-		eval(t.TEST("re.lastIndex === 6"));
+  // hasIndices tests
 
-		var re2 = new RE2("\\s+", "gy");
+  function test_matchHasIndices(t) {
+    'use strict';
 
-		eval(t.TEST("re2.match('Hello world, how are you?') === null"));
+    const re = new RE2('(aa)(?<b>b)?(?<c>ccc)', 'd'),
+      str1 = '1aabccc2',
+      str2 = '1aaccc2';
 
-		re2.lastIndex = 5;
+    eval(t.TEST("t.unify(str1.match(re), re.exec(str1))"));
+    eval(t.TEST("t.unify(str2.match(re), re.exec(str2))"));
+  },
 
-		eval(t.TEST("re2.match('Hello world, how are you?') === null"));
+  function test_matchHasIndicesGlobal(t) {
+    'use strict';
 
-		var re3 = new RE2(/[A-E]/giy);
-		var result3 = re3.match("ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz");
+    const re = new RE2('(?<zzz>a)', 'dg'),
+      result = 'abca'.match(re);
 
-		eval(t.TEST("t.unify(result3, ['A', 'B', 'C', 'D', 'E'])"));
-	}
+    eval(t.TEST("t.unify(result, ['a', 'a'])"));
+    eval(t.TEST("!('indices' in result)"));
+    eval(t.TEST("!('groups' in result)"));
+  }
 ]);

package/tests/test_replace.js

@@ -13,7 +13,10 @@ unit.add(module, [
     'use strict';
 
     var re = new RE2(/apples/gi);
-    var result = re.replace('Apples are round, and apples are juicy.', 'oranges');
+    var result = re.replace(
+      'Apples are round, and apples are juicy.',
+      'oranges'
+    );
     eval(t.TEST("result === 'oranges are round, and oranges are juicy.'"));
 
     re = new RE2(/xmas/i);
@@ -199,15 +202,24 @@ unit.add(module, [
     'use strict';
 
     var re = new RE2(/яблоки/gi);
-    var result = re.replace(new Buffer('Яблоки красны, яблоки сочны.'), 'апельсины');
+    var result = re.replace(
+      new Buffer('Яблоки красны, яблоки сочны.'),
+      'апельсины'
+    );
     eval(t.TEST('result instanceof Buffer'));
     eval(t.TEST("result.toString() === 'апельсины красны, апельсины сочны.'"));
 
-    result = re.replace(new Buffer('Яблоки красны, яблоки сочны.'), new Buffer('апельсины'));
+    result = re.replace(
+      new Buffer('Яблоки красны, яблоки сочны.'),
+      new Buffer('апельсины')
+    );
     eval(t.TEST('result instanceof Buffer'));
     eval(t.TEST("result.toString() === 'апельсины красны, апельсины сочны.'"));
 
-    result = re.replace('Яблоки красны, яблоки сочны.', new Buffer('апельсины'));
+    result = re.replace(
+      'Яблоки красны, яблоки сочны.',
+      new Buffer('апельсины')
+    );
     eval(t.TEST("typeof result == 'string'"));
     eval(t.TEST("result === 'апельсины красны, апельсины сочны.'"));
   },
@@ -284,7 +296,7 @@ unit.add(module, [
 
   // Non-matches
 
-  function test_ReplaceOneNonMatch(t) {
+  function test_replaceOneNonMatch(t) {
     'use strict';
 
     function replacer(match, capture, offset, string) {
@@ -300,10 +312,10 @@ unit.add(module, [
     var re = new RE2(/hello (world)?/);
     re.replace('hello ', replacer);
   },
-  function test_ReplaceTwoNonMatches(t) {
+  function test_replaceTwoNonMatches(t) {
     'use strict';
 
-    function replacer(match, capture1, capture2, offset, string) {
+    function replacer(match, capture1, capture2, offset, string, groups) {
       t.test(typeof offset == 'number');
       t.test(typeof match == 'string');
       t.test(typeof string == 'string');
@@ -312,10 +324,14 @@ unit.add(module, [
       t.test(offset === 1);
       t.test(match === 'b & y');
       t.test(string === 'ab & yz');
+      t.test(typeof groups == 'object');
+      t.test(Object.keys(groups).length == 2);
+      t.test(groups.a === undefined);
+      t.test(groups.b == undefined);
       return '';
     }
 
-    var re = new RE2(/b(1)? & (2)?y/);
+    var re = new RE2(/b(?<a>1)? & (?<b>2)?y/);
     var result = re.replace('ab & yz', replacer);
     eval(t.TEST("result === 'az'"));
   }