lukechilds/node - node - Gitea: Git with a cup of tea

Commit Graph

Author	SHA1	Message	Date
Ben Noordhuis	a7581d0859	src: replace ARRAY_SIZE with typesafe arraysize To prevent `ARRAY_SIZE(&arg)` (i.e., taking the array size of a pointer) from happening again. PR-URL: https://github.com/nodejs/node/pull/5969 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>	9 years ago
Ali Ijaz Sheikh	79dc1d7635	src: remove forwards for v8::GC*logueCallback These types are no longer used in the file and V8 4.9 no longer defines these types anymore. PR-URL: https://github.com/nodejs/node/pull/4381 Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>	9 years ago
Michaël Zasso	4abc896a82	src: replace usage of v8::Handle with v8::Local v8::Handle is deprecated: https://codereview.chromium.org/1224623004 PR-URL: https://github.com/nodejs/io.js/pull/2202 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>	10 years ago
isaacs	3e1b1dd4a9	Remove excessive copyright/license boilerplate The copyright and license notice is already in the LICENSE file. There is no justifiable reason to also require that it be included in every file, since the individual files are not individually distributed except as part of the entire package.	10 years ago
Ben Noordhuis	d3c317e08a	src: attach env directly to api functions Attach the per-context execution environment directly to API functions. Rationale: * Gets node one step closer to multi-isolate readiness. * Avoids multi-context confusion, e.g. when the caller and callee live in different contexts. * Avoids expensive calls to pthread_getspecific() on platforms where V8 does not know how to use the thread-local storage directly. (Linux, the BSDs.) PR-URL: https://github.com/node-forward/node/pull/18 Reviewed-By: Fedor Indutny <fedor@indutny.com>	11 years ago
Ben Noordhuis	23dfa71dd5	src: fix tracing infrastructure after v8 upgrade Fix up the dtrace/etw/systemtap infrastructure after the V8 upgrade in commit `1c7bf24`. The win32 changes are untested but can hardly make things worse because node doesn't build on windows right now. Fixes #7313 with some luck.	11 years ago
Fedor Indutny	75adde07f9	src: remove `node_isolate` from source fix #6899	11 years ago
Ben Noordhuis	f674b09f40	src: use v8::String::NewFrom() functions Change calls to String::New() and String::NewSymbol() to their respective one-byte, two-byte and UTF-8 counterparts. * Add a FIXED_ONE_BYTE_STRING macro that takes a string literal and turns it into a v8::Local<v8::String>. * Add helper functions that make v8::String::NewFromOneByte() easier to work with. Said function expects a `const uint8_t` but almost every call site deals with `const char` or `const unsigned char`. Helps us avoid doing reinterpret_casts all over the place. Code that handles file system paths keeps using UTF-8 for backwards compatibility reasons. At least now the use of UTF-8 is explicit. * Remove v8::String::NewSymbol() entirely. Almost all call sites were effectively minor de-optimizations. If you create a string only once, there is no point in making it a symbol. If you are create the same string repeatedly, it should probably be cached in a persistent handle.	12 years ago
Fedor Indutny	8e29ce9f13	src: lint c++ code	12 years ago
Ben Noordhuis	110a9cd8db	lib, src: upgrade after v8 api change This is a big commit that touches just about every file in the src/ directory. The V8 API has changed in significant ways. The most important changes are: * Binding functions take a const v8::FunctionCallbackInfo<T>& argument rather than a const v8::Arguments& argument. * Binding functions return void rather than v8::Handle<v8::Value>. The return value is returned with the args.GetReturnValue().Set() family of functions. * v8::Persistent<T> no longer derives from v8::Handle<T> and no longer allows you to directly dereference the object that the persistent handle points to. This means that the common pattern of caching oft-used JS values in a persistent handle no longer quite works, you first need to reconstruct a v8::Local<T> from the persistent handle with the Local<T>::New(isolate, persistent) factory method. A handful of (internal) convenience classes and functions have been added to make dealing with the new API a little easier. The most visible one is node::Cached<T>, which wraps a v8::Persistent<T> with some template sugar. It can hold arbitrary types but so far it's exclusively used for v8::Strings (which was by far the most commonly cached handle type.)	12 years ago
Trevor Norris	f65e14ed1d	src: pass Isolate to all applicable api Update the api to pass node_isolate to all supported methods. Much thanks to Ben Noordhuis and his work in `51f6e6a`.	12 years ago
Trevor Norris	0bba590283	bindings: update api All compile time warnings about using deprecated APIs have been suppressed by updating node's API. Though there are still many function calls that can accept Isolate, and still need to be updated. node_isolate had to be added as an extern variable in node.h and node_object_wrap.h Also a couple small fixes for Error handling. Before v8 3.16.6 the error stack message was lazily written when it was needed, which allowed you to change the message after instantiation. Then the stack would be written with the new message the first time it was accessed. Though that has changed. Now it creates the stack message on instantiation. So setting a different message afterwards won't be displayed. This is not a complete fix for the problem. Getting error without any message isn't very useful.	12 years ago
Ben Noordhuis	51f6e6a9b3	src, test: downgrade to v8 3.14 api	12 years ago
Ben Noordhuis	7788a6bf85	src: pass node_isolate to Undefined()	12 years ago
Scott Blomquist	f657ce685d	windows: add tracing with performance counters Patch by Henry Rawas and Scott Blomquist.	12 years ago

15 Commits (5f3f54d4bb0919f2913f8930427927bef3f50b2e)