FAQ
LGTM


https://codereview.appspot.com/9929043/diff/23001/ssh/test/test_unix_test.go
File ssh/test/test_unix_test.go (right):

https://codereview.appspot.com/9929043/diff/23001/ssh/test/test_unix_test.go#newcode154
ssh/test/test_unix_test.go:154: sysref int
Comment for sysref? What it does is not clear from the name.

sysRefCount?

https://codereview.appspot.com/9929043/diff/23001/ssh/test/test_unix_test.go#newcode166
ssh/test/test_unix_test.go:166: func (c *client) incref(closing bool)
error {
incRef?

https://codereview.appspot.com/9929043/diff/23001/ssh/test/test_unix_test.go#newcode173
ssh/test/test_unix_test.go:173: if closing {
could be just c.closing = closing.

https://codereview.appspot.com/9929043/diff/23001/ssh/test/test_unix_test.go#newcode179
ssh/test/test_unix_test.go:179: func (c *client) decref() {
decRef?

https://codereview.appspot.com/9929043/

--

---
You received this message because you are subscribed to the Google Groups "golang-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to golang-dev+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupgolang-dev @
categoriesgo
postedJun 4, '13 at 4:53p
activeJun 4, '13 at 4:53p
posts1
users1
websitegolang.org

1 user in discussion

Agl: 1 post

People

Translate

site design / logo © 2021 Grokbase