Skip to content

Commit 3766e04

Browse files
cjihrigMylesBorins
authored andcommitted
src: silence http2 -Wunused-result warnings
Backport-PR-URL: #18050 Backport-PR-URL: #20456 PR-URL: #17954 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
1 parent 35ee894 commit 3766e04

File tree

1 file changed

+11
-7
lines changed

1 file changed

+11
-7
lines changed

src/node_http2.cc

+11-7
Original file line numberDiff line numberDiff line change
@@ -560,13 +560,13 @@ inline void Http2Stream::EmitStatistics() {
560560
FIXED_ONE_BYTE_STRING(env->isolate(), "timeToFirstByte"),
561561
Number::New(env->isolate(),
562562
(entry->first_byte() - entry->startTimeNano()) / 1e6),
563-
attr);
563+
attr).FromJust();
564564
obj->DefineOwnProperty(
565565
context,
566566
FIXED_ONE_BYTE_STRING(env->isolate(), "timeToFirstHeader"),
567567
Number::New(env->isolate(),
568568
(entry->first_header() - entry->startTimeNano()) / 1e6),
569-
attr);
569+
attr).FromJust();
570570
entry->Notify(obj);
571571
}
572572
delete entry;
@@ -592,25 +592,29 @@ inline void Http2Session::EmitStatistics() {
592592
String::NewFromUtf8(env->isolate(),
593593
entry->typeName(),
594594
v8::NewStringType::kInternalized)
595-
.ToLocalChecked(), attr);
595+
.ToLocalChecked(), attr).FromJust();
596596
if (entry->ping_rtt() != 0) {
597597
obj->DefineOwnProperty(
598598
context,
599599
FIXED_ONE_BYTE_STRING(env->isolate(), "pingRTT"),
600-
Number::New(env->isolate(), entry->ping_rtt() / 1e6), attr);
600+
Number::New(env->isolate(), entry->ping_rtt() / 1e6),
601+
attr).FromJust();
601602
}
602603
obj->DefineOwnProperty(
603604
context,
604605
FIXED_ONE_BYTE_STRING(env->isolate(), "framesReceived"),
605-
Integer::NewFromUnsigned(env->isolate(), entry->frame_count()), attr);
606+
Integer::NewFromUnsigned(env->isolate(), entry->frame_count()),
607+
attr).FromJust();
606608
obj->DefineOwnProperty(
607609
context,
608610
FIXED_ONE_BYTE_STRING(env->isolate(), "streamCount"),
609-
Integer::New(env->isolate(), entry->stream_count()), attr);
611+
Integer::New(env->isolate(), entry->stream_count()),
612+
attr).FromJust();
610613
obj->DefineOwnProperty(
611614
context,
612615
FIXED_ONE_BYTE_STRING(env->isolate(), "streamAverageDuration"),
613-
Number::New(env->isolate(), entry->stream_average_duration()), attr);
616+
Number::New(env->isolate(), entry->stream_average_duration()),
617+
attr).FromJust();
614618
entry->Notify(obj);
615619
}
616620
delete entry;

0 commit comments

Comments
 (0)