From 0400d0cec317824cd9bbaa85c7de75d1c82a0127 Mon Sep 17 00:00:00 2001 From: Dave Collins Date: Sat, 11 May 2013 23:22:09 -0500 Subject: [PATCH] Remove dead error check in WriteMessage. The io.Writer.Write function always returns an error if the bytes written is less than provided, so there is no reason to further check if the payload length matches after a successful write. --- message.go | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/message.go b/message.go index f1ee1f4f..80b748a9 100644 --- a/message.go +++ b/message.go @@ -196,15 +196,10 @@ func WriteMessage(w io.Writer, msg Message, pver uint32, btcnet BitcoinNet) erro } // Write payload. - n, err := w.Write(payload) + _, err = w.Write(payload) if err != nil { return err } - if n != lenp { - str := fmt.Sprintf("failed to write payload - wrote %v bytes, "+ - "payload size is %v bytes", n, lenp) - return messageError("WriteMessage", str) - } return nil }