FAQ
Thank you for your review. I've made the changes you requested and added
some more words to try to describe the problem they are solving.


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
On 2013/06/04 16:53:53, agl1 wrote:
Comment for sysref? What it does is not clear from the name.
sysRefCount?
This code (and the support functions below) were cribbed from the net
package directly, their naming is mostly historical.

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 {
On 2013/06/04 16:53:53, agl1 wrote:
incRef?
Done.

https://codereview.appspot.com/9929043/diff/23001/ssh/test/test_unix_test.go#newcode173
ssh/test/test_unix_test.go:173: if closing {
On 2013/06/04 16:53:53, agl1 wrote:
could be just c.closing = closing.
This check is needed so that c.closing does not fall back to false.

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() {
On 2013/06/04 16:53:53, agl1 wrote:
decRef?
Done.

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

Discussion Posts

Follow ups

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 1 of 3 | next ›
Discussion Overview
groupgolang-dev @
categoriesgo
postedJun 5, '13 at 1:19a
activeJun 5, '13 at 1:46a
posts3
users2
websitegolang.org

2 users in discussion

Dave: 2 posts Agl: 1 post

People

Translate

site design / logo © 2022 Grokbase