You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by jb...@apache.org on 2012/02/08 15:08:45 UTC

svn commit: r1241903 - /karaf/branches/karaf-2.2.x/shell/commands/src/main/java/org/apache/karaf/shell/commands/SleepAction.java

Author: jbonofre
Date: Wed Feb  8 14:08:45 2012
New Revision: 1241903

URL: http://svn.apache.org/viewvc?rev=1241903&view=rev
Log:
[KARAF-1191] Switching back to milliseconds by default in shell:sleep and add the -s (second) option

Modified:
    karaf/branches/karaf-2.2.x/shell/commands/src/main/java/org/apache/karaf/shell/commands/SleepAction.java

Modified: karaf/branches/karaf-2.2.x/shell/commands/src/main/java/org/apache/karaf/shell/commands/SleepAction.java
URL: http://svn.apache.org/viewvc/karaf/branches/karaf-2.2.x/shell/commands/src/main/java/org/apache/karaf/shell/commands/SleepAction.java?rev=1241903&r1=1241902&r2=1241903&view=diff
==============================================================================
--- karaf/branches/karaf-2.2.x/shell/commands/src/main/java/org/apache/karaf/shell/commands/SleepAction.java (original)
+++ karaf/branches/karaf-2.2.x/shell/commands/src/main/java/org/apache/karaf/shell/commands/SleepAction.java Wed Feb  8 14:08:45 2012
@@ -16,6 +16,7 @@
  */
 package org.apache.karaf.shell.commands;
 
+import org.apache.felix.gogo.commands.Option;
 import org.apache.karaf.shell.console.AbstractAction;
 import org.apache.felix.gogo.commands.Argument;
 import org.apache.felix.gogo.commands.Command;
@@ -23,14 +24,22 @@ import org.apache.felix.gogo.commands.Co
 @Command(scope = "shell", name = "sleep", description = "Sleeps for a bit then wakes up.")
 public class SleepAction extends AbstractAction {
 
-    @Argument(index = 0, name = "duration", description = "The amount of seconds to sleep", required = true, multiValued = false)
+    @Argument(index = 0, name = "duration", description = "The amount of time to sleep. The default time unit is millisecond, use -s option to use second instead.", required = true, multiValued = false)
     private long time = -1;
+    
+    @Option(name = "-s", aliases = { "--second" }, description = "Use a duration time in seconds instead of milliseconds.", required = false, multiValued = false)
+    private boolean second = false;
 
     protected Object doExecute() throws Exception {
-        log.info("Sleeping for {} second(s)", time);
+        if (second) {
+            log.info("Sleeping for {} second(s)", time);
+            time = time * 1000;
+        } else {
+            log.info("Sleeping for {} millisecond(s)", time);
+        }
 
         try {
-            Thread.sleep(time * 1000);
+            Thread.sleep(time);
         }
         catch (InterruptedException ignore) {
             log.debug("Sleep was interrupted... :-(");