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

Bugfix for string replace. (#1409)

Browse Source 
Some test cases were failed with enabled show-opcodes build option.

Example source:
    "98765".replace(76)
    print ("\ud801\udc00".replace("\ud801", "#") === "#\udc00");

JerryScript-DCO-1.0-Signed-off-by: Robert Sipka rsipka.uszeged@partner.samsung.com
This commit is contained in:
Robert Sipka
2016-10-27 12:47:54 +02:00
committed by GitHub
parent a30b89c536
commit a02c586a51
1 changed files with 52 additions and 43 deletions
Download Patch File Download Diff File Expand all files Collapse all files
jerry-core/ecma/builtin-objects/ecma-builtin-string-prototype.c
+15 -6
View File
@@ -875,8 +875,19 @@ ecma_builtin_string_prototype_object_replace_get_string (ecma_builtin_replace_se
{
ecma_char_t action = LIT_CHAR_NULL;
if (*replace_str_curr_p++ == LIT_CHAR_DOLLAR_SIGN)
if (*replace_str_curr_p != LIT_CHAR_DOLLAR_SIGN)
{
/* if not a continuation byte */
if ((*replace_str_curr_p & LIT_UTF8_EXTRA_BYTE_MASK) != LIT_UTF8_EXTRA_BYTE_MARKER)
{
current_position++;
}
replace_str_curr_p++;
continue;
}
replace_str_curr_p++;
if (replace_str_curr_p < replace_str_end_p)
{
action = *replace_str_curr_p;
@@ -925,7 +936,6 @@ ecma_builtin_string_prototype_object_replace_get_string (ecma_builtin_replace_se
action = LIT_CHAR_NULL;
}
}
}
if (action != LIT_CHAR_NULL)
{
@@ -935,6 +945,7 @@ ecma_builtin_string_prototype_object_replace_get_string (ecma_builtin_replace_se
current_position,
true);
replace_str_curr_p++;
current_position++;
if (action == LIT_CHAR_DOLLAR_SIGN)
{
@@ -1015,11 +1026,9 @@ ecma_builtin_string_prototype_object_replace_get_string (ecma_builtin_replace_se
}
current_position++;
previous_start = current_position + 1;
previous_start = current_position;
}
/* if not a continuation byte */
if ((*replace_str_curr_p & LIT_UTF8_EXTRA_BYTE_MASK) != LIT_UTF8_EXTRA_BYTE_MARKER)
else
{
current_position++;
}