Skip to content

Commit 2089814

Browse files
mhdawsonMylesBorins
authored andcommitted
test: properly tag anonymous namespaces
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 <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent a667ac1 commit 2089814

File tree

8 files changed

+8
-8
lines changed

8 files changed

+8
-8
lines changed

test/addons-napi/test_make_callback/binding.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,6 @@ napi_value Init(napi_env env, napi_value exports) {
5454
return exports;
5555
}
5656

57-
} // namespace
57+
} // anonymous namespace
5858

5959
NAPI_MODULE(NODE_GYP_MODULE_NAME, Init)

test/addons-napi/test_make_callback_recurse/binding.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,6 @@ napi_value Init(napi_env env, napi_value exports) {
2828
return exports;
2929
}
3030

31-
} // namespace
31+
} // anonymous namespace
3232

3333
NAPI_MODULE(NODE_GYP_MODULE_NAME, Init)

test/addons/async-hooks-id/binding.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,6 @@ void Initialize(Local<Object> exports) {
2222
NODE_SET_METHOD(exports, "getTriggerAsyncId", GetTriggerAsyncId);
2323
}
2424

25-
} // namespace
25+
} // anonymous namespace
2626

2727
NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)

test/addons/async-resource/binding.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,6 @@ void Initialize(Local<Object> exports) {
109109
NODE_SET_METHOD(exports, "getResource", GetResource);
110110
}
111111

112-
} // namespace
112+
} // anonymous namespace
113113

114114
NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)

test/addons/callback-scope/binding.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,6 @@ void Initialize(v8::Local<v8::Object> exports) {
6969
NODE_SET_METHOD(exports, "testResolveAsync", TestResolveAsync);
7070
}
7171

72-
} // namespace
72+
} // anonymous namespace
7373

7474
NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)

test/addons/make-callback-recurse/binding.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,6 @@ void Initialize(Local<Object> exports) {
2626
NODE_SET_METHOD(exports, "makeCallback", MakeCallback);
2727
}
2828

29-
} // namespace
29+
} // anonymous namespace
3030

3131
NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)

test/addons/make-callback/binding.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,6 @@ void Initialize(v8::Local<v8::Object> exports) {
3434
NODE_SET_METHOD(exports, "makeCallback", MakeCallback);
3535
}
3636

37-
} // namespace
37+
} // anonymous namespace
3838

3939
NODE_MODULE(NODE_GYP_MODULE_NAME, Initialize)

test/cctest/test_inspector_socket_server.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -398,7 +398,7 @@ static const std::string WsHandshakeRequest(const std::string& target_id) {
398398
"Sec-WebSocket-Key: aaa==\r\n"
399399
"Sec-WebSocket-Version: 13\r\n\r\n";
400400
}
401-
} // namespace
401+
} // anonymous namespace
402402

403403

404404
TEST_F(InspectorSocketServerTest, InspectorSessions) {

0 commit comments

Comments
 (0)