Fixed crash issue with List & improved simple lists.

Signed-off-by: Grant Skinner <[email protected]>

Author: gskinner

js/documentation.js

@@ -69,7 +69,7 @@ var library = {
 				{
 				label:"List",
 				desc:"The <b>List</b> tool lists all found matches."+
-					"<p>You can specify either a simple delimiter (ex. <code>,</code> or <code>\\n</code>), or use <a href='javascript:showLib(\"subst\")'>substitution tokens</a> (ex. <code>$&</code> or <code>$1</code>) to generate more advanced reports.</p>"+
+					"<p>You can specify either a simple delimiter (ex. <code>,</code> or <code>\\n</code>), or use <a href='javascript:showLib(\"subst\")'>substitution tokens</a> to generate more advanced reports. For example, <code>$1\\n</code> would list all group 1 results.</p>"+
 					"<p>Escaped characters compatible with the JS string format are supported, such as <code>\\n</code>, <code>\\t</code> & <code>\\u0009</code>.</p>"+
 					"<p>Roll over tokens for information.</p>"
 				},

js/views/DocView.js

@@ -608,16 +608,21 @@ p.updateTool = function (source, regex) {
 			if (this.tool == "replace") {
 				result = source.replace(regex || this.getRegEx(), str);
 			} else {
-				var repl, ref, regex = this.getRegEx(false);
-				if (str.search(/\$[&1-9`']/) === -1) { str = "$&"+str; }
+				var repl, ref, regex = this.getRegEx(false), lastIndex = -1, trimR = 0;
+				if (str.search(/\$[&1-9`']/) === -1) {
+					trimR = str.length;
+					str = "$&"+str;
+				}
 				while (true) {
 					ref = source.replace(regex, "\b");
 					var index = ref.indexOf("\b");
-					if (index === -1) { break; }
+					if (index === -1 || index === lastIndex) { break; }
 					repl = source.replace(regex, str);
 					result += repl.substr(index, repl.length-ref.length+1);
 					source = ref.substr(index+1);
+					lastIndex = index;
 				}
+				if (trimR) { result = result.substr(0,result.length-trimR); }
 			}
 		}
 		this.toolsOutCM.setValue(result);