[squash] bnoordhuis nits

src/stream_base-inl.h

@@ -37,8 +37,7 @@ inline StreamReq* StreamReq::FromObject(v8::Local<v8::Object> req_wrap_obj) {
 }
 
 inline void StreamReq::Dispose() {
-  object()->SetAlignedPointerInInternalField(
-      kStreamReqField, nullptr);
+  object()->SetAlignedPointerInInternalField(kStreamReqField, nullptr);
   delete this;
 }
 
@@ -153,10 +152,6 @@ inline void StreamBase::AfterRequest(Wrap* req_wrap, EmitEvent emit) {
   v8::HandleScope handle_scope(env->isolate());
   v8::Context::Scope context_scope(env->context());
 
-  // Remove the reference to the .handle property
-  v8::Local<v8::Object> req_wrap_obj = req_wrap->object();
-  req_wrap_obj->Delete(env->context(), env->handle_string()).FromJust();
-
   emit();
   req_wrap->Dispose();
 }

src/stream_base.cc

@@ -168,7 +168,7 @@ int StreamBase::WriteBuffer(const FunctionCallbackInfo<Value>& args) {
   Local<Object> req_wrap_obj = args[0].As<Object>();
 
   uv_buf_t buf;
-  buf.base = const_cast<char*>(Buffer::Data(args[1]));
+  buf.base = Buffer::Data(args[1]);
   buf.len = Buffer::Length(args[1]);
 
   StreamWriteResult res = Write(&buf, 1, nullptr, req_wrap_obj);

src/stream_base.h

@@ -116,13 +116,13 @@ class StreamListener {
 
   // This is called once a write has finished. `status` may be 0 or,
   // if negative, a libuv error code.
-  // By the fault, this is simply passed on to the previous listener
+  // By default, this is simply passed on to the previous listener
   // (and raises an assertion if there is none).
   virtual void OnStreamAfterWrite(WriteWrap* w, int status);
 
   // This is called once a shutdown has finished. `status` may be 0 or,
   // if negative, a libuv error code.
-  // By the fault, this is simply passed on to the previous listener
+  // By default, this is simply passed on to the previous listener
   // (and raises an assertion if there is none).
   virtual void OnStreamAfterShutdown(ShutdownWrap* w, int status);