YourWishes/jerryscript
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove asserts from ecma_number_is_zero and is_inf (#3466)

Browse Source 
There is no reason to force the developer to call ecma_number_is_nan
check before all ecma_number_is_zero and ecma_number_is_infinity.
These functions work fine and return false if NaN is passed.

JerryScript-DCO-1.0-Signed-off-by: Csaba Osztrogonác oszi@inf.u-szeged.hu
This commit is contained in:
Csaba Osztrogonác
2019-12-19 15:36:56 +01:00
committed by Robert Fancsik
parent d27a07edf4
commit 456fb046ba
1 changed files with 0 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
jerry-core/ecma/base/ecma-helpers-number.c
-4
View File
@@ -298,8 +298,6 @@ ecma_number_is_negative (ecma_number_t num) /**< ecma-number */
bool bool
ecma_number_is_zero (ecma_number_t num) /**< ecma-number */ ecma_number_is_zero (ecma_number_t num) /**< ecma-number */
{ {
JERRY_ASSERT (!ecma_number_is_nan (num));
bool is_zero = (num == ECMA_NUMBER_ZERO); bool is_zero = (num == ECMA_NUMBER_ZERO);
#ifndef JERRY_NDEBUG #ifndef JERRY_NDEBUG
@@ -323,8 +321,6 @@ ecma_number_is_zero (ecma_number_t num) /**< ecma-number */
bool bool
ecma_number_is_infinity (ecma_number_t num) /**< ecma-number */ ecma_number_is_infinity (ecma_number_t num) /**< ecma-number */
{ {
JERRY_ASSERT (!ecma_number_is_nan (num));
uint32_t biased_exp = ecma_number_get_biased_exponent_field (num); uint32_t biased_exp = ecma_number_get_biased_exponent_field (num);
uint64_t fraction = ecma_number_get_fraction_field (num); uint64_t fraction = ecma_number_get_fraction_field (num);