Rename bytesRead/Written to bytesReceived/Sent.

This makes it a little more clear the variables reprsent bytes sent across
the network as opposed to from disk.
This commit is contained in:
Dave Collins 2014-02-04 22:30:34 -06:00
parent 7cfef69f23
commit f8c843e2e3
2 changed files with 8 additions and 8 deletions

12
peer.go
View file

@ -137,8 +137,8 @@ type peer struct {
timeConnected time.Time timeConnected time.Time
lastSend time.Time lastSend time.Time
lastRecv time.Time lastRecv time.Time
bytesRead uint64 bytesReceived uint64
bytesWritten uint64 bytesSent uint64
inbound bool inbound bool
connected int32 connected int32
disconnect int32 // only to be used atomically disconnect int32 // only to be used atomically
@ -956,10 +956,10 @@ func (p *peer) handlePongMsg(msg *btcwire.MsgPong) {
func (p *peer) readMessage() (btcwire.Message, []byte, error) { func (p *peer) readMessage() (btcwire.Message, []byte, error) {
n, msg, buf, err := btcwire.ReadMessageN(p.conn, p.protocolVersion, p.btcnet) n, msg, buf, err := btcwire.ReadMessageN(p.conn, p.protocolVersion, p.btcnet)
if err != nil { if err != nil {
p.bytesRead += uint64(n) p.bytesReceived += uint64(n)
return nil, nil, err return nil, nil, err
} }
p.bytesRead += uint64(n) p.bytesReceived += uint64(n)
// Use closures to log expensive operations so they are only run when // Use closures to log expensive operations so they are only run when
// the logging level requires it. // the logging level requires it.
@ -1025,12 +1025,12 @@ func (p *peer) writeMessage(msg btcwire.Message) {
// Write the message to the peer. // Write the message to the peer.
n, err := btcwire.WriteMessageN(p.conn, msg, p.protocolVersion, p.btcnet) n, err := btcwire.WriteMessageN(p.conn, msg, p.protocolVersion, p.btcnet)
if err != nil { if err != nil {
p.bytesWritten += uint64(n) p.bytesSent += uint64(n)
p.Disconnect() p.Disconnect()
p.logError("Can't send message: %v", err) p.logError("Can't send message: %v", err)
return return
} }
p.bytesWritten += uint64(n) p.bytesSent += uint64(n)
} }
// isAllowedByRegression returns whether or not the passed error is allowed by // isAllowedByRegression returns whether or not the passed error is allowed by

View file

@ -330,8 +330,8 @@ func (s *server) handleQuery(querymsg interface{}, state *peerState) {
Services: fmt.Sprintf("%08d", p.services), Services: fmt.Sprintf("%08d", p.services),
LastSend: p.lastSend.Unix(), LastSend: p.lastSend.Unix(),
LastRecv: p.lastRecv.Unix(), LastRecv: p.lastRecv.Unix(),
BytesSent: p.bytesWritten, BytesSent: p.bytesSent,
BytesRecv: p.bytesRead, BytesRecv: p.bytesReceived,
ConnTime: p.timeConnected.Unix(), ConnTime: p.timeConnected.Unix(),
Version: p.protocolVersion, Version: p.protocolVersion,
SubVer: p.userAgent, SubVer: p.userAgent,