From e15020be96f8dc87a71504f1cebd99e7509629fa Mon Sep 17 00:00:00 2001 From: tkeri Date: Mon, 20 Nov 2017 15:41:11 +0100 Subject: [PATCH] Fix api inconsistency for promises (#2113) The change makes the jerry_value_is_promise api method available in all cases, just like other promise api methods JerryScript-DCO-1.0-Signed-off-by: Tamas Keri tkeri@inf.u-szeged.hu --- jerry-core/api/jerry.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/jerry-core/api/jerry.c b/jerry-core/api/jerry.c index be6c961f1..45116c121 100644 --- a/jerry-core/api/jerry.c +++ b/jerry-core/api/jerry.c @@ -715,8 +715,6 @@ jerry_value_is_object (const jerry_value_t value) /**< api value */ return ecma_is_value_object (jerry_get_arg_value (value)); } /* jerry_value_is_object */ -#ifndef CONFIG_DISABLE_ES2015_PROMISE_BUILTIN - /** * Check if the specified value is promise. * @@ -727,13 +725,15 @@ bool jerry_value_is_promise (const jerry_value_t value) /**< api value */ { jerry_assert_api_available (); - +#ifndef CONFIG_DISABLE_ES2015_PROMISE_BUILTIN jerry_value_t promise = jerry_get_arg_value (value); return (ecma_is_value_object (promise) && ecma_is_promise (ecma_get_object_from_value (promise))); -} /* jerry_value_is_promise */ - +#else /* CONFIG_DISABLE_ES2015_PROMISE_BUILTIN */ + JERRY_UNUSED (value); + return false; #endif /* !CONFIG_DISABLE_ES2015_PROMISE_BUILTIN */ +} /* jerry_value_is_promise */ /** * Check if the specified value is string.