From: Ryan <ry@tinyclouds.org>
Date: Fri, 24 Jul 2009 18:09:26 +0000 (+0200)
Subject: Missing HandleScope in GetMethod(). Thanks Brian.
X-Git-Tag: v0.1.1~6
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=854538d09440c4acc89dab7e9596995094ea8051;p=platform%2Fupstream%2Fnodejs.git

Missing HandleScope in GetMethod(). Thanks Brian.
---

diff --git a/src/http.cc b/src/http.cc
index f7e0f05..dfa76dc 100644
--- a/src/http.cc
+++ b/src/http.cc
@@ -134,23 +134,26 @@ HTTPConnection::on_header_value (http_parser *parser, const char *buf, size_t le
 static inline Local<String>
 GetMethod (int method)
 {
+  const char *s;
   switch (method) {
-    case HTTP_COPY:       return String::NewSymbol("COPY");
-    case HTTP_DELETE:     return String::NewSymbol("DELETE");
-    case HTTP_GET:        return String::NewSymbol("GET");
-    case HTTP_HEAD:       return String::NewSymbol("HEAD");
-    case HTTP_LOCK:       return String::NewSymbol("LOCK");
-    case HTTP_MKCOL:      return String::NewSymbol("MKCOL");
-    case HTTP_MOVE:       return String::NewSymbol("MOVE");
-    case HTTP_OPTIONS:    return String::NewSymbol("OPTIONS");
-    case HTTP_POST:       return String::NewSymbol("POST");
-    case HTTP_PROPFIND:   return String::NewSymbol("PROPFIND");
-    case HTTP_PROPPATCH:  return String::NewSymbol("PROPPATCH");
-    case HTTP_PUT:        return String::NewSymbol("PUT");
-    case HTTP_TRACE:      return String::NewSymbol("TRACE");
-    case HTTP_UNLOCK:     return String::NewSymbol("UNLOCK");
+    case HTTP_COPY:       s = "COPY"; break;
+    case HTTP_DELETE:     s = "DELETE"; break;
+    case HTTP_GET:        s = "GET"; break;
+    case HTTP_HEAD:       s = "HEAD"; break;
+    case HTTP_LOCK:       s = "LOCK"; break;
+    case HTTP_MKCOL:      s = "MKCOL"; break;
+    case HTTP_MOVE:       s = "MOVE"; break;
+    case HTTP_OPTIONS:    s = "OPTIONS"; break;
+    case HTTP_POST:       s = "POST"; break;
+    case HTTP_PROPFIND:   s = "PROPFIND"; break;
+    case HTTP_PROPPATCH:  s = "PROPPATCH"; break;
+    case HTTP_PUT:        s = "PUT"; break;
+    case HTTP_TRACE:      s = "TRACE"; break;
+    case HTTP_UNLOCK:     s = "UNLOCK"; break;
   }
-  return Local<String>();
+  HandleScope scope;
+  Local<String> method = String::NewSymbol(s);
+  return scope.Close(method);
 }
 
 int