Skip to content

Commit 866d81c

Browse files
andrewjdmccannjasnell
authored andcommitted
src: use default initializers over settings fields on the constructor
PR-URL: #23532 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
1 parent ffab8ba commit 866d81c

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

src/spawn_sync.cc

+1-3
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,7 @@ using v8::String;
5050
using v8::Value;
5151

5252

53-
SyncProcessOutputBuffer::SyncProcessOutputBuffer()
54-
: used_(0),
55-
next_(nullptr) {
53+
SyncProcessOutputBuffer::SyncProcessOutputBuffer() {
5654
}
5755

5856

src/spawn_sync.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,9 @@ class SyncProcessOutputBuffer {
5656
private:
5757
// Use unsigned int because that's what `uv_buf_init` takes.
5858
mutable char data_[kBufferSize];
59-
unsigned int used_;
59+
unsigned int used_ = 0;
6060

61-
SyncProcessOutputBuffer* next_;
61+
SyncProcessOutputBuffer* next_ = nullptr;
6262
};
6363

6464

0 commit comments

Comments
 (0)