Skip to content

Commit

Permalink
src: make Environment::is_stopping_worker inline
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
maclover7 authored and targos committed Jul 12, 2018
1 parent 6920091 commit 2770778
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 8 deletions.
6 changes: 6 additions & 0 deletions src/env-inl.h
Original file line number Diff line number Diff line change
Expand Up @@ -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>
Expand Down Expand Up @@ -623,6 +624,11 @@ inline void Environment::remove_sub_worker_context(worker::Worker* context) {
sub_worker_contexts_.erase(context);
}

inline bool Environment::is_stopping_worker() const {
CHECK(!is_main_thread());
return worker_context_->is_stopped();
}

inline performance::performance_state* Environment::performance_state() {
return performance_state_.get();
}
Expand Down
5 changes: 0 additions & 5 deletions src/env.cc
Original file line number Diff line number Diff line change
Expand Up @@ -653,9 +653,4 @@ void Environment::stop_sub_worker_contexts() {
}
}

bool Environment::is_stopping_worker() const {
CHECK(!is_main_thread());
return worker_context_->is_stopped();
}

} // namespace node
4 changes: 1 addition & 3 deletions src/env.h
Original file line number Diff line number Diff line change
Expand Up @@ -723,9 +723,6 @@ class Environment {
inline bool can_call_into_js() const;
inline void set_can_call_into_js(bool can_call_into_js);

// TODO(addaleax): This should be inline.
bool is_stopping_worker() const;

inline bool is_main_thread() const;
inline uint64_t thread_id() const;
inline void set_thread_id(uint64_t id);
Expand All @@ -734,6 +731,7 @@ class Environment {
inline void add_sub_worker_context(worker::Worker* context);
inline void remove_sub_worker_context(worker::Worker* context);
void stop_sub_worker_contexts();
inline bool is_stopping_worker() const;

inline void ThrowError(const char* errmsg);
inline void ThrowTypeError(const char* errmsg);
Expand Down

0 comments on commit 2770778

Please sign in to comment.