Code review comment for lp:~rogpeppe/goyaml/goyaml-rune-changes

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

LGTM

https://codereview.appspot.com/5496045/diff/1/goyaml.go
File goyaml.go (right):

https://codereview.appspot.com/5496045/diff/1/goyaml.go#newcode200
goyaml.go:200: case rune('c'):
Do we really need the typing in these cases? The constant should work I
think.

https://codereview.appspot.com/5496045/

« Back to merge proposal