[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wireguard
Subject:    [PATCH 08/12] conn_linux: move comment to make its meaning more obvious
From:       Simon Ruderich <simon () ruderich ! org>
Date:       2018-01-01 10:52:58
Message-ID: e4cd6deb1ac95a3d29803e4c04f5db8863c1a2a1.1514803815.git.simon () ruderich ! org
[Download RAW message or body]

At first I didn't understand the connection between errno == 0 and
"error instance", putting it immediately before the nil should help.
---
 src/conn_linux.go | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/conn_linux.go b/src/conn_linux.go
index 63ad2ec..571c882 100644
--- a/src/conn_linux.go
+++ b/src/conn_linux.go
@@ -477,9 +477,8 @@ func send4(sock int, end *NativeEndpoint, buff []byte) error {
 		)
 	}
 
-	// errno = 0 is still an error instance
-
 	if errno == 0 {
+		// errno = 0 is still an error instance
 		return nil
 	}
 
-- 
2.15.1

_______________________________________________
WireGuard mailing list
WireGuard@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/wireguard
[prev in list] [next in list] [prev in thread] [next in thread]