Skip to content

Commit

Permalink
test: properly tag anonymous namespaces
Browse files Browse the repository at this point in the history
For tests that use anonymous namespaces, some tagged the close
of the namespace with 'namespace' while others used
'anonymous namespace'. It was suggested I should use
'anonymous namespace' in a recent PR review so make all of the
tests consistent with this.

PR-URL: #18583
Reviewed-By: Eugene Ostroukhov <eostroukhov@google.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
mhdawson committed Feb 8, 2018
1 parent 6963a93 commit fe442f6
Show file tree
Hide file tree
Showing 8 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion test/addons-napi/test_make_callback/binding.cc
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,6 @@ napi_value Init(napi_env env, napi_value exports) {
return exports;
}

} // namespace
} // anonymous namespace

NAPI_MODULE(NODE_GYP_MODULE_NAME, Init)
2 changes: 1 addition & 1 deletion test/addons-napi/test_make_callback_recurse/binding.cc
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,6 @@ napi_value Init(napi_env env, napi_value exports) {
return exports;
}

} // namespace
} // anonymous namespace

NAPI_MODULE(NODE_GYP_MODULE_NAME, Init)
2 changes: 1 addition & 1 deletion test/addons/async-hooks-id/binding.cc
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,6 @@ void Initialize(Local<Object> exports) {
NODE_SET_METHOD(exports, "getTriggerAsyncId", GetTriggerAsyncId);
}

} // namespace
} // anonymous namespace

NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)
2 changes: 1 addition & 1 deletion test/addons/async-resource/binding.cc
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,6 @@ void Initialize(Local<Object> exports) {
NODE_SET_METHOD(exports, "getResource", GetResource);
}

} // namespace
} // anonymous namespace

NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)
2 changes: 1 addition & 1 deletion test/addons/callback-scope/binding.cc
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,6 @@ void Initialize(v8::Local<v8::Object> exports) {
NODE_SET_METHOD(exports, "testResolveAsync", TestResolveAsync);
}

} // namespace
} // anonymous namespace

NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)
2 changes: 1 addition & 1 deletion test/addons/make-callback-recurse/binding.cc
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,6 @@ void Initialize(Local<Object> exports) {
NODE_SET_METHOD(exports, "makeCallback", MakeCallback);
}

} // namespace
} // anonymous namespace

NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)
2 changes: 1 addition & 1 deletion test/addons/make-callback/binding.cc
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,6 @@ void Initialize(v8::Local<v8::Object> exports) {
NODE_SET_METHOD(exports, "makeCallback", MakeCallback);
}

} // namespace
} // anonymous namespace

NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)
2 changes: 1 addition & 1 deletion test/cctest/test_inspector_socket_server.cc
Original file line number Diff line number Diff line change
Expand Up @@ -398,7 +398,7 @@ static const std::string WsHandshakeRequest(const std::string& target_id) {
"Sec-WebSocket-Key: aaa==\r\n"
"Sec-WebSocket-Version: 13\r\n\r\n";
}
} // namespace
} // anonymous namespace


TEST_F(InspectorSocketServerTest, InspectorSessions) {
Expand Down

0 comments on commit fe442f6

Please sign in to comment.