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/node_options.h | |
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/node_options.h')
-rw-r--r-- | src/node_options.h | 2 |
1 files changed, 1 insertions, 1 deletions
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 |