1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Revert "Add HandleScope to http-parser binding"

Browse Source 
This commit did not actually fix the production crashes.

This reverts commit 73cf8e82e768af870964d6f3375ab758e774165c.
This commit is contained in:
Ryan Dahl 2011-12-28 14:08:19 -08:00
parent d483acc5d9
commit 448c5e07ca
1 changed files with 10 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/node_http_parser.cc
View File

@ -120,7 +120,6 @@ static size_t current_buffer_len;
#define HTTP_CB(name) \ #define HTTP_CB(name) \
static int name(http_parser* p_) { \ static int name(http_parser* p_) { \
HandleScope scope; \
Parser* self = container_of(p_, Parser, parser_); \ Parser* self = container_of(p_, Parser, parser_); \
return self->name##_(); \ return self->name##_(); \
} \ } \
@ -129,7 +128,6 @@ static size_t current_buffer_len;
#define HTTP_DATA_CB(name) \ #define HTTP_DATA_CB(name) \
static int name(http_parser* p_, const char* at, size_t length) { \ static int name(http_parser* p_, const char* at, size_t length) { \
HandleScope scope; \
Parser* self = container_of(p_, Parser, parser_); \ Parser* self = container_of(p_, Parser, parser_); \
return self->name##_(at, length); \ return self->name##_(at, length); \
} \ } \
@ -214,12 +212,10 @@ struct StringPtr {
Handle<String> ToString() const { Handle<String> ToString() const {
HandleScope scope; if (str_)
if (str_) { return String::New(str_, size_);
return scope.Close(String::New(str_, size_)); else
} else { return String::Empty();
return scope.Close(String::Empty());
}
} }
@ -517,8 +513,6 @@ public:
private: private:
Local<Array> CreateHeaders() { Local<Array> CreateHeaders() {
HandleScope scope;
// num_values_ is either -1 or the entry # of the last header // num_values_ is either -1 or the entry # of the last header
// so num_values_ == 0 means there's a single header // so num_values_ == 0 means there's a single header
Local<Array> headers = Array::New(2 * (num_values_ + 1)); Local<Array> headers = Array::New(2 * (num_values_ + 1));
@ -528,7 +522,7 @@ private:
headers->Set(2 * i + 1, values_[i].ToString()); headers->Set(2 * i + 1, values_[i].ToString());
} }
return scope.Close(headers); return headers;
} }