From: wmorgan <wmorgan@5c8cc53c-5e98-4d25-b20a-d8db53a31250>
Date: Sat, 10 Feb 2007 18:40:51 +0000 (+0000)
Subject: minor code cleanups
X-Git-Url: https://git.notmuchmail.org/git?a=commitdiff_plain;h=86db2c43393c9981291945ba810c53272849f0df;p=sup

minor code cleanups

git-svn-id: svn://rubyforge.org/var/svn/sup/trunk@315 5c8cc53c-5e98-4d25-b20a-d8db53a31250
---

diff --git a/lib/sup/buffer.rb b/lib/sup/buffer.rb
index 0e2b85b..d4fe549 100644
--- a/lib/sup/buffer.rb
+++ b/lib/sup/buffer.rb
@@ -359,9 +359,9 @@ class BufferManager
     ret
   end
 
+  ## returns true (y), false (n), or nil (ctrl-g / cancel)
   def ask_yes_or_no question
-    r = ask_getch(question, "ynYN")
-    case r
+    case(r = ask_getch question, "ynYN")
     when ?y, ?Y
       true
     when nil
@@ -399,6 +399,7 @@ class BufferManager
 
   def say s, id=nil
     new_id = nil
+
     @minibuf_mutex.synchronize do
       new_id = id.nil?
       id ||= @minibuf_stack.length
diff --git a/lib/sup/mode.rb b/lib/sup/mode.rb
index 88227e9..640fbbe 100644
--- a/lib/sup/mode.rb
+++ b/lib/sup/mode.rb
@@ -48,12 +48,9 @@ class Mode
   end
 
   def handle_input c
-    if(action = resolve_input c)
-      send action
-      true
-    else
-      false
-    end
+    action = resolve_input(c) or return false
+    send action
+    true
   end
 
   def help_text