From dc320164d82c222e5be85f81d290df4d9b5581d9 Mon Sep 17 00:00:00 2001
From: Paul Sokolovsky <pfalcon@users.sourceforge.net>
Date: Tue, 1 Mar 2016 00:19:43 +0200
Subject: [PATCH] unix/modsocket: Add comment regarding close() error checking
 (which is none).

---
 unix/modsocket.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/unix/modsocket.c b/unix/modsocket.c
index b5a6041b1..427a36b3c 100644
--- a/unix/modsocket.c
+++ b/unix/modsocket.c
@@ -114,6 +114,13 @@ STATIC mp_uint_t socket_write(mp_obj_t o_in, const void *buf, mp_uint_t size, in
 
 STATIC mp_obj_t socket_close(mp_obj_t self_in) {
     mp_obj_socket_t *self = MP_OBJ_TO_PTR(self_in);
+    // There's a POSIX drama regarding return value of close in general,
+    // and EINTR error in particular. See e.g.
+    // http://lwn.net/Articles/576478/
+    // http://austingroupbugs.net/view.php?id=529
+    // The rationale MicroPython follows is that close() just releases
+    // file descriptor. If you're interested to catch I/O errors before
+    // closing fd, fsync() it.
     close(self->fd);
     return mp_const_none;
 }
-- 
GitLab