Skip to content

Commit

Permalink
merge revision(s) 15464:
Browse files Browse the repository at this point in the history
	* lib/timeout.rb (Timeout::timeout): made sensitive to location on the
	  stack.  [ruby-core:15458]


git-svn-id: http://svn.ruby-lang.org/repos/ruby/branches/ruby_1_8_6@17158 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
  • Loading branch information
shyouhei committed Jun 14, 2008
1 parent 0d04ec4 commit 00a8c1b
Show file tree
Hide file tree
Showing 3 changed files with 29 additions and 7 deletions.
5 changes: 5 additions & 0 deletions ChangeLog
@@ -1,3 +1,8 @@
Sat Jun 14 16:49:41 2008 Nobuyoshi Nakada <nobu@ruby-lang.org>

* lib/timeout.rb (Timeout::timeout): made sensitive to location on the
stack. [ruby-core:15458]

Fri Jun 13 13:14:31 2008 Yukihiro Matsumoto <matz@ruby-lang.org>

* ext/dl/ptr.c (dlmem_each_i): typo fixed. a patch from IKOMA
Expand Down
23 changes: 20 additions & 3 deletions lib/timeout.rb
Expand Up @@ -32,8 +32,13 @@ module Timeout
##
# Raised by Timeout#timeout when the block times out.

class Error<Interrupt
class Error < Interrupt
end
class ExitException < ::Exception # :nodoc:
end

THIS_FILE = /\A#{Regexp.quote(__FILE__)}:/o
CALLER_OFFSET = ((c = caller[0]) && THIS_FILE =~ c) ? 1 : 0

##
# Executes the method's block. If the block execution terminates before +sec+
Expand All @@ -44,9 +49,10 @@ class Error<Interrupt
# Timeout' into your classes so they have a #timeout method, as well as a
# module method, so you can call it directly as Timeout.timeout().

def timeout(sec, exception=Error)
def timeout(sec, klass = nil)
return yield if sec == nil or sec.zero?
raise ThreadError, "timeout within critical session" if Thread.critical
exception = klass || Class.new(ExitException)
begin
x = Thread.current
y = Thread.start {
Expand All @@ -55,6 +61,17 @@ def timeout(sec, exception=Error)
}
yield sec
# return true
rescue exception => e
rej = /\A#{Regexp.quote(__FILE__)}:#{__LINE__-4}\z/o
(bt = e.backtrace).reject! {|m| rej =~ m}
level = -caller(CALLER_OFFSET).size
while THIS_FILE =~ bt[level]
bt.delete_at(level)
level += 1
end
raise if klass # if exception class is specified, it
# would be expected outside.
raise Error, e.message, e.backtrace
ensure
y.kill if y and y.alive?
end
Expand All @@ -72,7 +89,7 @@ def timeout(sec, exception=Error)
# Defined for backwards compatibility with earlier versions of timeout.rb, see
# Timeout#timeout.

def timeout(n, e=Timeout::Error, &block) # :nodoc:
def timeout(n, e = nil, &block) # :nodoc:
Timeout::timeout(n, e, &block)
end

Expand Down
8 changes: 4 additions & 4 deletions version.h
@@ -1,15 +1,15 @@
#define RUBY_VERSION "1.8.6"
#define RUBY_RELEASE_DATE "2008-06-13"
#define RUBY_RELEASE_DATE "2008-06-14"
#define RUBY_VERSION_CODE 186
#define RUBY_RELEASE_CODE 20080613
#define RUBY_PATCHLEVEL 175
#define RUBY_RELEASE_CODE 20080614
#define RUBY_PATCHLEVEL 176

#define RUBY_VERSION_MAJOR 1
#define RUBY_VERSION_MINOR 8
#define RUBY_VERSION_TEENY 6
#define RUBY_RELEASE_YEAR 2008
#define RUBY_RELEASE_MONTH 6
#define RUBY_RELEASE_DAY 13
#define RUBY_RELEASE_DAY 14

#ifdef RUBY_EXTERN
RUBY_EXTERN const char ruby_version[];
Expand Down

0 comments on commit 00a8c1b

Please sign in to comment.