You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am seeing parser errors on Ruby lambdas. Here is a test case:
$ cat ruby-parser-errors.rb
def foo
puts "hi from foo"
end
foo() { boo( & lambda {}) }
foo() { boo( & ->{}) }
$
$ ruby -v
ruby 3.0.3p157 (2021-11-24 revision 3fb7d2cadc) [x86_64-linux-gnu]
$
$ ruby ruby-parser-errors.rb
hi from foo
hi from foo
$
$ ruby -c ruby-parser-errors.rb
Syntax OK
$
$ codeql database create test-db --language=ruby --overwrite
...
[2022-05-24 20:40:17] [build-stdout] ERROR ruby-parser-errors.rb:5: parse error
[2022-05-24 20:40:17] [build-stdout] ERROR ruby-parser-errors.rb:6: parse error: expecting 'identifier'
[2022-05-24 20:40:17] [build-stdout] ERROR ruby-parser-errors.rb:6: missing value for field: binary::left
Finalizing database at .../test-db.
Successfully created database at .../test-db.
Please let me know if you need further information - Thank you
The cause of the problem is that the scanner does not allow a space after a "block ampersand". I can make the examples work by removing the space after the &. The following parse fine:
foo() { boo( &lambda {}) }
foo() { boo( &->{}) }
Removing the scan check in the scanner, however, causes other problems as it is intended to disambiguate between "bitwise-and" and "block argument" expressions. For example:
foo & bar # a bitwise and of `foo` and `bar`
foo &bar # a call to `foo` with block argument `bar`
The text was updated successfully, but these errors were encountered:
I suppose we can fix this by changing the scanner to allow a space after a "block_ampersand" in cases where bitwise-and operators are not expected. Something like !(valid_symbols[BITWISE_AND] && iswspace(lexer->lookahead)) could perhaps work.
Originally reported by @grajagandev at github/codeql#9313
The cause of the problem is that the scanner does not allow a space after a "block ampersand". I can make the examples work by removing the space after the
&
. The following parse fine:Removing the scan check in the scanner, however, causes other problems as it is intended to disambiguate between "bitwise-and" and "block argument" expressions. For example:
The text was updated successfully, but these errors were encountered: