code review 4839056: exp/template: remove reflect from the API

exp/template: remove reflect from the API
It means keeping two sets of maps, but things look cleaner from
the outside.

[r, 2011-08-09 06:20:27]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[dsymonds, 2011-08-09 06:43:59]

LGTM

http://codereview.appspot.com/4839056/diff/1/src/pkg/exp/template/parse/parse.go
File src/pkg/exp/template/parse/parse.go (right):

http://codereview.appspot.com/4839056/diff/1/src/pkg/exp/template/parse/parse...
src/pkg/exp/template/parse/parse.go:408: func (t *Tree) findFunction(name
string) bool {
maybe s/findFunction/hasFunction/?

http://codereview.appspot.com/4839056/diff/1/src/pkg/exp/template/parse/parse...
src/pkg/exp/template/parse/parse.go:413: if fn := funcMap[name]; fn != nil {
don't need fn any more.

[r, 2011-08-09 06:49:46]

*** Submitted as http://code.google.com/p/go/source/detail?r=3041fde93860 ***

exp/template: remove reflect from the API
It means keeping two sets of maps, but things look cleaner from
the outside.

R=golang-dev, dsymonds
CC=golang-dev
http://codereview.appspot.com/4839056