GitHub PushEvent at 2014-12-05 12:27:47
Repository: node
Actor: trevnorris
Payload: {"shas":[["df3a2b2cf21274fe7afc19d14ec0259b964e13f7","37ac5e111a9b2f779e373f78efa4f7678b93feb1@janeasystems.com","test: runner support for flaky tests\n\nAdding --flaky-tests option, to allow regarding flaky tests failures\nas non-fatal.\n\nCurrently only observed by the TapProgressIndicator, which will\nadd a # TODO directive to tests classified as flaky. According to the\nTAP specification, the test harness is supposed to treat failures\nthat have a # TODO directive as non-fatal.","Alexis Campailla",true],["0674cbaceb81ba1130355aa7eb8e6389824e3cc4","37ac5e111a9b2f779e373f78efa4f7678b93feb1@janeasystems.com","test: mark current flaky tests as flaky","Alexis Campailla",true],["62eb806e9d9068344d879d7b43fda7799fd0a72a","9cb5491eb5770dd8e0bf77a8a0335abf5f0488c5@gmail.com","src: remove async listener\n\nAsync listener will have minor reimplementation in a future commit in\nthe form of hooks for user implemented API. Though for simplicity of\ntesting it's being completely removed first.\n\nPR-URL: https://github.com/joyent/node/pull/8110","Trevor Norris",true],["0c7ab976b78133684ec6bf1cef5365314e9c2c66","9cb5491eb5770dd8e0bf77a8a0335abf5f0488c5@gmail.com","async-wrap: move MakeCallback to .cc\n\nMakeCallback is too large a function to have as an inline. Correct that\nby moving it to a .cc file.\n\nPR-URL: https://github.com/joyent/node/pull/8110","Trevor Norris",true],["e8dea22209ff02ebef0655f3a08f90b1e984a9c0","9cb5491eb5770dd8e0bf77a8a0335abf5f0488c5@gmail.com","node, async-wrap: remove MakeDomainCallback\n\nA lot of code duplication for the removal of a single if statement is\nnot worth the added code complexity.\n\nAlso fixed using v8::Value::As<T>() before verifying the type of the\nValue. Casting to an incorrect type will cause V8 to abort in debug\nmode.\n\nPR-URL: https://github.com/joyent/node/pull/8110","Trevor Norris",true],["dd7d161660a335c25cead1b825d8c74ae7ef3461","9cb5491eb5770dd8e0bf77a8a0335abf5f0488c5@gmail.com","node: fix throws before timer module is loaded\n\nAn edge case could occur when the setImmediate() in _fatalException()\nwould fire before the timers module had been loaded globally and cause\nNode to crash.\n\nPR-URL: https://github.com/joyent/node/pull/8110","Trevor Norris",true],["6a91f1f56f95c72e48dd0dd12dff20791c93329e","9cb5491eb5770dd8e0bf77a8a0335abf5f0488c5@gmail.com","src: all wrap's now use actual FunctionTemplate\n\nInstead of just creating a new Object to contain the connection\ninformation, instantiate a new instance of a FunctionTemplate. This will\nallow future improvements for debugging and performance probes.\n\nAdditionally class providers have been clarified such that the\n\"provider\" field in req_wrap.h is no longer optional.\n\nPR-URL: https://github.com/joyent/node/pull/8110","Trevor Norris",true],["71d6fbc648a3d2d39ff01bb93d3f90682bf9a9f9","9cb5491eb5770dd8e0bf77a8a0335abf5f0488c5@gmail.com","src: remove unnecessary template parameter\n\nPR-URL: https://github.com/joyent/node/pull/8110","Trevor Norris",true],["f53c3c9f03908440abef83474a3db1a0618276a3","9cb5491eb5770dd8e0bf77a8a0335abf5f0488c5@gmail.com","async-wrap: expose async-wrap as binding\n\nExpose basic hooks for AsyncWrap via the async_wrap binding. Right now\nonly the PROVIDER types are exposed. This is a preliminary step before\nmore functionality is added.\n\nPR-URL: https://github.com/joyent/node/pull/8110","Trevor Norris",true],["ec0846d03ec6dbecdf8b34beed5b192ff0c6ef34","9cb5491eb5770dd8e0bf77a8a0335abf5f0488c5@gmail.com","async-wrap: explicitly pass parent\n\nWhen instantiating a new AsyncWrap allow the parent AsyncWrap to be\npassed. This is useful for cases like TCP incoming connections, so the\nconnection can be tied to the server receiving the connection.\n\nBecause the current architecture instantiates the *Wrap inside a\nv8::FunctionCallback, the parent pointer is currently wrapped inside a\nnew v8::External every time and passed as an argument. A future\noptimization would be to add the v8::External as the data field when\ncreating the v8::FunctionTemplate, change the pointer just before making\nthe call then NULL'ing it out afterwards. Though this will only knock\noff an estimated 80ns, and the added code complexity is high. So not\nworth doing until the current approach is proven to be a bottleneck.\n\nPR-URL: https://github.com/joyent/node/pull/8110","Trevor Norris",true],["174a82ebcf1344bb58f67843be17b24e29d980f0","9cb5491eb5770dd8e0bf77a8a0335abf5f0488c5@gmail.com","async-wrap: add event hooks\n\nCall a user-defined callback at specific points in the lifetime of an\nasynchronous event. Which are on instantiation, just before/after the\ncallback has been run. If the error callback also throws an exception\nthere will be no mercy and the application will be forced to shutdown.\n\nCurrently these only tie into AsyncWrap, meaning no call to a hook\ncallback will be made for timers or process.nextTick() events. Though\nthose will be added in a future commit.\n\nHere are a few notes on how to make the hooks work:\n\n- The \"this\" of all event hook callbacks is the request object.\n- The zero field (kCallInitHook) of the flags object passed to\n  setupHooks() must be set != 0 before the init callback will be called.\n- In the \"init\" callback the property \"_asyncQueue\" must be attached to\n  the request object. e.g.\n\n  function initHook() {\n    this._asyncQueue = {};\n  }\n\n- DO NOT inspect the properties of the object in the init callback.\n  Since the object is in the middle of being instantiated there are some\n  cases when a getter is not complete, and doing so will cause Node to\n  crash.\n\nPR-URL: https://github.com/joyent/node/pull/8110","Trevor Norris",true]],"size":11,"ref":"refs/heads/remove-al-js","head":"174a82ebcf1344bb58f67843be17b24e29d980f0"}