Remove unnecessary code from the exit command's implementation
The parameters of IRB.irb_exit were never used. But there are some
libraries seem to call it with arguments + it's declared on the top-level
IRB constant. So I changed the params to anonymous splat instead of removing them.
Context#exit was completely unnecessary as IRB.irb_exit doesn't use
the @irb instance it passes. And since it's (or should be treated as)
a private method, I simply removed it.
The exit command doesn't use the status argument it receives at all.
But to avoid raising errors on usages like exit 1, I changed the argument to
anonymous splat instead removing it.
Added by st0012 (Stan Lo) over 1 year ago
[ruby/irb] Refactor exit command
(https://github.com/ruby/irb/pull/835)
IRB.irb_exitwere never used. But there are somelibraries seem to call it with arguments + it's declared on the top-level
IRB constant. So I changed the params to anonymous splat instead of removing them.
Context#exitwas completely unnecessary asIRB.irb_exitdoesn't usethe
@irbinstance it passes. And since it's (or should be treated as)a private method, I simply removed it.
exitcommand doesn't use the status argument it receives at all.But to avoid raising errors on usages like
exit 1, I changed the argument toanonymous splat instead removing it.
Make exit an actual command
Update readme
https://github.com/ruby/irb/commit/452b543a65