s/Routes/routes in the private router struct

Author: ant0ine

handler.go

@@ -138,17 +138,17 @@ func (self *ResourceHandler) SetRoutes(routes ...Route) error {
 	self.internalRouter = &router{}
 
 	for _, route := range routes {
-		self.internalRouter.Routes = append(
-			self.internalRouter.Routes,
+		self.internalRouter.routes = append(
+			self.internalRouter.routes,
 			route,
 		)
 	}
 
 	// add the status route as the last route.
 	if self.EnableStatusService == true {
 		self.statusService = newStatusService()
-		self.internalRouter.Routes = append(
-			self.internalRouter.Routes,
+		self.internalRouter.routes = append(
+			self.internalRouter.routes,
 			Route{
 				HttpMethod: "GET",
 				PathExp:    "/.status",

router.go

@@ -10,24 +10,24 @@ import (
 // support for #param placeholder ?
 
 type router struct {
-	// list of Routes, the order matters, if multiple Routes match, the first defined will be used.
-	Routes                 []Route
+	routes                 []Route
 	disableTrieCompression bool
 	index                  map[*Route]int
 	trie                   *trie.Trie
 }
 
 // This validates the Routes and prepares the Trie data structure.
 // It must be called once the Routes are defined and before trying to find Routes.
+// The order matters, if multiple Routes match, the first defined will be used.
 func (self *router) start() error {
 
 	self.trie = trie.New()
 	self.index = map[*Route]int{}
 
-	for i, _ := range self.Routes {
+	for i, _ := range self.routes {
 
 		// pointer to the Route
-		route := &self.Routes[i]
+		route := &self.routes[i]
 
 		// insert in the Trie
 		err := self.trie.AddRoute(

router_benchmark_test.go

@@ -58,7 +58,7 @@ func BenchmarkNoCompression(b *testing.B) {
 	b.StopTimer()
 
 	r := router{
-		Routes:                 routes(),
+		routes:                 routes(),
 		disableTrieCompression: true,
 	}
 	r.start()
@@ -78,7 +78,7 @@ func BenchmarkCompression(b *testing.B) {
 	b.StopTimer()
 
 	r := router{
-		Routes: routes(),
+		routes: routes(),
 	}
 	r.start()
 	urlObjs := requestUrls()

router_test.go

@@ -8,7 +8,7 @@ import (
 func TestFindRouteAPI(t *testing.T) {
 
 	r := router{
-		Routes: []Route{
+		routes: []Route{
 			Route{
 				HttpMethod: "GET",
 				PathExp:    "/",
@@ -64,7 +64,7 @@ func TestFindRouteAPI(t *testing.T) {
 func TestNoRoute(t *testing.T) {
 
 	r := router{
-		Routes: []Route{},
+		routes: []Route{},
 	}
 
 	err := r.start()
@@ -89,7 +89,7 @@ func TestNoRoute(t *testing.T) {
 func TestDuplicatedRoute(t *testing.T) {
 
 	r := router{
-		Routes: []Route{
+		routes: []Route{
 			Route{
 				HttpMethod: "GET",
 				PathExp:    "/",
@@ -110,7 +110,7 @@ func TestDuplicatedRoute(t *testing.T) {
 func TestRouteOrder(t *testing.T) {
 
 	r := router{
-		Routes: []Route{
+		routes: []Route{
 			Route{
 				HttpMethod: "GET",
 				PathExp:    "/r/:id",
@@ -144,7 +144,7 @@ func TestRouteOrder(t *testing.T) {
 func TestSimpleExample(t *testing.T) {
 
 	r := router{
-		Routes: []Route{
+		routes: []Route{
 			Route{
 				HttpMethod: "GET",
 				PathExp:    "/resources/:id",