Commit a6203f8b authored by Mike Kaplinskiy's avatar Mike Kaplinskiy Committed by Alexandre Julliard

server: Fix some comments.

Apparently windows reports only currently valid events for WSAEnumNetworkEvents.
parent 93db86a7
...@@ -3126,9 +3126,6 @@ static void test_events(int useMessages) ...@@ -3126,9 +3126,6 @@ static void test_events(int useMessages)
ok(ret == 2, "Failed to send buffer %d err %d\n", ret, GetLastError()); ok(ret == 2, "Failed to send buffer %d err %d\n", ret, GetLastError());
broken_seq[0] = read_read_seq; /* win9x */ broken_seq[0] = read_read_seq; /* win9x */
broken_seq[1] = NULL; broken_seq[1] = NULL;
/* we like to erase pmask in server, so we have varying behavior here *
* it is only fixed for now because we refuse to send notifications with
* any kind of asyncs requests running */
ok_event_seq(src, hEvent, empty_seq, broken_seq, 0); ok_event_seq(src, hEvent, empty_seq, broken_seq, 0);
dwRet = WaitForSingleObject(ov.hEvent, 100); dwRet = WaitForSingleObject(ov.hEvent, 100);
......
...@@ -961,7 +961,9 @@ DECL_HANDLER(enable_socket_event) ...@@ -961,7 +961,9 @@ DECL_HANDLER(enable_socket_event)
FILE_WRITE_ATTRIBUTES, &sock_ops))) FILE_WRITE_ATTRIBUTES, &sock_ops)))
return; return;
sock->pmask &= ~req->mask; /* is this safe? */ /* for event-based notification, windows erases stale events */
sock->pmask &= ~req->mask;
sock->hmask &= ~req->mask; sock->hmask &= ~req->mask;
sock->state |= req->sstate; sock->state |= req->sstate;
sock->state &= ~req->cstate; sock->state &= ~req->cstate;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment