Bug #20817
openRuby 3.4.0dev emits `warning: possibly useless use of + in void context` while Ruby 3.3.5 does not
Description
Ruby 3.4.0dev emits warning: possibly useless use of + in void context
while Ruby 3.3.5 does not
This issue is based on Rails Action Pack test case against Ruby 3.4.0dev.
Here is the minimum script to reproduce it.
Steps to reproduce¶
- Create a sample foo.rb file
class C
class_eval "def throw_syntax_error; eval %(
'abc' + pluralize 'def'
); end", "lib/file.rb", 42
end
c = C.new
c.throw_syntax_error
- Run this foo.rb using Ruby 3.4.0dev
Expected behavior¶
It should report SyntaxError, but it shoud not emit any warnings as Ruby 3.3.5 does not.
$ ruby -v
ruby 3.3.5 (2024-09-03 revision ef084cc8f4) [x86_64-linux]
$ ruby -w foo.rb
lib/file.rb:42:in `eval': (eval at lib/file.rb:42):2: syntax error, unexpected string literal, expecting `do' or '{' or '(' (SyntaxError)
'abc' + pluralize 'def'
^
from lib/file.rb:42:in `throw_syntax_error'
from foo.rb:7:in `<main>'
Actual behavior¶
It reports It should SyntaxError and it emits the warning: possibly useless use of + in void context
$ ruby -v
ruby 3.4.0dev (2024-10-25T11:52:32Z master 9c8c140d73) +PRISM [x86_64-linux]
$ ruby -w foo.rb
(eval at lib/file.rb:42):2: warning: possibly useless use of + in void context
lib/file.rb:42:in 'Kernel#eval': (eval at lib/file.rb:42):2: syntax error found (SyntaxError)
1 |
> 2 | 'abc' + pluralize 'def'
| ^ unexpected string literal, expecting end-of-input
3 |
from lib/file.rb:42:in 'C#throw_syntax_error'
from foo.rb:7:in '<main>'
$
Updated by yahonda (Yasuo Honda) 29 days ago
Here is the original Rails CI log file.
https://buildkite.com/rails/rails-nightly/builds/1214#0192c57b-b994-43cb-9f48-63b1b41e6f4e/1177-1185
"(eval at lib/file.rb:42):2: warning: possibly useless use of + in void context"
How to reproduce this behavior locally¶
git clone https://github.com/rails/rails
cd rails/actionpack
bundle install
bin/test test/dispatch/exception_wrapper_test.rb -n /source_extracts_works_with_eval_syntax_error/
$ ruby -v
ruby 3.4.0dev (2024-10-25T11:52:32Z master 9c8c140d73) +PRISM [x86_64-linux]
$ bin/test test/dispatch/exception_wrapper_test.rb -n /source_extracts_works_with_eval_syntax_error/
/home/yahonda/.rbenv/versions/trunk/lib/ruby/gems/3.4.0+0/gems/minitest-5.25.1/lib/minitest/mock.rb:33: warning: redefining 'object_id' may cause serious problems
/home/yahonda/.rbenv/versions/trunk/lib/ruby/gems/3.4.0+0/gems/capybara-3.40.0/lib/capybara/session/config.rb:95: warning: URI::RFC3986_PARSER.make_regexp is obsoleted. Use URI::RFC2396_PARSER.make_regexp explicitly.
Running 20 tests in a single process (parallelization threshold is 50)
Run options: -n /source_extracts_works_with_eval_syntax_error/ --seed 47290
# Running:
(eval at lib/file.rb:42):2: warning: possibly useless use of + in void context
.
Finished in 0.013903s, 71.9264 runs/s, 143.8528 assertions/s.
1 runs, 2 assertions, 0 failures, 0 errors, 0 skips
$
Updated by yahonda (Yasuo Honda) 29 days ago
Using parse.y parser works as Ruby 3.3.5 does. No warnings emitted.
$ RUBYOPT=--parser=parse.y ruby -v
ruby 3.4.0dev (2024-10-25T11:52:32Z master 9c8c140d73) [x86_64-linux]
$ RUBYOPT=--parser=parse.y ruby -w foo.rb
lib/file.rb:42:in 'Kernel#eval': (eval at lib/file.rb:42):2: syntax error, unexpected string literal, expecting 'do' or '{' or '(' (SyntaxError)
'abc' + pluralize 'def'
^
from lib/file.rb:42:in 'C#throw_syntax_error'
from foo.rb:7:in '<main>'
$
Updated by Earlopain (Earlopain _) 29 days ago
Related to https://bugs.ruby-lang.org/issues/20736
Updated by mame (Yusuke Endoh) 28 days ago
- Related to Bug #20736: prism emits wrong warnings in syntax-error code added
Updated by nobu (Nobuyoshi Nakada) 28 days ago
This seems the case that the recovery works wrongly.
When no syntax error, also the "useless use" warning occurs inside eval
.
Not in eval
, the warning is emitted regardless the syntax error.
Updated by kddnewton (Kevin Newton) 26 days ago
- Assignee set to kddnewton (Kevin Newton)
Updated by kddnewton (Kevin Newton) 18 days ago
- Assignee changed from kddnewton (Kevin Newton) to prism
Updated by hsbt (Hiroshi SHIBATA) 11 days ago
- Status changed from Open to Assigned