You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by ff...@apache.org on 2013/02/01 04:39:09 UTC

svn commit: r1441305 - /karaf/branches/karaf-2.3.x/shell/console/src/main/java/org/apache/karaf/shell/console/jline/Console.java

Author: ffang
Date: Fri Feb  1 03:39:08 2013
New Revision: 1441305

URL: http://svn.apache.org/viewvc?rev=1441305&view=rev
Log:
[KARAF-2154]Entering an empty command removes previous command in history 

Modified:
    karaf/branches/karaf-2.3.x/shell/console/src/main/java/org/apache/karaf/shell/console/jline/Console.java

Modified: karaf/branches/karaf-2.3.x/shell/console/src/main/java/org/apache/karaf/shell/console/jline/Console.java
URL: http://svn.apache.org/viewvc/karaf/branches/karaf-2.3.x/shell/console/src/main/java/org/apache/karaf/shell/console/jline/Console.java?rev=1441305&r1=1441304&r2=1441305&view=diff
==============================================================================
--- karaf/branches/karaf-2.3.x/shell/console/src/main/java/org/apache/karaf/shell/console/jline/Console.java (original)
+++ karaf/branches/karaf-2.3.x/shell/console/src/main/java/org/apache/karaf/shell/console/jline/Console.java Fri Feb  1 03:39:08 2013
@@ -244,7 +244,12 @@ public class Console implements Runnable
             if (reader.getHistory().size()==0) {
                 reader.getHistory().add(command);
             } else {
-                reader.getHistory().replace(command);
+                // jline doesn't add blank lines to the history so we don't 
+                // need to replace the command in jline's console history with
+                // an indented one
+                if (command.length() > 0 && !" ".equals(command)) {
+                    reader.getHistory().replace(command);    
+                }                                
             }
             try {
                 new Parser(command).program();