Nikhil Marathe avatar Nikhil Marathe committed 6b676cb

Changed non-static, non v8 Parse(Complete) return types to void

Comments (0)

Files changed (2)

 Handle::~Handle()
 {
     yajl_free( yc_handle );
+    yc_handle = NULL;
 }
 
 v8::Handle<Value> Handle::Parse( const Arguments& args )
     HandleScope scope;
     Handle *yh = Unwrap<Handle>( args.This() );
     yh->ParseComplete();
+    return Null();
 }
 
-v8::Handle<Value> Handle::Parse( unsigned char* str, int len )
+void Handle::Parse( unsigned char* str, int len )
 {
     int status = yajl_parse( yc_handle, str, len );
     if( status != yajl_status_ok
     }
 }
 
-v8::Handle<Value> Handle::ParseComplete()
+void Handle::ParseComplete()
 {
-    int status = yajl_parse_complete( yc_handle );
+    yajl_status status = yajl_parse_complete( yc_handle );
     if( status != yajl_status_ok
         && status != yajl_status_insufficient_data )
     {
     ~Handle();
 
   private:
-    v8::Handle<v8::Value> Parse( unsigned char*, int );
-    v8::Handle<v8::Value> ParseComplete();
+    void Parse( unsigned char*, int );
+    void ParseComplete();
     yajl_handle yc_handle;
     yajl_callbacks callbacks;
 };
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.