github/node-v0.x-archive
1
1
Fork 0
mirror of https://github.com/nodejs/node-v0.x-archive.git synced 2026-04-28 03:01:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

Add string class that uses ExternalAsciiStringResource.

Browse Source 
Change the natives to use this class instead of creating completely new
strings. Reduces memory usage by about 1 MB.
This commit is contained in:
Tom Hughes
2011-03-08 00:00:51 -06:00
committed by Ryan Dahl
parent 81d3de7e6d
commit 74954ce7d8
8 changed files with 74 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/node_javascript.cc
View File

@@ -1,6 +1,7 @@
#include <v8.h>
#include "node.h"
#include "node_natives.h"
#include "node_string.h"
#include <string.h>
#include <strings.h>
@@ -8,8 +9,8 @@ using namespace v8;
namespace node {
const char* MainSource() {
return node_native;
Handle<String> MainSource() {
return BUILTIN_ASCII_ARRAY(node_native, sizeof(node_native)-1);
}
void DefineJavaScript(v8::Handle<v8::Object> target) {
@@ -18,14 +19,10 @@ void DefineJavaScript(v8::Handle<v8::Object> target) {
for (int i = 0; natives[i].name; i++) {
if (natives[i].source != node_native) {
Local<String> name = String::New(natives[i].name);
// TODO: Use ExternalAsciiStringResource for source
// Might need to do some assertions in js2c about chars > 128
Local<String> source = String::New(natives[i].source);
Handle<String> source = BUILTIN_ASCII_ARRAY(natives[i].source, natives[i].source_len);
target->Set(name, source);
}
}
}
} // namespace node