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) almost 2 years 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