From 78c6de9daf7970ae40d50899ee0ceb67c0297a94 Mon Sep 17 00:00:00 2001
From: David Heinemeier Hansson <david@loudthinking.com>
Date: Wed, 22 Dec 2004 13:17:35 +0000
Subject: [PATCH] Fixed CTRL-C exists from the Breakpointer to be a clean
 affair without error dumping [Kent Sibilev]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@246 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
---
 railties/CHANGELOG                | 2 ++
 railties/lib/breakpoint_client.rb | 7 ++++++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/railties/CHANGELOG b/railties/CHANGELOG
index 38f4cb1928..8d5c089558 100644
--- a/railties/CHANGELOG
+++ b/railties/CHANGELOG
@@ -1,5 +1,7 @@
 *SVN*
 
+* Fixed CTRL-C exists from the Breakpointer to be a clean affair without error dumping [Kent Sibilev]
+
 * Fixed "rake stats" to work with sub-directories in models and controllers and to report the code to test ration [Scott Baron]
 
 * Added that Active Record associations are now reloaded instead of cleared to work with the new const_missing hook in Active Record.
diff --git a/railties/lib/breakpoint_client.rb b/railties/lib/breakpoint_client.rb
index 8a8bc75594..fa93c11f3e 100644
--- a/railties/lib/breakpoint_client.rb
+++ b/railties/lib/breakpoint_client.rb
@@ -62,6 +62,10 @@
 
 options[:ServerURI] = ARGV[0] if ARGV[0]
 
+$running = true
+
+trap("INT"){$running = false}
+
 puts "Waiting for initial breakpoint..."
 
 loop do
@@ -138,7 +142,7 @@
 
       puts "Connection established. Waiting for breakpoint...", "" if options[:Verbose]
 
-      loop do
+      while $running
         begin
           service.ping
         rescue DRb::DRbConnError => error
@@ -152,6 +156,7 @@
       service.unregister_handler
     end
   rescue Exception => error
+    break unless $running
     if options[:RetryDelay] > 0 then
       puts "No connection to breakpoint service at #{options[:ServerURI]}:", "  (#{error.inspect})" if options[:Verbose]
       error.backtrace if $DEBUG
-- 
GitLab