Support an empty value as a Candidate that only suggests but doesn't …

…complete, fixes #89
jline · Jan 13, 2017 · 2144e6a · 2144e6a
1 parent 6f7a2e5
commit 2144e6a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/builtins/src/test/java/org/jline/example/Example.java b/builtins/src/test/java/org/jline/example/Example.java
@@ -128,7 +128,13 @@ public static void main(String[] args) throws IOException {
                     case "foo":
                         completer = new ArgumentCompleter(
                                 new StringsCompleter("foo11", "foo12", "foo13"),
-                                new StringsCompleter("foo21", "foo22", "foo23"));
+                                new StringsCompleter("foo21", "foo22", "foo23"),
+                                new Completer() {
+                                    @Override
+                                    public void complete(LineReader reader, ParsedLine line, List<Candidate> candidates) {
+                                        candidates.add(new Candidate("", "", null, "frequency in MHz", null, null, false));
+                                    }
+                                });
                         break label;
                     case "param":
                         completer = (reader, line, candidates) -> {

diff --git a/reader/src/main/java/org/jline/reader/impl/LineReaderImpl.java b/reader/src/main/java/org/jline/reader/impl/LineReaderImpl.java
@@ -3751,7 +3751,7 @@ else if (isSet(Option.RECOGNIZE_EXACT)) {
                     .findFirst().orElse(null);
         }
         // Complete and exit
-        if (completion != null) {
+        if (completion != null && !completion.value().isEmpty()) {
             if (prefix) {
                 buf.backspace(line.wordCursor());
             } else {