diff options
author | aarzilli <alessandro.arzilli@gmail.com> | 2015-11-08 10:56:15 +0100 |
---|---|---|
committer | Přemysl Janouch <p@janouch.name> | 2018-09-08 16:49:24 +0200 |
commit | baff8c19067489ea26bbb804ef693e966202b511 (patch) | |
tree | 4ae49572739ae2ad7d65e8877ccfc1ee36e77336 | |
parent | 5451e59f884298fdf97ef4420a7bc1001093c3b7 (diff) | |
download | haven-baff8c19067489ea26bbb804ef693e966202b511.tar.gz haven-baff8c19067489ea26bbb804ef693e966202b511.tar.xz haven-baff8c19067489ea26bbb804ef693e966202b511.zip |
sendRequest reads req.buf after closing req.seq
NewRequest says you can avoid reallocating a new buffer for each request by calling it directly.
This is not true if req.seq is closed before req.buf is read.
-rw-r--r-- | nexgb/xgb.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/nexgb/xgb.go b/nexgb/xgb.go index 487ae16..44634fe 100644 --- a/nexgb/xgb.go +++ b/nexgb/xgb.go @@ -337,9 +337,9 @@ func (c *Conn) sendRequests() { } } req.cookie.Sequence = c.newSequenceId() - close(req.seq) c.cookieChan <- req.cookie c.writeBuffer(req.buf) + close(req.seq) } response := make(chan struct{}) c.closing <- response |