Commit 24a5929
src: make debug_options getters public
This simplifies requires for those using DebugOptions,
since debug_options was defined in src/node_options-inl.h and thus
embedders would need to require an extra file to do what could
trivially be consolidated into one.
PR-URL: #30494
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>1 parent 2d23502 commit 24a5929
2 files changed
Lines changed: 3 additions & 10 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -13,14 +13,6 @@ PerIsolateOptions* PerProcessOptions::get_per_isolate_options() { | |||
| 13 | 13 | return per_isolate.get(); | |
| 14 | 14 | } | |
| 15 | 15 | ||
| 16 | - DebugOptions* EnvironmentOptions::get_debug_options() { | ||
| 17 | - return &debug_options_; | ||
| 18 | - } | ||
| 19 | - | ||
| 20 | - const DebugOptions& EnvironmentOptions::debug_options() const { | ||
| 21 | - return debug_options_; | ||
| 22 | - } | ||
| 23 | - | ||
| 24 | 16 | EnvironmentOptions* PerIsolateOptions::get_per_env_options() { | |
| 25 | 17 | return per_env.get(); | |
| 26 | 18 | } | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -172,8 +172,9 @@ class EnvironmentOptions : public Options { | |||
| 172 | 172 | ||
| 173 | 173 | std::vector<std::string> user_argv; | |
| 174 | 174 | ||
| 175 | - inline DebugOptions* get_debug_options(); | ||
| 176 | - inline const DebugOptions& debug_options() const; | ||
| 175 | + inline DebugOptions* get_debug_options() { return &debug_options_; } | ||
| 176 | + inline const DebugOptions& debug_options() const { return debug_options_; } | ||
| 177 | + | ||
| 177 | 178 | void CheckOptions(std::vector<std::string>* errors) override; | |
| 178 | 179 | ||
| 179 | 180 | private: | |
0 commit comments