Commit Graph

6 Commits (329f364ea23b1a081442790102823dd20ce325da)

Author SHA1 Message Date
Trevor Norris 752585db63 src: silence clang warnings
Mark several methods "override" in order to remove build warnings.

PR-URL: https://github.com/iojs/io.js/pull/531
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
2015-01-20 15:28:05 -08:00
Ben Noordhuis c3d3c8e8cd src: fixups after v8 upgrade
* v8::Platform has a new MonotonicallyIncreasingTime() method,
  implement it.

* The ASCII apocalypse continues with the replacement of external
  ASCII strings with external one byte strings.
2014-11-14 16:34:59 +01:00
Ben Noordhuis 3543c550c6 src: simplify v8 thread pool implementation
This commit drops the semaphore in exchange for a second condition
variable and makes the task ring an array member instead of allocating
it on the heap.  That in turn makes size calculations a little easier
because of the array's fixed size.

PR-URL: https://github.com/node-forward/node/pull/34
Reviewed-By: Fedor Indutny <fedor@indutny.com>
2014-10-28 12:56:46 +01:00
Ben Noordhuis 78e38f59ad src: fix uninitialized memory dereference
The elements of the heap-allocated TaskQueue::ring_ array in
src/node_v8_platform.cc were compared against without being
initialized first.

Fixes node-forward/node#33.

PR-URL: https://github.com/node-forward/node/pull/34
Reviewed-By: Fedor Indutny <fedor@indutny.com>
2014-10-28 12:42:14 +01:00
Ben Noordhuis 9f5800ab81 src: mark virtual functions with override keyword
Add `override` keywords where appropriate.  Makes maintenance easier
because the compiler will shout at you when a base class changes in
an incompatible way.
2014-10-23 22:49:59 +02:00
Fedor Indutny 50839a042e v8_platform: provide default v8::Platform impl
Provide default Platform implementation for v8's purposes.
2014-10-12 02:05:02 +04:00