Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

Commit

Permalink
chakrashim: Added missing API in v8.h
Browse files Browse the repository at this point in the history
PR-URL: #108
Reviewed-By: Sandeep Agarwal <[email protected]>
  • Loading branch information
kunalspathak committed Aug 17, 2016
1 parent 913ea18 commit 45ff9e9
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion deps/chakrashim/include/v8-version.h
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
#define V8_MAJOR_VERSION 5
#define V8_MINOR_VERSION 1
#define V8_BUILD_NUMBER 281
#define V8_PATCH_LEVEL 69
#define V8_PATCH_LEVEL 80

// Use 1 for candidates and 0 otherwise.
// (Boolean macro values are not supported by all preprocessors.)
Expand Down
2 changes: 2 additions & 0 deletions deps/chakrashim/include/v8.h
Original file line number Diff line number Diff line change
Expand Up @@ -1603,6 +1603,8 @@ class PropertyCallbackInfo {
Local<Value> Data() const { return _data; }
Local<Object> This() const { return _thisObject; }
Local<Object> Holder() const { return _holder; }
// CHAKRA-TODO
bool ShouldThrowOnError() const { return true; }
ReturnValue<T> GetReturnValue() const {
return ReturnValue<T>(
&(const_cast<PropertyCallbackInfo<T>*>(this)->_returnValue));
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-repl.js
Original file line number Diff line number Diff line change
Expand Up @@ -389,7 +389,7 @@ function error_test() {
// Avoid emitting stack trace
{ client: client_unix, send: 'a = 3.5e',
expect: /^(?!\s+at\s)/gm },
].filter((v) => !common.engineSpecificMessage(v)));
].filter((v) => !common.engineSpecificMessage(v)));
}

function tcp_test() {
Expand Down

0 comments on commit 45ff9e9

Please sign in to comment.