Browse Source

Fix comment/test for Mac getaddinfo() bug.

Note: the test case was broken because on Macintosh the client connects
synchronously. This is undesirable but ok for now.
v0.7.4-release
Ryan 16 years ago
parent
commit
f657a6324a
  1. 5
      src/net.cc
  2. 2
      test/test-tcp-pingpong.js

5
src/net.cc

@ -184,7 +184,10 @@ Connection::Connect (const Arguments& args)
}
#ifdef __APPLE__
/* Bypass the thread pool and do it sync on this broken ass platform */
/* HACK: Bypass the thread pool and do it sync on Macintosh.
* Experiecing strange error where execution halts on
* getaddrinfo() and CPU goes to 100%. FIXME.
*/
eio_req *req = static_cast<eio_req*>(malloc(sizeof(eio_req)));
req->data = connection;
Connection::Resolve(req);

2
test/test-tcp-pingpong.js

@ -68,8 +68,8 @@ function onLoad() {
client.onEOF = function () {
};
client.connect(port);
assertEquals("closed", client.readyState);
client.connect(port);
}
function onExit () {

Loading…
Cancel
Save