From: nagachika00@...
Date: 2014-08-30T15:40:49+00:00
Subject: [ruby-core:64673] [ruby-trunk - Feature #10173] Remove meaningless	ensure

Issue #10173 has been updated by Tomoyuki Chikanaga.


The patch looks good to me.
But rubygems has upstream repository.

https://github.com/rubygems/rubygems/

Please report to it first.

----------------------------------------
Feature #10173: Remove meaningless ensure
https://bugs.ruby-lang.org/issues/10173#change-48568

* Author: gogo tanaka
* Status: Open
* Priority: Normal
* Assignee: 
* Category: lib
* Target version: 
----------------------------------------
I suppose 'empty ensure' like that ��� has no meaning,

```
begin
  r.document args
rescue Errno::EACCES => e
  dirname = File.dirname e.message.split("-")[1].strip
  raise Gem::FilePermissionError, dirname
rescue Interrupt => e
  raise e
rescue Exception => ex
  alert_error "While generating documentation for #{@spec.full_name}"
  ui.errs.puts "... MESSAGE:   #{ex}"
  ui.errs.puts "... RDOC args: #{args.join(' ')}"
  ui.backtrace ex
  ui.errs.puts "(continuing with the rest of the installation)"
ensure
end
```

So I remove that.

---Files--------------------------------
EmptyEnsure.PATCH (385 Bytes)


-- 
https://bugs.ruby-lang.org/