News for hackers. Stuff that matters.

A comment on Show HN: Create generative background images from a string in Go. Parent comment

Re:Show HN: Create generative background images from a strin... by betamike on Tuesday October 28, 2014 @04:49PM (#8523498)
Another minor nitpick, here [1] you are using reflection. You can pretty easily convert it to a type switch [2], which is more idiomatic.

[1] https://github.com/pravj/geo_pattern/blob/master/svg/svg.go#...

[2] https://golang.org/doc/effective_go.html#type_switch

[ Parent | on HN ]

There are no comments on this comment yet.