From 776c3b04b5f289527f929e17b5d2c35dff0708ed Mon Sep 17 00:00:00 2001 From: Chengzhong Wu Date: Mon, 26 May 2025 17:54:06 +0100 Subject: [PATCH] src: remove deprecated node::EmitBeforeExit and node::EmitExit --- src/api/hooks.cc | 12 ------------ src/node.h | 4 ---- 2 files changed, 16 deletions(-) diff --git a/src/api/hooks.cc b/src/api/hooks.cc index f74950ae3de219..7b73f7db2427d9 100644 --- a/src/api/hooks.cc +++ b/src/api/hooks.cc @@ -26,10 +26,6 @@ void AtExit(Environment* env, void (*cb)(void* arg), void* arg) { env->AtExit(cb, arg); } -void EmitBeforeExit(Environment* env) { - USE(EmitProcessBeforeExit(env)); -} - Maybe EmitProcessBeforeExit(Environment* env) { TRACE_EVENT0(TRACING_CATEGORY_NODE1(environment), "BeforeExit"); if (!env->destroy_async_id_list()->empty()) @@ -50,14 +46,6 @@ Maybe EmitProcessBeforeExit(Environment* env) { : Just(true); } -static ExitCode EmitExitInternal(Environment* env) { - return EmitProcessExitInternal(env).FromMaybe(ExitCode::kGenericUserError); -} - -int EmitExit(Environment* env) { - return static_cast(EmitExitInternal(env)); -} - Maybe EmitProcessExitInternal(Environment* env) { // process.emit('exit') Isolate* isolate = env->isolate(); diff --git a/src/node.h b/src/node.h index f773096c959b59..ff376554a9aeab 100644 --- a/src/node.h +++ b/src/node.h @@ -867,13 +867,9 @@ NODE_EXTERN void SetTracingController(v8::TracingController* controller); // Run `process.emit('beforeExit')` as it would usually happen when Node.js is // run in standalone mode. NODE_EXTERN v8::Maybe EmitProcessBeforeExit(Environment* env); -NODE_DEPRECATED("Use Maybe version (EmitProcessBeforeExit) instead", - NODE_EXTERN void EmitBeforeExit(Environment* env)); // Run `process.emit('exit')` as it would usually happen when Node.js is run // in standalone mode. The return value corresponds to the exit code. NODE_EXTERN v8::Maybe EmitProcessExit(Environment* env); -NODE_DEPRECATED("Use Maybe version (EmitProcessExit) instead", - NODE_EXTERN int EmitExit(Environment* env)); // Runs hooks added through `AtExit()`. This is part of `FreeEnvironment()`, // so calling it manually is typically not necessary.