Skip to content
Browse files

src: add missing option parser template for the DebugOptionsParser

This allows embedders to run `node::options_parser::Parse` for a

PR-URL: #28543
Reviewed-By: Anna Henningsen <>
Reviewed-By: Joyee Cheung <>
Reviewed-By: Eugene Ostroukhov <>
Reviewed-By: Rich Trott <>
Reviewed-By: Colin Ihrig <>
  • Loading branch information...
MarshallOfSound authored and targos committed Jul 4, 2019
1 parent b4aa7d3 commit 1f0fd1bb789362a68e58cc5d2e9c0abb2658d2c5
Showing with 11 additions and 0 deletions.
  1. +11 −0 src/
@@ -226,6 +226,17 @@ class PerProcessOptionsParser : public OptionsParser<PerProcessOptions> {
const DebugOptionsParser _dop_instance{};
const EnvironmentOptionsParser _eop_instance{_dop_instance};

// This Parse is not dead code. It is used by embedders (e.g., Electron).
template <>
void Parse(
StringVector* const args, StringVector* const exec_args,
StringVector* const v8_args,
DebugOptions* const options,
OptionEnvvarSettings required_env_settings, StringVector* const errors) {
args, exec_args, v8_args, options, required_env_settings, errors);
const EnvironmentOptionsParser _eop_instance{};

0 comments on commit 1f0fd1b

Please sign in to comment.
You can’t perform that action at this time.