From: Mike Samuel <mikesamuel@gmail.com>
Date: Mon, 26 Sep 2011 07:10:21 +0000 (-0700)
Subject: exp/template/html: remove TODO comments that have been done or mooted
X-Git-Tag: weekly.2011-10-06~103
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=b3d8e6d7f456eac13f29b21a9930e2b2ce0c4cae;p=gostls13.git

exp/template/html: remove TODO comments that have been done or mooted

R=nigeltao
CC=golang-dev
https://golang.org/cl/5128041
---

diff --git a/src/pkg/exp/template/html/transition.go b/src/pkg/exp/template/html/transition.go
index dd8cd59a6f..15548043b6 100644
--- a/src/pkg/exp/template/html/transition.go
+++ b/src/pkg/exp/template/html/transition.go
@@ -9,9 +9,6 @@ import (
 	"strings"
 )
 
-// TODO: ensure transition error messages contain template name and ideally
-// line info.
-
 // transitionFunc is the array of context transition functions for text nodes.
 // A transition function takes a context and template text input, and returns
 // the updated context and the number of bytes consumed from the front of the
diff --git a/src/pkg/exp/template/html/url.go b/src/pkg/exp/template/html/url.go
index 8a43e63645..8fdc8f77e8 100644
--- a/src/pkg/exp/template/html/url.go
+++ b/src/pkg/exp/template/html/url.go
@@ -21,13 +21,9 @@ func urlFilter(args ...interface{}) string {
 	if i >= 0 && strings.IndexRune(s[:i], '/') < 0 {
 		protocol := strings.ToLower(s[:i])
 		if protocol != "http" && protocol != "https" && protocol != "mailto" {
-			// Return a value that someone investigating a bug
-			// report can put into a search engine.
 			return "#" + filterFailsafe
 		}
 	}
-	// TODO: Once we handle <style>#id { background: url({{.Img}}) }</style>
-	// we will need to stop this from HTML escaping and pipeline sanitizers.
 	return s
 }