Skip to content

Commit d2956fe

Browse files
sjungwon03aduh95
authored andcommitted
src: fix comment typos
Correct typos in builtin, platform, and QUIC source comments. Signed-off-by: sjungwon03 <sjungwon03@gmail.com> PR-URL: #64509 Reviewed-By: Tim Perry <pimterry@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
1 parent a6f3da4 commit d2956fe

4 files changed

Lines changed: 5 additions & 5 deletions

File tree

src/node_builtins.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -557,7 +557,7 @@ bool BuiltinLoader::CompileAllBuiltinsAndCopyCodeCache(
557557
std::unordered_set(eager_builtins.begin(), eager_builtins.end());
558558

559559
for (const auto& id : ids) {
560-
// Eagerly compile primordials/boostrap/main scripts during code cache
560+
// Eagerly compile primordials/bootstrap/main scripts during code cache
561561
// generation.
562562
if (id.starts_with(primordial_prefix) || id.starts_with(bootstrap_prefix) ||
563563
id.starts_with(main_prefix)) {

src/node_builtins.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -207,7 +207,7 @@ class NODE_EXTERN_PRIVATE BuiltinLoader {
207207

208208
const UnionBytes config_;
209209

210-
// If any bulitins should be eagerly compiled i.e. with inner functions
210+
// If any builtins should be eagerly compiled i.e. with inner functions
211211
// compiled too, either use should_eager_compile_ to compile all builtins
212212
// eagerly, or use to_eager_compile_ to compile specific builtins eagerly.
213213
// Currently we set should_eager_compile_ to true when compiling primordials,

src/node_platform.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -597,9 +597,9 @@ void NodePlatform::DrainTasks(Isolate* isolate) {
597597
// However, not blocking on the worker tasks at all can lead to loss of some
598598
// critical user-blocking worker tasks e.g. wasm async compilation tasks,
599599
// which should block the main thread until they are completed, as the
600-
// documentation suggets. As a compromise, we currently only block on
600+
// documentation suggests. As a compromise, we currently only block on
601601
// user-blocking tasks to reduce the chance of deadlocks while making sure
602-
// that criticl user-blocking tasks are not lost.
602+
// that critical user-blocking tasks are not lost.
603603
worker_thread_task_runner_->BlockingDrain();
604604
} while (per_isolate->FlushForegroundTasksInternal());
605605
}

src/quic/streams.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1262,7 +1262,7 @@ void Stream::NotifyStreamOpened(stream_id id) {
12621262
headers->flags);
12631263
}
12641264
}
1265-
// If the stream is not a local undirectional stream and is_readable is
1265+
// If the stream is not a local unidirectional stream and is_readable is
12661266
// false, then we should shutdown the streams readable side now.
12671267
if (!is_local_unidirectional() && !is_readable()) {
12681268
NotifyReadableEnded(pending_close_read_code_);

0 commit comments

Comments
 (0)