realistschuckle / gohaml

An implementation of the popular XHTML Abstraction Markup Language using the Go language.
MIT License
95 stars 13 forks source link

refactored for Go 1 #6

Closed a2800276 closed 11 years ago

a2800276 commented 11 years ago

Hi curtis,

I tinkered around a bit to make gohaml work with go 1.

Main changes:

I could not for the life of me figure out why the order of the attributes in the output seems to bob around randomly, though my guess is it has to do with storing attributes in maps and that you were having similar problems that you dealt with by using .orexpected in the tests. I added .orexpected values to failing tests because I didn't want to screw up the implementation beyond recognition.

(Also fixed the attributes show up in no-deterministic-order problem and updated the tests.)

I also ran gofmt over the code, which may or may not have been a good idea as it makes the diffs harder to read for you. The formatting and fixes are in distinct commits though, if you want to examine them individually.

INext steps: I'd like to add an http.Handler function that will prepare input (fyi and to hear your thoughts)

realistschuckle commented 11 years ago

Thanks for that update, Tim. I look forward to continue working with you.