Skip to content

Commit 3dc4a5f

Browse files
santigimenoitaloacasas
authored andcommitted
tracing: fix -Wunused-private-field warning
Remove `external_buffer_` from `InternalTraceBuffer` as it seems not to be used anywhere. PR-URL: #10416 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent 8a918bf commit 3dc4a5f

File tree

2 files changed

+5
-8
lines changed

2 files changed

+5
-8
lines changed

src/tracing/node_trace_buffer.cc

+4-5
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,9 @@ namespace node {
44
namespace tracing {
55

66
InternalTraceBuffer::InternalTraceBuffer(size_t max_chunks, uint32_t id,
7-
NodeTraceWriter* trace_writer, NodeTraceBuffer* external_buffer)
7+
NodeTraceWriter* trace_writer)
88
: flushing_(false), max_chunks_(max_chunks),
9-
trace_writer_(trace_writer), external_buffer_(external_buffer),
10-
id_(id) {
9+
trace_writer_(trace_writer), id_(id) {
1110
chunks_.resize(max_chunks);
1211
}
1312

@@ -90,8 +89,8 @@ void InternalTraceBuffer::ExtractHandle(
9089
NodeTraceBuffer::NodeTraceBuffer(size_t max_chunks,
9190
NodeTraceWriter* trace_writer, uv_loop_t* tracing_loop)
9291
: tracing_loop_(tracing_loop), trace_writer_(trace_writer),
93-
buffer1_(max_chunks, 0, trace_writer, this),
94-
buffer2_(max_chunks, 1, trace_writer, this) {
92+
buffer1_(max_chunks, 0, trace_writer),
93+
buffer2_(max_chunks, 1, trace_writer) {
9594
current_buf_.store(&buffer1_);
9695

9796
flush_signal_.data = this;

src/tracing/node_trace_buffer.h

+1-3
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,7 @@ class NodeTraceBuffer;
2020
class InternalTraceBuffer {
2121
public:
2222
InternalTraceBuffer(size_t max_chunks, uint32_t id,
23-
NodeTraceWriter* trace_writer,
24-
NodeTraceBuffer* external_buffer);
23+
NodeTraceWriter* trace_writer);
2524

2625
TraceObject* AddTraceEvent(uint64_t* handle);
2726
TraceObject* GetEventByHandle(uint64_t handle);
@@ -44,7 +43,6 @@ class InternalTraceBuffer {
4443
bool flushing_;
4544
size_t max_chunks_;
4645
NodeTraceWriter* trace_writer_;
47-
NodeTraceBuffer* external_buffer_;
4846
std::vector<std::unique_ptr<TraceBufferChunk>> chunks_;
4947
size_t total_chunks_ = 0;
5048
uint32_t current_chunk_seq_ = 1;

0 commit comments

Comments
 (0)