From e7eecfa8ea2d59b39eeb4c8aab69a1be862857a7 Mon Sep 17 00:00:00 2001 From: Zeh Rizzatti Date: Thu, 23 Aug 2012 00:39:24 -0400 Subject: [PATCH] do not propagate nils from completion_proc Fix issue #656 The default completion_proc will generate an array which might contain nils, and Coolline is not able to handle them gracefully. --- lib/pry/pry_instance.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/pry/pry_instance.rb b/lib/pry/pry_instance.rb index f228a022..5092c65a 100644 --- a/lib/pry/pry_instance.rb +++ b/lib/pry/pry_instance.rb @@ -599,7 +599,8 @@ class Pry if defined? Coolline and input.is_a? Coolline input.completion_proc = proc do |cool| - completion_proc.call cool.completed_word + completions = completion_proc.call cool.completed_word + completions.compact end elsif input.respond_to? :completion_proc= input.completion_proc = completion_proc