diff options
author | Refael Ackermann <refack@gmail.com> | 2019-02-25 15:42:28 -0500 |
---|---|---|
committer | Refael Ackermann <refack@gmail.com> | 2019-03-03 20:03:31 -0500 |
commit | 2c6d94f3db89ed4dfa3e68e7e945c10e3662ea41 (patch) | |
tree | 2ef84cb998275f475d1812f94ca9a59cbeacd583 /src | |
parent | af8b92c0731603e5337cca3202603402d345c0a9 (diff) | |
download | android-node-v8-2c6d94f3db89ed4dfa3e68e7e945c10e3662ea41.tar.gz android-node-v8-2c6d94f3db89ed4dfa3e68e7e945c10e3662ea41.tar.bz2 android-node-v8-2c6d94f3db89ed4dfa3e68e7e945c10e3662ea41.zip |
src: fix warnings around node_options
* header explicit usage, order, and reduce use of `*-inl.h`
* pointer -> const reference when possible
* no variable recyclicng
* `std::begin/end` prefered over `instance.begin/end`
* `USE` for explicit unused resaults
PR-URL: https://github.com/nodejs/node/pull/26280
Fixes: https://github.com/nodejs/node/issues/25593
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/node_config.cc | 2 | ||||
-rw-r--r-- | src/node_options-inl.h | 10 | ||||
-rw-r--r-- | src/node_options.cc | 12 | ||||
-rw-r--r-- | src/node_options.h | 2 | ||||
-rw-r--r-- | src/node_process_object.cc | 4 | ||||
-rw-r--r-- | src/node_worker.cc | 4 |
6 files changed, 18 insertions, 16 deletions
diff --git a/src/node_config.cc b/src/node_config.cc index f1aa748bf5..b89a668419 100644 --- a/src/node_config.cc +++ b/src/node_config.cc @@ -1,7 +1,7 @@ #include "env-inl.h" #include "node.h" #include "node_i18n.h" -#include "node_options-inl.h" +#include "node_options.h" #include "util-inl.h" namespace node { diff --git a/src/node_options-inl.h b/src/node_options-inl.h index 052c847f7e..001bdecbb6 100644 --- a/src/node_options-inl.h +++ b/src/node_options-inl.h @@ -213,15 +213,15 @@ auto OptionsParser<Options>::Convert( template <typename Options> template <typename ChildOptions> void OptionsParser<Options>::Insert( - const OptionsParser<ChildOptions>* child_options_parser, + const OptionsParser<ChildOptions>& child_options_parser, ChildOptions* (Options::* get_child)()) { - aliases_.insert(child_options_parser->aliases_.begin(), - child_options_parser->aliases_.end()); + aliases_.insert(std::begin(child_options_parser.aliases_), + std::end(child_options_parser.aliases_)); - for (const auto& pair : child_options_parser->options_) + for (const auto& pair : child_options_parser.options_) options_.emplace(pair.first, Convert(pair.second, get_child)); - for (const auto& pair : child_options_parser->implications_) + for (const auto& pair : child_options_parser.implications_) implications_.emplace(pair.first, Convert(pair.second, get_child)); } diff --git a/src/node_options.cc b/src/node_options.cc index 42304fc772..0a8ac91c34 100644 --- a/src/node_options.cc +++ b/src/node_options.cc @@ -1,8 +1,9 @@ -#include <cerrno> #include "env-inl.h" #include "node_binding.h" #include "node_options-inl.h" +#include <cstdlib> // strtoul, errno + using v8::Boolean; using v8::Context; using v8::FunctionCallbackInfo; @@ -123,7 +124,7 @@ class EnvironmentOptionsParser : public OptionsParser<EnvironmentOptions> { EnvironmentOptionsParser(); explicit EnvironmentOptionsParser(const DebugOptionsParser& dop) : EnvironmentOptionsParser() { - Insert(&dop, &EnvironmentOptions::get_debug_options); + Insert(dop, &EnvironmentOptions::get_debug_options); } }; @@ -391,7 +392,7 @@ PerIsolateOptionsParser::PerIsolateOptionsParser( kAllowedInEnvironment); #endif // NODE_REPORT - Insert(&eop, &PerIsolateOptions::get_per_env_options); + Insert(eop, &PerIsolateOptions::get_per_env_options); } PerProcessOptionsParser::PerProcessOptionsParser( @@ -501,7 +502,7 @@ PerProcessOptionsParser::PerProcessOptionsParser( #endif #endif - Insert(&iop, &PerProcessOptions::get_per_isolate_options); + Insert(iop, &PerProcessOptions::get_per_isolate_options); } inline std::string RemoveBrackets(const std::string& host) { @@ -515,7 +516,8 @@ inline int ParseAndValidatePort(const std::string& port, std::vector<std::string>* errors) { char* endptr; errno = 0; - const long result = strtol(port.c_str(), &endptr, 10); // NOLINT(runtime/int) + const unsigned long result = // NOLINT(runtime/int) + strtoul(port.c_str(), &endptr, 10); if (errno != 0 || *endptr != '\0'|| (result != 0 && result < 1024) || result > 65535) { errors->push_back(" must be 0 or in range 1024 to 65535."); diff --git a/src/node_options.h b/src/node_options.h index 02e1e84e50..b5e614d413 100644 --- a/src/node_options.h +++ b/src/node_options.h @@ -290,7 +290,7 @@ class OptionsParser { // a method that yields the target options type from this parser's options // type. template <typename ChildOptions> - void Insert(const OptionsParser<ChildOptions>* child_options_parser, + void Insert(const OptionsParser<ChildOptions>& child_options_parser, ChildOptions* (Options::* get_child)()); // Parse a sequence of options into an options struct, a list of diff --git a/src/node_process_object.cc b/src/node_process_object.cc index cbec233c27..1b0aad3029 100644 --- a/src/node_process_object.cc +++ b/src/node_process_object.cc @@ -1,5 +1,3 @@ -#include <climits> // PATH_MAX - #include "env-inl.h" #include "node_internals.h" #include "node_options-inl.h" @@ -8,6 +6,8 @@ #include "node_revert.h" #include "util-inl.h" +#include <climits> // PATH_MAX + namespace node { using v8::Context; using v8::DEFAULT; diff --git a/src/node_worker.cc b/src/node_worker.cc index e22503863d..5ab4fad5d4 100644 --- a/src/node_worker.cc +++ b/src/node_worker.cc @@ -487,13 +487,13 @@ void Worker::New(const FunctionCallbackInfo<Value>& args) { // The first argument is program name. invalid_args.erase(invalid_args.begin()); if (errors.size() > 0 || invalid_args.size() > 0) { - v8::Local<v8::Value> value = + v8::Local<v8::Value> error = ToV8Value(env->context(), errors.size() > 0 ? errors : invalid_args) .ToLocalChecked(); Local<String> key = FIXED_ONE_BYTE_STRING(env->isolate(), "invalidExecArgv"); - args.This()->Set(env->context(), key, value).FromJust(); + USE(args.This()->Set(env->context(), key, error).FromJust()); return; } } |