[PRISM] Do not warn ambiguous ampersand when symbol literal

Fixes [Bug #20735]
This commit is contained in:
Kevin Newton 2024-09-13 12:47:33 -04:00
Родитель 9afc6a981d
Коммит 77521afac1
2 изменённых файлов: 19 добавлений и 1 удалений

Просмотреть файл

@ -11277,7 +11277,16 @@ parser_lex(pm_parser_t *parser) {
pm_token_type_t type = PM_TOKEN_AMPERSAND;
if (lex_state_spcarg_p(parser, space_seen)) {
pm_parser_warn_token(parser, &parser->current, PM_WARN_AMBIGUOUS_PREFIX_AMPERSAND);
if ((peek(parser) != ':') || (peek_offset(parser, 1) == '\0')) {
pm_parser_warn_token(parser, &parser->current, PM_WARN_AMBIGUOUS_PREFIX_AMPERSAND);
} else {
const uint8_t delim = peek_offset(parser, 1);
if ((delim != '\'') && (delim != '"') && !char_is_identifier(parser, parser->current.end + 1)) {
pm_parser_warn_token(parser, &parser->current, PM_WARN_AMBIGUOUS_PREFIX_AMPERSAND);
}
}
type = PM_TOKEN_UAMPERSAND;
} else if (lex_state_beg_p(parser)) {
type = PM_TOKEN_UAMPERSAND;

Просмотреть файл

@ -22,6 +22,15 @@ module Prism
assert_warning("a /b/", "wrap regexp in parentheses")
end
def test_ambiguous_ampersand
assert_warning("a &b", "argument prefix")
assert_warning("a &:+", "argument prefix")
refute_warning("a &:b")
refute_warning("a &:'b'")
refute_warning("a &:\"b\"")
end
def test_binary_operator
[
[:**, "argument prefix"],