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

[ruby/yarp] Fix invalid unescape sequence error for lower w

Browse Source 
https://github.com/ruby/yarp/commit/5a2c78367a
This commit is contained in:
Jemma Issroff 2023-06-21 12:49:41 -04:00 committed by git
parent fcf2f5a522
commit b0a3a6ade0
3 changed files with 17 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/yarp/fixtures/arrays.txt
View File

@ -78,3 +78,5 @@ foo[bar] = baz
%w{one two three} %w{one two three}
%x{one two three} %x{one two three}
%w[\C:]

9
test/yarp/snapshots/arrays.txt
View File

@ -1,6 +1,6 @@
ProgramNode(0...502)( ProgramNode(0...511)(
[], [],
StatementsNode(0...502)( StatementsNode(0...511)(
[ArrayNode(0...4)( [ArrayNode(0...4)(
[SplatNode(1...3)( [SplatNode(1...3)(
(1...2), (1...2),
@ -691,6 +691,11 @@ ProgramNode(0...502)(
(488...501), (488...501),
(501...502), (501...502),
"one two three" "one two three"
),
ArrayNode(504...511)(
[StringNode(507...510)(nil, (507...510), nil, "\\C:")],
(504...507),
(510...511)
)] )]
) )
) )

9
yarp/yarp.c
View File

@ -6477,7 +6477,14 @@ parser_lex(yp_parser_t *parser) {
// If we hit escapes, then we need to treat the next token // If we hit escapes, then we need to treat the next token
// literally. In this case we'll skip past the next character and // literally. In this case we'll skip past the next character and
// find the next breakpoint. // find the next breakpoint.
size_t difference = yp_unescape_calculate_difference(breakpoint, parser->end, YP_UNESCAPE_ALL, false, &parser->error_list);
yp_unescape_type_t unescape_type;
if (parser->lex_modes.current->as.list.interpolation) {
unescape_type = YP_UNESCAPE_ALL;
} else {
unescape_type = YP_UNESCAPE_MINIMAL;
}
size_t difference = yp_unescape_calculate_difference(breakpoint, parser->end, unescape_type, false, &parser->error_list);
// If the result is an escaped newline, then we need to // If the result is an escaped newline, then we need to
// track that newline. // track that newline.