* [sup-talk] [PATCH] limit option parser nil check
@ 2009-05-13 9:00 Marcus Williams
2009-05-18 18:26 ` William Morgan
0 siblings, 1 reply; 2+ messages in thread
From: Marcus Williams @ 2009-05-13 9:00 UTC (permalink / raw)
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
^ permalink raw reply [flat|nested] 2+ messages in thread
* [sup-talk] [PATCH] limit option parser nil check
2009-05-13 9:00 [sup-talk] [PATCH] limit option parser nil check Marcus Williams
@ 2009-05-18 18:26 ` William Morgan
0 siblings, 0 replies; 2+ messages in thread
From: William Morgan @ 2009-05-18 18:26 UTC (permalink / raw)
Reformatted excerpts from Marcus Williams's message of 2009-05-13:
> 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.
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>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-05-18 18:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-05-13 9:00 [sup-talk] [PATCH] limit option parser nil check Marcus Williams
2009-05-18 18:26 ` William Morgan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox