Skip to content

Commit

Permalink
src: guard default_inspector_port
Browse files Browse the repository at this point in the history
When configuring and building --without-ssl the following warning is
reported:
../src/node_debug_options.cc:12:11: warning: unused variable
'default_inspector_port' [-Wunused-const-variable]
const int default_inspector_port = 9229;

The commit adds a HAVE_INSPECTOR guard to this constant.

PR-URL: nodejs#12303
Reviewed-By: cjihrig - Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
danbev committed Apr 12, 2017
1 parent a235ccd commit f9bc835
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/node_debug_options.cc
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,9 @@ namespace node {

namespace {
const int default_debugger_port = 5858;
#if HAVE_INSPECTOR
const int default_inspector_port = 9229;
#endif // HAVE_INSPECTOR

inline std::string remove_brackets(const std::string& host) {
if (!host.empty() && host.front() == '[' && host.back() == ']')
Expand Down

0 comments on commit f9bc835

Please sign in to comment.