From: Mitch Mitch <mitch.lloyd@...>
Date: 2008-02-12T11:06:49+09:00
Subject: DRb rescue:  Shouldn't this be changed?

I'm a ruby newbie so I wanted to ask a forum about this before I emailed
the author.  The current drb/observer.rb file consists of this code:

require 'observer'

module DRb
  module DRbObservable
    include Observable

    def notify_observers(*arg)
      if defined? @observer_state and @observer_state
  if defined? @observer_peers
    for i in @observer_peers.dup
      begin
        i.update(*arg)
      rescue
        delete_observer(i)
      end
    end
  end
  @observer_state = false
      end
    end
  end
end

My issue is with the rescue.  I after a bunch of time trying to track
down a bug I realized that I had a typo in an observer method which was
causing the observer to be deleted.

My suggestion is to change the line to:

rescue RangeError

Which only rescues the error that the author was intending (I believe).

Am I missing anything here?
-- 
Posted via http://www.ruby-forum.com/.