Subject: [sup-talk] [PATCH] limit option parser nil check

From: [email-masked] (Marcus Williams)
Date: Wed, 13 May 2009 10:00:31 +0100

The limit search option doesnt check if the current search term is nil
(when chronic has failed for instance). This patch (against next) adds
the check.
---
 lib/sup/index.rb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/lib/sup/index.rb b/lib/sup/index.rb
index c66a24e..4ebc966 100644
--- a/lib/sup/index.rb
+++ b/lib/sup/index.rb
@@ -584,7 +584,7 @@ protected
         BufferManager.flash "Can't understand limit #{lim.inspect}!"
         subs = nil
       end
-    end
+    end unless subs.nil?
     
     if subs
       [@qparser.parse(subs), extraopts]
-- 
1.5.4.1

---

From: [email-masked] (William Morgan)
Date: Mon, 18 May 2009 11:26:10 -0700

> (when chronic has failed for instance). This patch (against next) adds

I've reworked this code in the parser-user-query-fix branch, currently
merged into next, and it should no longer require this fix.
-- 
William <wmorgan-sup at masanjin.net>