Skip to content

Commit

Permalink
src: correct json writer placement in process.report
Browse files Browse the repository at this point in the history
PR-URL: nodejs#28433
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information
himself65 authored and Trott committed Jul 6, 2019
1 parent 6aafee1 commit 7fa9abf
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/node_report.cc
Original file line number Diff line number Diff line change
Expand Up @@ -366,7 +366,6 @@ static void PrintJavaScriptStack(JSONWriter* writer,
int line = ss.find('\n');
if (line == -1) {
writer->json_keyvalue("message", ss);
writer->json_objectend();
} else {
std::string l = ss.substr(0, line);
writer->json_keyvalue("message", l);
Expand All @@ -382,8 +381,8 @@ static void PrintJavaScriptStack(JSONWriter* writer,
ss = ss.substr(line + 1);
line = ss.find('\n');
}
writer->json_arrayend();
}
writer->json_arrayend();
writer->json_objectend();
}

Expand Down

0 comments on commit 7fa9abf

Please sign in to comment.