Skip to content

Commit 91ef427

Browse files
legendecasjuanarbol
legendecas
authored and
juanarbol
committed
src: nest namespace report in namespace node
PR-URL: #44069 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Minwoo Jung <[email protected]>
1 parent 70563b5 commit 91ef427

File tree

4 files changed

+53
-54
lines changed

4 files changed

+53
-54
lines changed

‎src/node_report.cc

+17-26
Original file line numberDiff line numberDiff line change
@@ -27,15 +27,9 @@ constexpr int NODE_REPORT_VERSION = 2;
2727
constexpr int NANOS_PER_SEC = 1000 * 1000 * 1000;
2828
constexpr double SEC_PER_MICROS = 1e-6;
2929

30+
namespace node {
3031
namespace report {
31-
using node::arraysize;
32-
using node::ConditionVariable;
33-
using node::DiagnosticFilename;
34-
using node::Environment;
35-
using node::JSONWriter;
36-
using node::Mutex;
37-
using node::NativeSymbolDebuggingContext;
38-
using node::TIME_TYPE;
32+
3933
using node::worker::Worker;
4034
using v8::Array;
4135
using v8::Context;
@@ -54,8 +48,6 @@ using v8::TryCatch;
5448
using v8::V8;
5549
using v8::Value;
5650

57-
namespace per_process = node::per_process;
58-
5951
// Internal/static function declarations
6052
static void WriteNodeReport(Isolate* isolate,
6153
Environment* env,
@@ -129,7 +121,7 @@ std::string TriggerNodeReport(Isolate* isolate,
129121
// Regular file. Append filename to directory path if one was specified
130122
if (report_directory.length() > 0) {
131123
std::string pathname = report_directory;
132-
pathname += node::kPathSeparator;
124+
pathname += kPathSeparator;
133125
pathname += filename;
134126
outfile.open(pathname, std::ios::out | std::ios::binary);
135127
} else {
@@ -260,9 +252,9 @@ static void WriteNodeReport(Isolate* isolate,
260252
}
261253

262254
// Report out the command line.
263-
if (!node::per_process::cli_options->cmdline.empty()) {
255+
if (!per_process::cli_options->cmdline.empty()) {
264256
writer.json_arraystart("commandLine");
265-
for (const std::string& arg : node::per_process::cli_options->cmdline) {
257+
for (const std::string& arg : per_process::cli_options->cmdline) {
266258
writer.json_element(arg);
267259
}
268260
writer.json_arrayend();
@@ -377,8 +369,8 @@ static void PrintVersionInformation(JSONWriter* writer) {
377369

378370
// Report Process word size
379371
writer->json_keyvalue("wordSize", sizeof(void*) * 8);
380-
writer->json_keyvalue("arch", node::per_process::metadata.arch);
381-
writer->json_keyvalue("platform", node::per_process::metadata.platform);
372+
writer->json_keyvalue("arch", per_process::metadata.arch);
373+
writer->json_keyvalue("platform", per_process::metadata.platform);
382374

383375
// Report deps component versions
384376
PrintComponentVersions(writer);
@@ -528,7 +520,7 @@ static Maybe<std::string> ErrorToString(Isolate* isolate,
528520
maybe_str = error->ToString(context);
529521
} else if (error->IsObject()) {
530522
MaybeLocal<Value> stack = error.As<Object>()->Get(
531-
context, node::FIXED_ONE_BYTE_STRING(isolate, "stack"));
523+
context, FIXED_ONE_BYTE_STRING(isolate, "stack"));
532524
if (!stack.IsEmpty() && stack.ToLocalChecked()->IsString()) {
533525
maybe_str = stack.ToLocalChecked().As<String>();
534526
}
@@ -656,7 +648,7 @@ static void PrintGCStatistics(JSONWriter* writer, Isolate* isolate) {
656648
static void PrintResourceUsage(JSONWriter* writer) {
657649
// Get process uptime in seconds
658650
uint64_t uptime =
659-
(uv_hrtime() - node::per_process::node_start_time) / (NANOS_PER_SEC);
651+
(uv_hrtime() - per_process::node_start_time) / (NANOS_PER_SEC);
660652
if (uptime == 0) uptime = 1; // avoid division by zero.
661653

662654
// Process and current thread usage statistics
@@ -714,7 +706,7 @@ static void PrintSystemInformation(JSONWriter* writer) {
714706
writer->json_objectstart("environmentVariables");
715707

716708
{
717-
Mutex::ScopedLock lock(node::per_process::env_var_mutex);
709+
Mutex::ScopedLock lock(per_process::env_var_mutex);
718710
r = uv_os_environ(&envitems, &envcount);
719711
}
720712

@@ -794,8 +786,7 @@ static void PrintComponentVersions(JSONWriter* writer) {
794786

795787
writer->json_objectstart("componentVersions");
796788

797-
#define V(key) \
798-
writer->json_keyvalue(#key, node::per_process::metadata.versions.key);
789+
#define V(key) writer->json_keyvalue(#key, per_process::metadata.versions.key);
799790
NODE_VERSIONS_KEYS(V)
800791
#undef V
801792

@@ -805,22 +796,22 @@ static void PrintComponentVersions(JSONWriter* writer) {
805796
// Report runtime release information.
806797
static void PrintRelease(JSONWriter* writer) {
807798
writer->json_objectstart("release");
808-
writer->json_keyvalue("name", node::per_process::metadata.release.name);
799+
writer->json_keyvalue("name", per_process::metadata.release.name);
809800
#if NODE_VERSION_IS_LTS
810-
writer->json_keyvalue("lts", node::per_process::metadata.release.lts);
801+
writer->json_keyvalue("lts", per_process::metadata.release.lts);
811802
#endif
812803

813804
#ifdef NODE_HAS_RELEASE_URLS
814805
writer->json_keyvalue("headersUrl",
815-
node::per_process::metadata.release.headers_url);
816-
writer->json_keyvalue("sourceUrl",
817-
node::per_process::metadata.release.source_url);
806+
per_process::metadata.release.headers_url);
807+
writer->json_keyvalue("sourceUrl", per_process::metadata.release.source_url);
818808
#ifdef _WIN32
819-
writer->json_keyvalue("libUrl", node::per_process::metadata.release.lib_url);
809+
writer->json_keyvalue("libUrl", per_process::metadata.release.lib_url);
820810
#endif // _WIN32
821811
#endif // NODE_HAS_RELEASE_URLS
822812

823813
writer->json_objectend();
824814
}
825815

826816
} // namespace report
817+
} // namespace node

‎src/node_report.h

+12-3
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
1-
#pragma once
1+
#ifndef SRC_NODE_REPORT_H_
2+
#define SRC_NODE_REPORT_H_
3+
4+
#if defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS
25

36
#include "node.h"
47
#include "node_buffer.h"
@@ -12,17 +15,18 @@
1215

1316
#include <iomanip>
1417

18+
namespace node {
1519
namespace report {
1620

1721
// Function declarations - functions in src/node_report.cc
1822
std::string TriggerNodeReport(v8::Isolate* isolate,
19-
node::Environment* env,
23+
Environment* env,
2024
const char* message,
2125
const char* trigger,
2226
const std::string& name,
2327
v8::Local<v8::Value> error);
2428
void GetNodeReport(v8::Isolate* isolate,
25-
node::Environment* env,
29+
Environment* env,
2630
const char* message,
2731
const char* trigger,
2832
v8::Local<v8::Value> error,
@@ -45,3 +49,8 @@ void WriteReport(const v8::FunctionCallbackInfo<v8::Value>& info);
4549
void GetReport(const v8::FunctionCallbackInfo<v8::Value>& info);
4650

4751
} // namespace report
52+
} // namespace node
53+
54+
#endif // defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS
55+
56+
#endif // SRC_NODE_REPORT_H_

‎src/node_report_module.cc

+21-20
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
#include <atomic>
1616
#include <sstream>
1717

18+
namespace node {
1819
namespace report {
1920
using node::Environment;
2021
using node::Mutex;
@@ -76,48 +77,48 @@ void GetReport(const FunctionCallbackInfo<Value>& info) {
7677
}
7778

7879
static void GetCompact(const FunctionCallbackInfo<Value>& info) {
79-
node::Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
80-
info.GetReturnValue().Set(node::per_process::cli_options->report_compact);
80+
Mutex::ScopedLock lock(per_process::cli_options_mutex);
81+
info.GetReturnValue().Set(per_process::cli_options->report_compact);
8182
}
8283

8384
static void SetCompact(const FunctionCallbackInfo<Value>& info) {
84-
node::Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
85+
Mutex::ScopedLock lock(per_process::cli_options_mutex);
8586
Environment* env = Environment::GetCurrent(info);
8687
Isolate* isolate = env->isolate();
8788
bool compact = info[0]->ToBoolean(isolate)->Value();
88-
node::per_process::cli_options->report_compact = compact;
89+
per_process::cli_options->report_compact = compact;
8990
}
9091

9192
static void GetDirectory(const FunctionCallbackInfo<Value>& info) {
92-
node::Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
93+
Mutex::ScopedLock lock(per_process::cli_options_mutex);
9394
Environment* env = Environment::GetCurrent(info);
94-
std::string directory = node::per_process::cli_options->report_directory;
95+
std::string directory = per_process::cli_options->report_directory;
9596
auto result = String::NewFromUtf8(env->isolate(), directory.c_str());
9697
info.GetReturnValue().Set(result.ToLocalChecked());
9798
}
9899

99100
static void SetDirectory(const FunctionCallbackInfo<Value>& info) {
100-
node::Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
101+
Mutex::ScopedLock lock(per_process::cli_options_mutex);
101102
Environment* env = Environment::GetCurrent(info);
102103
CHECK(info[0]->IsString());
103104
Utf8Value dir(env->isolate(), info[0].As<String>());
104-
node::per_process::cli_options->report_directory = *dir;
105+
per_process::cli_options->report_directory = *dir;
105106
}
106107

107108
static void GetFilename(const FunctionCallbackInfo<Value>& info) {
108-
node::Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
109+
Mutex::ScopedLock lock(per_process::cli_options_mutex);
109110
Environment* env = Environment::GetCurrent(info);
110-
std::string filename = node::per_process::cli_options->report_filename;
111+
std::string filename = per_process::cli_options->report_filename;
111112
auto result = String::NewFromUtf8(env->isolate(), filename.c_str());
112113
info.GetReturnValue().Set(result.ToLocalChecked());
113114
}
114115

115116
static void SetFilename(const FunctionCallbackInfo<Value>& info) {
116-
node::Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
117+
Mutex::ScopedLock lock(per_process::cli_options_mutex);
117118
Environment* env = Environment::GetCurrent(info);
118119
CHECK(info[0]->IsString());
119120
Utf8Value name(env->isolate(), info[0].As<String>());
120-
node::per_process::cli_options->report_filename = *name;
121+
per_process::cli_options->report_filename = *name;
121122
}
122123

123124
static void GetSignal(const FunctionCallbackInfo<Value>& info) {
@@ -135,15 +136,14 @@ static void SetSignal(const FunctionCallbackInfo<Value>& info) {
135136
}
136137

137138
static void ShouldReportOnFatalError(const FunctionCallbackInfo<Value>& info) {
138-
Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
139-
info.GetReturnValue().Set(
140-
node::per_process::cli_options->report_on_fatalerror);
139+
Mutex::ScopedLock lock(per_process::cli_options_mutex);
140+
info.GetReturnValue().Set(per_process::cli_options->report_on_fatalerror);
141141
}
142142

143143
static void SetReportOnFatalError(const FunctionCallbackInfo<Value>& info) {
144144
CHECK(info[0]->IsBoolean());
145-
Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
146-
node::per_process::cli_options->report_on_fatalerror = info[0]->IsTrue();
145+
Mutex::ScopedLock lock(per_process::cli_options_mutex);
146+
per_process::cli_options->report_on_fatalerror = info[0]->IsTrue();
147147
}
148148

149149
static void ShouldReportOnSignal(const FunctionCallbackInfo<Value>& info) {
@@ -197,7 +197,7 @@ static void Initialize(Local<Object> exports,
197197
SetReportOnUncaughtException);
198198
}
199199

200-
void RegisterExternalReferences(node::ExternalReferenceRegistry* registry) {
200+
void RegisterExternalReferences(ExternalReferenceRegistry* registry) {
201201
registry->Register(WriteReport);
202202
registry->Register(GetReport);
203203
registry->Register(GetCompact);
@@ -217,6 +217,7 @@ void RegisterExternalReferences(node::ExternalReferenceRegistry* registry) {
217217
}
218218

219219
} // namespace report
220+
} // namespace node
220221

221-
NODE_MODULE_CONTEXT_AWARE_INTERNAL(report, report::Initialize)
222-
NODE_MODULE_EXTERNAL_REFERENCE(report, report::RegisterExternalReferences)
222+
NODE_MODULE_CONTEXT_AWARE_INTERNAL(report, node::report::Initialize)
223+
NODE_MODULE_EXTERNAL_REFERENCE(report, node::report::RegisterExternalReferences)

‎src/node_report_utils.cc

+3-5
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,9 @@
33
#include "node_report.h"
44
#include "util-inl.h"
55

6+
namespace node {
67
namespace report {
78

8-
using node::JSONWriter;
9-
using node::MallocedBuffer;
10-
119
static constexpr auto null = JSONWriter::Null{};
1210

1311
// Utility function to format socket information.
@@ -210,8 +208,7 @@ void WalkHandle(uv_handle_t* h, void* arg) {
210208
// SIGWINCH is used by libuv so always appears.
211209
// See http://docs.libuv.org/en/v1.x/signal.html
212210
writer->json_keyvalue("signum", handle->signal.signum);
213-
writer->json_keyvalue("signal",
214-
node::signo_string(handle->signal.signum));
211+
writer->json_keyvalue("signal", signo_string(handle->signal.signum));
215212
break;
216213
default:
217214
break;
@@ -276,3 +273,4 @@ void WalkHandle(uv_handle_t* h, void* arg) {
276273
}
277274

278275
} // namespace report
276+
} // namespace node

0 commit comments

Comments
 (0)