Skip to content

Conversation

@LionNatsu
Copy link
Contributor

Changes proposed in this pull request:

  • go: simplify nested literals
  • go: prune assignment of range for
  • go: omit the high index

I used gofmt -s -w to do this work, and committed them separately.

It is okay for Go>=1.5, because these changes are compatible to the old version (at least 6 yrs ago) of Go specs (Go 1.0 is about 4 yrs ago). the commit in tree of Go that introduced these grammar are as follows:

@coveralls
Copy link

coveralls commented Feb 2, 2017

Coverage Status

Coverage remained the same at 90.475% when pulling 45a1952 on LionNatsu:patch-simplify into 66c871e on yuin:master.

@yuin
Copy link
Owner

yuin commented Feb 2, 2017

Sorry, I prefer explicit code.

@yuin yuin closed this Feb 2, 2017
@LionNatsu LionNatsu deleted the patch-simplify branch February 3, 2017 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants