From: "Mon_Ouie (Mon ouïe)" <mon.ouie@...>
Date: 2013-02-24T21:30:15+09:00
Subject: [ruby-core:52813] [ruby-trunk - Bug #7940][Open] Mistaken use of inline rescues in stdlib


Issue #7940 has been reported by Mon_Ouie (Mon ou��e).

----------------------------------------
Bug #7940: Mistaken use of inline rescues in stdlib
https://bugs.ruby-lang.org/issues/7940

Author: Mon_Ouie (Mon ou��e)
Status: Open
Priority: Low
Assignee: 
Category: lib
Target version: 
ruby -v: ruby 1.9.3p392 (2013-02-22 revision 39386) [x86_64-linux]


Hey,

There are two uses of inline rescues in stdlib that look dubious. They are unlikely to ever cause any harm, but still. They've been there for a while now, and are still in trunk. I attached a patch that changes both of them.

In cgi/session.rb:

      def delete
        File::unlink @path+".lock" rescue nil
        File::unlink @path+".new" rescue nil
        File::unlink @path rescue Errno::ENOENT
      end

The ``rescue`` part should be on the next line (this has been introduced in a commit that changed indentation).

In scanf.rb:660:

   seek(start_position + matched_so_far, IO::SEEK_SET) rescue Errno::ESPIPE

was more likely meant to just silence Errno::ESPIPE exceptions.


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