lib,src,test: fix comments

tniessen · MylesBorins · commit 1f01830ede52 · 2018-05-22T15:24:54.000-04:00
PR-URL: #20846 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
diff --git a/lib/_stream_readable.js b/lib/_stream_readable.js
@@ -834,7 +834,7 @@ Readable.prototype.removeListener = function(ev, fn) {
 
   if (ev === 'readable') {
     // We need to check if there is someone still listening to
-    // to readable and reset the state. However this needs to happen
+    // readable and reset the state. However this needs to happen
     // after readable has been emitted but before I/O (nextTick) to
     // support once('readable', fn) cycles. This means that calling
     // resume within the same tick will have no
@@ -850,7 +850,7 @@ Readable.prototype.removeAllListeners = function(ev) {
 
   if (ev === 'readable' || ev === undefined) {
     // We need to check if there is someone still listening to
-    // to readable and reset the state. However this needs to happen
+    // readable and reset the state. However this needs to happen
     // after readable has been emitted but before I/O (nextTick) to
     // support once('readable', fn) cycles. This means that calling
     // resume within the same tick will have no
diff --git a/src/node.cc b/src/node.cc
@@ -2384,7 +2384,7 @@ static void EnvGetter(Local<Name> property,
                                          arraysize(buffer));
   // If result >= sizeof buffer the buffer was too small. That should never
   // happen. If result == 0 and result != ERROR_SUCCESS the variable was not
-  // not found.
+  // found.
   if ((result > 0 || GetLastError() == ERROR_SUCCESS) &&
       result < arraysize(buffer)) {
     const uint16_t* two_byte_buffer = reinterpret_cast<const uint16_t*>(buffer);
diff --git a/src/stream_base-inl.h b/src/stream_base-inl.h
@@ -70,7 +70,7 @@ inline StreamResource::~StreamResource() {
     StreamListener* listener = listener_;
     listener->OnStreamDestroy();
     // Remove the listener if it didn&rsquo;t remove itself. This makes the logic
-    // logic in `OnStreamDestroy()` implementations easier, because they
+    // in `OnStreamDestroy()` implementations easier, because they
     // may call generic cleanup functions which can just remove the
     // listener unconditionally.
     if (listener == listener_)
diff --git a/src/tracing/node_trace_buffer.cc b/src/tracing/node_trace_buffer.cc
@@ -134,7 +134,7 @@ bool NodeTraceBuffer::Flush() {
 }
 
 // Attempts to set current_buf_ such that it references a buffer that can
-// can write at least one trace event. If both buffers are unavailable this
+// write at least one trace event. If both buffers are unavailable this
 // method returns false; otherwise it returns true.
 bool NodeTraceBuffer::TryLoadAvailableBuffer() {
   InternalTraceBuffer* prev_buf = current_buf_.load();
diff --git a/src/tracing/node_trace_writer.cc b/src/tracing/node_trace_writer.cc
@@ -153,7 +153,7 @@ void NodeTraceWriter::WriteToFile(std::string&& str, int highest_request_id) {
       write_req->str.length());
   request_mutex_.Lock();
   // Manage a queue of WriteRequest objects because the behavior of uv_write is
-  // is undefined if the same WriteRequest object is used more than once
+  // undefined if the same WriteRequest object is used more than once
   // between WriteCb calls. In addition, this allows us to keep track of the id
   // of the latest write request that actually been completed.
   write_req_queue_.push(write_req);
diff --git a/test/sequential/test-timers-blocking-callback.js b/test/sequential/test-timers-blocking-callback.js
@@ -3,7 +3,7 @@
 /*
  * This is a regression test for
  * https://github.com/nodejs/node-v0.x-archive/issues/15447 and
- * and https://github.com/nodejs/node-v0.x-archive/issues/9333.
+ * https://github.com/nodejs/node-v0.x-archive/issues/9333.
  *
  * When a timer is added in another timer's callback, its underlying timer
  * handle was started with a timeout that was actually incorrect.

Original file line number	Diff line number	Diff line change
`@@ -134,7 +134,7 @@ bool NodeTraceBuffer::Flush() {`
`134`	`134`	`}`
`135`	`135`
`136`	`136`	`// Attempts to set current_buf_ such that it references a buffer that can`
`137`		`-// can write at least one trace event. If both buffers are unavailable this`
	`137`	`+// write at least one trace event. If both buffers are unavailable this`
`138`	`138`	`// method returns false; otherwise it returns true.`
`139`	`139`	`bool NodeTraceBuffer::TryLoadAvailableBuffer() {`
`140`	`140`	`InternalTraceBuffer* prev_buf = current_buf_.load();`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`/*`
`4`	`4`	`* This is a regression test for`
`5`	`5`	`* https://github.com/nodejs/node-v0.x-archive/issues/15447 and`
`6`		`- * and https://github.com/nodejs/node-v0.x-archive/issues/9333.`
	`6`	`+ * https://github.com/nodejs/node-v0.x-archive/issues/9333.`
`7`	`7`	`*`
`8`	`8`	`* When a timer is added in another timer's callback, its underlying timer`
`9`	`9`	`* handle was started with a timeout that was actually incorrect.`