set trailing slash optional in route matching

Author: touletan

api/tests/test-router.js

@@ -225,6 +225,30 @@ o.spec("route", function() {
 					)
 				})
 
+				o("remove trailing slash to match route if it is before rest operator match (...) ", function() {
+					$window.location.href = prefix + "/test/d/"
+					route(root, "/test/:a...", {
+						"/test/:a" : {
+							view: lock(function(vnode) {
+								return JSON.stringify(route.param()) + " " +
+									JSON.stringify(vnode.attrs) + " " +
+									route.get()
+							})
+						},
+						"/test/:a..." : {
+							view: lock(function(vnode) {
+								return JSON.stringify(route.param()) + " " +
+									JSON.stringify(vnode.attrs) + " " +
+									route.get()
+							})
+						},
+					})
+
+					o(root.firstChild.nodeValue).equals(
+						'{"a":"d"} {"a":"d"} /test/d/'
+					)
+				})
+
 				o("handles route with search", function() {
 					$window.location.href = prefix + "/test?a=b&c=d"
 					route(root, "/test", {

pathname/compileTemplate.js

@@ -24,7 +24,7 @@ module.exports = function(template) {
 			if (extra === ".") return "([^/]+)\\."
 			return "([^/]+)" + (extra || "")
 		}
-	) + "$")
+	) + "\\/?$")
 	return function(data) {
 		// First, check the params. Usually, there isn't any, and it's just
 		// checking a static set.