src: simplify HandleWrap initialization

src/fs_event_wrap.cc

@@ -53,7 +53,6 @@ private:
 
 FSEventWrap::FSEventWrap(Handle<Object> object)
     : HandleWrap(object, reinterpret_cast<uv_handle_t*>(&handle_)) {
-  handle_.data = static_cast<void*>(this);
   initialized_ = false;
 }
 

src/handle_wrap.cc

@@ -108,9 +108,7 @@ Handle<Value> HandleWrap::Close(const Arguments& args) {
 HandleWrap::HandleWrap(Handle<Object> object, uv_handle_t* h) {
   flags_ = 0;
   handle__ = h;
-  if (h) {
+  handle__->data = this;
-    h->data = this;
-  }
 
   HandleScope scope(node_isolate);
   assert(object_.IsEmpty());
@@ -121,12 +119,6 @@ HandleWrap::HandleWrap(Handle<Object> object, uv_handle_t* h) {
 }
 
 
-void HandleWrap::SetHandle(uv_handle_t* h) {
-  handle__ = h;
-  h->data = this;
-}
-
-
 HandleWrap::~HandleWrap() {
   assert(object_.IsEmpty());
   ngx_queue_remove(&handle_wrap_queue_);

src/handle_wrap.h

@@ -65,8 +65,6 @@ class HandleWrap {
     HandleWrap(v8::Handle<v8::Object> object, uv_handle_t* handle);
     virtual ~HandleWrap();
 
-    virtual void SetHandle(uv_handle_t* h);
-
     v8::Persistent<v8::Object> object_;
 
   private:

src/pipe_wrap.cc

@@ -139,7 +139,6 @@ PipeWrap::PipeWrap(Handle<Object> object, bool ipc)
   int r = uv_pipe_init(uv_default_loop(), &handle_, ipc);
   assert(r == 0); // How do we proxy this error up to javascript?
                   // Suggestion: uv_pipe_init() returns void.
-  handle_.data = static_cast<void*>(this);
   UpdateWriteQueueSize();
 }
 

src/process_wrap.cc

@@ -87,8 +87,12 @@ class ProcessWrap : public HandleWrap {
     return scope.Close(args.This());
   }
 
-  ProcessWrap(Handle<Object> object) : HandleWrap(object, NULL) { }
+  ProcessWrap(Handle<Object> object)
-  ~ProcessWrap() { }
+      : HandleWrap(object, reinterpret_cast<uv_handle_t*>(&process_)) {
+  }
+
+  ~ProcessWrap() {
+  }
 
   static void ParseStdioOptions(Local<Object> js_options,
                                 uv_process_options_t* options) {
@@ -248,7 +252,6 @@ class ProcessWrap : public HandleWrap {
       SetErrno(uv_last_error(uv_default_loop()));
     }
     else {
-      wrap->SetHandle(reinterpret_cast<uv_handle_t*>(&wrap->process_));
       assert(wrap->process_.data == wrap);
       wrap->object_->Set(String::New("pid"),
                          Integer::New(wrap->process_.pid, node_isolate));

src/stream_wrap.cc

@@ -112,9 +112,6 @@ void StreamWrap::Initialize(Handle<Object> target) {
 StreamWrap::StreamWrap(Handle<Object> object, uv_stream_t* stream)
     : HandleWrap(object, reinterpret_cast<uv_handle_t*>(stream)) {
   stream_ = stream;
-  if (stream) {
-    stream->data = this;
-  }
 }
 
 
@@ -131,13 +128,6 @@ Handle<Value> StreamWrap::GetFD(Local<String>, const AccessorInfo& args) {
 }
 
 
-void StreamWrap::SetHandle(uv_handle_t* h) {
-  HandleWrap::SetHandle(h);
-  stream_ = reinterpret_cast<uv_stream_t*>(h);
-  stream_->data = this;
-}
-
-
 void StreamWrap::UpdateWriteQueueSize() {
   HandleScope scope(node_isolate);
   object_->Set(write_queue_size_sym,

src/stream_wrap.h

@@ -57,7 +57,6 @@ class StreamWrap : public HandleWrap {
   static size_t WriteBuffer(v8::Handle<v8::Value> val, uv_buf_t* buf);
 
   StreamWrap(v8::Handle<v8::Object> object, uv_stream_t* stream);
-  virtual void SetHandle(uv_handle_t* h);
   void StateChange() { }
   void UpdateWriteQueueSize();
 

src/timer_wrap.cc

@@ -86,7 +86,6 @@ class TimerWrap : public HandleWrap {
       : HandleWrap(object, reinterpret_cast<uv_handle_t*>(&handle_)) {
     int r = uv_timer_init(uv_default_loop(), &handle_);
     assert(r == 0);
-    handle_.data = this;
   }
 
   ~TimerWrap() {

src/udp_wrap.cc

@@ -69,7 +69,6 @@ UDPWrap::UDPWrap(Handle<Object> object)
     : HandleWrap(object, reinterpret_cast<uv_handle_t*>(&handle_)) {
   int r = uv_udp_init(uv_default_loop(), &handle_);
   assert(r == 0); // can't fail anyway
-  handle_.data = reinterpret_cast<void*>(this);
 }