1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

deps: add missing HandleScope in FieldType::PrintTo

Browse Source 
Refs: https://github.com/nodejs/node/issues/22775

PR-URL: https://github.com/nodejs/node/pull/22890
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Refael Ackermann <refack@gmail.com>
This commit is contained in:
Yang Guo 2018-09-17 07:43:46 +02:00
parent fee4d3ab90
commit dafaa6ecb5
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
common.gypi
View File

@ -29,7 +29,7 @@
# Reset this number to 0 on major V8 upgrades. # Reset this number to 0 on major V8 upgrades.
# Increment by one for each non-official patch applied to deps/v8. # Increment by one for each non-official patch applied to deps/v8.
'v8_embedder_string': '-node.11', 'v8_embedder_string': '-node.12',
# Enable disassembler for `--print-code` v8 options # Enable disassembler for `--print-code` v8 options
'v8_enable_disassembler': 1, 'v8_enable_disassembler': 1,

1
deps/v8/src/field-type.cc vendored
View File

@ -78,6 +78,7 @@ void FieldType::PrintTo(std::ostream& os) {
os << "None"; os << "None";
} else { } else {
DCHECK(IsClass()); DCHECK(IsClass());
HandleScope scope(Map::cast(this)->GetIsolate());
os << "Class(" << static_cast<void*>(*AsClass()) << ")"; os << "Class(" << static_cast<void*>(*AsClass()) << ")";
} }
} }