From 8a8416f84169c553380704ab3a754db0a1735877 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sat, 8 Aug 2020 16:39:05 -0700 Subject: [PATCH] lib: use full URL to GitHub issues in comments Don't assume the reader of the code will know where to find the issue tracker. Provide the full URL. This is especially important if the issue tracker should move again. PR-URL: https://github.com/nodejs/node/pull/34686 Reviewed-By: Anna Henningsen Reviewed-By: Ricky Zhou <0x19951125@gmail.com> Reviewed-By: Richard Lau Reviewed-By: Luigi Pinca Reviewed-By: Ruben Bridgewater Reviewed-By: Mary Marchini Reviewed-By: Denys Otrishko Reviewed-By: James M Snell --- lib/internal/event_target.js | 7 ++++--- lib/internal/stream_base_commons.js | 6 ++++-- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/lib/internal/event_target.js b/lib/internal/event_target.js index a2a6452a3f..8f0657c8d5 100644 --- a/lib/internal/event_target.js +++ b/lib/internal/event_target.js @@ -201,7 +201,8 @@ function initEventTarget(self) { class EventTarget { // Used in checking whether an object is an EventTarget. This is a well-known - // symbol as EventTarget may be used cross-realm. See discussion in #33661. + // symbol as EventTarget may be used cross-realm. + // Ref: https://github.com/nodejs/node/pull/33661 static [kIsEventTarget] = true; constructor() { @@ -520,8 +521,8 @@ function validateEventListenerOptions(options) { // Test whether the argument is an event object. This is far from a fool-proof // test, for example this input will result in a false positive: // > isEventTarget({ constructor: EventTarget }) -// It stands in its current implementation as a compromise. For the relevant -// discussion, see #33661. +// It stands in its current implementation as a compromise. +// Ref: https://github.com/nodejs/node/pull/33661 function isEventTarget(obj) { return obj && obj.constructor && obj.constructor[kIsEventTarget]; } diff --git a/lib/internal/stream_base_commons.js b/lib/internal/stream_base_commons.js index cef87c4cfd..0dfee0c9c7 100644 --- a/lib/internal/stream_base_commons.js +++ b/lib/internal/stream_base_commons.js @@ -208,7 +208,8 @@ function onStreamRead(arrayBuffer) { } if (nread !== UV_EOF) { - // #34375 CallJSOnreadMethod expects the return value to be a buffer. + // CallJSOnreadMethod expects the return value to be a buffer. + // Ref: https://github.com/nodejs/node/pull/34375 stream.destroy(errnoException(nread, 'read')); return; } @@ -228,7 +229,8 @@ function onStreamRead(arrayBuffer) { if (handle.readStop) { const err = handle.readStop(); if (err) { - // #34375 CallJSOnreadMethod expects the return value to be a buffer. + // CallJSOnreadMethod expects the return value to be a buffer. + // Ref: https://github.com/nodejs/node/pull/34375 stream.destroy(errnoException(err, 'read')); return; }