src: make Environment::is_stopping_worker inline

Fixes a TODO comment.
PR-URL: #21720
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

Author: maclover7

src/env-inl.h

@@ -33,6 +33,7 @@
 #include"node_perf_common.h"
 #include"node_context_data.h"
 #include"tracing/agent.h"
+#include"node_worker.h"
 
 #include<stddef.h>
 #include<stdint.h>
@@ -623,6 +624,11 @@ inline void Environment::remove_sub_worker_context(worker::Worker* context) {
  sub_worker_contexts_.erase(context);
 }
 
+inlineboolEnvironment::is_stopping_worker() const {
+CHECK(!is_main_thread());
+return worker_context_->is_stopped();
+}
+
 inline performance::performance_state* Environment::performance_state() {
 return performance_state_.get();
 }

src/env.cc

@@ -653,9 +653,4 @@ void Environment::stop_sub_worker_contexts() {
  }
 }
 
-boolEnvironment::is_stopping_worker() const {
-CHECK(!is_main_thread());
-return worker_context_->is_stopped();
-}
-
 } // namespace node

src/env.h

@@ -723,9 +723,6 @@ class Environment {
 inlineboolcan_call_into_js() const;
 inlinevoidset_can_call_into_js(bool can_call_into_js);
 
-// TODO(addaleax): This should be inline.
-boolis_stopping_worker() const;
-
 inlineboolis_main_thread() const;
 inlineuint64_tthread_id() const;
 inlinevoidset_thread_id(uint64_t id);
@@ -734,6 +731,7 @@ class Environment {
 inlinevoidadd_sub_worker_context(worker::Worker* context);
 inlinevoidremove_sub_worker_context(worker::Worker* context);
 voidstop_sub_worker_contexts();
+inlineboolis_stopping_worker() const;
 
 inlinevoidThrowError(constchar* errmsg);
 inlinevoidThrowTypeError(constchar* errmsg);