pry-rescue 0.13.pre.3 → 0.13.pre.4

data/lib/pry-rescue/core_ext.rb

@@ -37,7 +37,22 @@ class << Pry
     if i = (@raised || []).index{ |(ee, _)| ee == e }
       PryRescue.enter_exception_context(@raised[0..i])
     else
-      raise "Tried to inspect an exception that was not raised in a Pry::rescue{ } block"
+      case e
+      when SystemStackError
+        # Interception cannot reliably interept SystemStackErrors as it needs
+        # to call a function at the point the stack runs out.
+        # We use a special error message here, as it seems nicer to assume that
+        # the user knows what they are doing, and it's the software that's
+        # terrible.
+        warn "WARNING: Insufficient stack space to inspect exception"
+      else
+        # We used to raise an exception at this point, but that turned out to
+        # not be very helpful as it obscured the original cause of the problem.
+        # I considered adding an explicit 'raise e' here, but decided against
+        # it on the grounds that the Pry::rescued call is normally in someone
+        # else's error handler already.
+        warn "WARNING: Tried to inspect an exception that was not raised within Pry::rescue{ }"
+      end
     end
 
   ensure

data/pry-rescue.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name          = 'pry-rescue'
-  s.version       = '0.13.pre.3'
+  s.version       = '0.13.pre.4'
   s.summary       = 'Open a pry session on any unhandled exceptions'
   s.description   = 'Allows you to wrap code in Pry::rescue{ } to open a pry session at any unhandled exceptions'
   s.homepage      = 'https://github.com/ConradIrwin/pry-rescue'

data/spec/core_ext_spec.rb

@@ -100,17 +100,15 @@ describe "Pry.rescued" do
     begin
       raise "foo"
     rescue => e
-      lambda{
-        Pry.rescued(e)
-      }.should raise_error(/Pry::rescue/)
+      Pry.should_receive(:warn)
+      Pry.rescued(e)
     end
   end
 
   it "should raise an error if used on an exception not raised" do
     Pry::rescue do
-      lambda{
-        Pry.rescued(RuntimeError.new("foo").exception)
-      }.should raise_error(/not raised/)
+      Pry.should_receive(:warn).with("WARNING: Tried to inspect an exception that was not raised within Pry::rescue{ }")
+      Pry.rescued(RuntimeError.new("foo").exception)
     end
   end
 

metadata

@@ -1,98 +1,109 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: pry-rescue
-version: !ruby/object:Gem::Version 
-  prerelease: true
-  segments: 
-  - 0
-  - 13
-  - pre
-  - 3
-  version: 0.13.pre.3
+version: !ruby/object:Gem::Version
+  version: 0.13.pre.4
+  prerelease: 5
 platform: ruby
-authors: 
+authors:
 - Conrad Irwin
 - banisterfiend
 - epitron
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2012-10-23 00:00:00 -07:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2012-10-31 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: pry
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: interception
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        - 3
-        version: "0.3"
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: interception
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0.3'
   type: :runtime
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: rspec
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0.3'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: yard
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
-  name: redcarpet
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: redcarpet
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id005
-description: Allows you to wrap code in Pry::rescue{ } to open a pry session at any unhandled exceptions
-email: 
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+description: Allows you to wrap code in Pry::rescue{ } to open a pry session at any
+  unhandled exceptions
+email:
 - conrad.irwin@gmail.com
 - jrmair@gmail.com
 - chris@ill-logic.com
-executables: 
+executables:
 - rescue
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
 - .gitignore
 - Gemfile
 - LICENSE.MIT
@@ -131,37 +142,29 @@ files:
 - spec/fixtures/uri.rb
 - spec/peek_spec.rb
 - spec/pry_rescue_spec.rb
-has_rdoc: true
 homepage: https://github.com/ConradIrwin/pry-rescue
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  requirements: 
-  - - ">"
-    - !ruby/object:Gem::Version 
-      segments: 
-      - 1
-      - 3
-      - 1
+required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>'
+    - !ruby/object:Gem::Version
       version: 1.3.1
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.3.6
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Open a pry session on any unhandled exceptions
 test_files: []
-
+has_rdoc: