codebase cleanup#238
Conversation
|
This PR (HEAD: 5965a06) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/net/+/675795. Important tips:
|
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/675795. |
|
Message from Ian Lance Taylor: Patch Set 1: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/675795. |
|
This PR (HEAD: 769e917) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/net/+/675795. Important tips:
|
ravichandra-eluri
left a comment
There was a problem hiding this comment.
LGTM. Nice cleanup! Switching to range over ints is definitely the move here.
|
This PR (HEAD: e5b46bb) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/net/+/675795. Important tips:
|
for loop refactoring,
remove unused func newStaticCloseChecker from http2/tansport_test/