Windows IPC working, mostly.

The IOCP code has been refactored to improve reuse, and hopefully
will be easier to use with TCP now.  Windows IPC using Named Pipes
is mostly working -- mostly because there is a gnarly close-race.
It seems that we need to take some more care to ensure that the
pipe is not released while requests may be outstanding -- so some
deeper synchronization between the IOCP callback logic and the
win_event code is needed.  In short, we need to add a condvar to
the event, and notice when we have submitted work for async completion,
and make sure we flag the event "idle" after either completion or
cancellation of the event.

1 files changed, 15 insertions, 6 deletions

diff --git a/src/platform/windows/win_debug.c b/src/platform/windows/win_debug.c
index bc3d5bfa..cc2adb88 100644
--- a/src/platform/windows/win_debug.c
+++ b/src/platform/windows/win_debug.c
@@ -42,11 +42,20 @@ static struct {
 	int sys_err;
 	int nng_err;
 } nni_plat_errnos[] = {
-	{ ENOENT, NNG_ENOENT }, { EINTR, NNG_EINTR }, { EINVAL, NNG_EINVAL },
-	{ ENOMEM, NNG_ENOMEM }, { EACCES, NNG_EPERM }, { EAGAIN, NNG_EAGAIN },
-	{ EBADF, NNG_ECLOSED }, { EBUSY, NNG_EBUSY },
-	{ ENAMETOOLONG, NNG_EINVAL }, { EPERM, NNG_EPERM },
-	{ EPIPE, NNG_ECLOSED }, { 0, 0 } // must be last
+	// clang-format off
+	{ ENOENT,	NNG_ENOENT },
+	{ EINTR,	NNG_EINTR },
+	{ EINVAL,	NNG_EINVAL },
+	{ ENOMEM,	NNG_ENOMEM },
+	{ EACCES,	NNG_EPERM },
+	{ EAGAIN,	NNG_EAGAIN },
+	{ EBADF,	NNG_ECLOSED },
+	{ EBUSY,	NNG_EBUSY },
+	{ ENAMETOOLONG,	NNG_EINVAL },
+	{ EPERM,	NNG_EPERM },
+	{ EPIPE,	NNG_ECLOSED },
+	{ 0,		0 } // must be last
+	// clang-format on
 };
 
 int
@@ -107,7 +116,7 @@ nni_win_error(int errnum)
 	if (errnum == 0) {
 		return (0);
 	}
-	for (i = 0; nni_win_errnos[i].nng_err != 0; i++) {
+	for (i = 0; nni_win_errnos[i].win_err != 0; i++) {
 		if (errnum == nni_win_errnos[i].win_err) {
 			return (nni_win_errnos[i].nng_err);
 		}