Skip to content

Commit d619aab

Browse files
atlowCheminodejs-github-bot
authored andcommitted
events: rename high & low watermark for consistency
PR-URL: #52080 Fixes: #52078 Refs: #41276 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
1 parent ec15504 commit d619aab

File tree

2 files changed

+9
-7
lines changed

2 files changed

+9
-7
lines changed

lib/events.js

+8-6
Original file line numberDiff line numberDiff line change
@@ -1053,8 +1053,8 @@ function eventTargetAgnosticAddListener(emitter, name, listener, flags) {
10531053
* @param {{
10541054
* signal: AbortSignal;
10551055
* close?: string[];
1056-
* highWatermark?: number,
1057-
* lowWatermark?: number
1056+
* highWaterMark?: number,
1057+
* lowWaterMark?: number
10581058
* }} [options]
10591059
* @returns {AsyncIterator}
10601060
*/
@@ -1065,10 +1065,12 @@ function on(emitter, event, options = kEmptyObject) {
10651065
validateAbortSignal(signal, 'options.signal');
10661066
if (signal?.aborted)
10671067
throw new AbortError(undefined, { cause: signal?.reason });
1068-
const highWatermark = options.highWatermark ?? NumberMAX_SAFE_INTEGER;
1069-
validateInteger(highWatermark, 'options.highWatermark', 1);
1070-
const lowWatermark = options.lowWatermark ?? 1;
1071-
validateInteger(lowWatermark, 'options.lowWatermark', 1);
1068+
// Support both highWaterMark and highWatermark for backward compatibility
1069+
const highWatermark = options.highWaterMark ?? options.highWatermark ?? NumberMAX_SAFE_INTEGER;
1070+
validateInteger(highWatermark, 'options.highWaterMark', 1);
1071+
// Support both lowWaterMark and lowWatermark for backward compatibility
1072+
const lowWatermark = options.lowWaterMark ?? options.lowWatermark ?? 1;
1073+
validateInteger(lowWatermark, 'options.lowWaterMark', 1);
10721074

10731075
// Preparing controlling queues and variables
10741076
FixedQueue ??= require('internal/fixed_queue');

lib/internal/readline/interface.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1359,7 +1359,7 @@ class Interface extends InterfaceConstructor {
13591359
this[kLineObjectStream] = EventEmitter.on(
13601360
this, 'line', {
13611361
close: ['close'],
1362-
highWatermark: 1024,
1362+
highWaterMark: 1024,
13631363
[kFirstEventParam]: true,
13641364
});
13651365
}

0 commit comments

Comments
 (0)