[maven-release-plugin] prepare release system-0.0.2

Signed-off-by: Emmanuel Bigeon <emmanuel@bigeon.fr>

Jenkinsfile

@@ -0,0 +1,65 @@
+pipeline {
+  agent any
+  stages {
+    stage('Site GCLC') {
+      when {
+        tag "gclc-*"
+      }
+      steps {
+        dir('gclc') {
+          sh 'mvn site:site'
+          sh 'mkdir -p /home/maven-sites/net/bigeon/gclc'
+          sh 'cp -r target/site /home/maven-sites/net/bigeon/gclc/gclc'
+        }
+      }
+    }
+    stage('Site GCLC-Socket') {
+      when {
+        tag "socket-*"
+      }
+      steps {
+        dir('gclc-socket') {
+          sh 'mvn site:site'
+          sh 'mkdir -p /home/maven-sites/net/bigeon/gclc'
+          sh 'cp -r target/site /home/maven-sites/net/bigeon/gclc/gclc-socket'
+        }
+      }
+    }
+    stage('Site GCLC-Swt') {
+      when {
+        tag "swt-*"
+      }
+      steps {
+        dir('gclc-swt') {
+          sh 'mvn site:site'
+          sh 'mkdir -p /home/maven-sites/net/bigeon/gclc'
+          sh 'cp -r target/site /home/maven-sites/net/bigeon/gclc/gclc-swt'
+        }
+      }
+    }
+    stage('Site GCLC-Processes') {
+      when {
+        tag "process-*"
+      }
+      steps {
+        dir('gclc-process') {
+          sh 'mvn site:site'
+          sh 'mkdir -p /home/maven-sites/net/bigeon/gclc'
+          sh 'cp -r target/site /home/maven-sites/net/bigeon/gclc/gclc-process'
+        }
+      }
+    }
+    stage('Site GCLC-System') {
+      when {
+        tag "system-*"
+      }
+      steps {
+        dir('gclc.system') {
+          sh 'mvn site:site'
+          sh 'mkdir -p /home/maven-sites/net/bigeon/gclc'
+          sh 'cp -r target/site /home/maven-sites/net/bigeon/gclc/gclc-system'
+        }
+      }
+    }
+  }
+}

gclc-process/pom.xml

@@ -1,36 +1,12 @@
-
-<!-- process, Distribution repositories and basic setup for Emmanuel Bigeon projects                                   -->
-<!-- Copyright (C) 2014-2018 E. Bigeon                      -->
-<!-- mailto:emmanuel@bigeon.fr                                                           -->
-<!--                                                                           -->
-<!-- This software is governed by the CeCILL license under French law and      -->
-<!-- abiding by the rules of distribution of free software.  You can  use,     -->
-<!-- modify and/or redistribute the software under the terms of the CeCILL     -->
-<!-- license as circulated by CEA, CNRS and INRIA at the following URL         -->
-<!-- "http://www.cecill.info".                                                 -->
-<!--                                                                           -->
-<!-- As a counterpart to the access to the source code and  rights to copy,    -->
-<!-- modify and redistribute granted by the license, users are provided only   -->
-<!-- with a limited warranty  and the software's author,  the holder of the    -->
-<!-- economic rights,  and the successive licensors  have only  limited        -->
-<!-- liability.                                                                -->
-<!--                                                                           -->
-<!-- In this respect, the user's attention is drawn to the risks associated    -->
-<!-- with loading,  using,  modifying and/or developing or reproducing the     -->
-<!-- software by the user in light of its specific status of free software,    -->
-<!-- that may mean  that it is complicated to manipulate,  and  that  also     -->
-<!-- therefore means  that it is reserved for developers  and  experienced     -->
-<!-- professionals having in-depth computer knowledge. Users are therefore     -->
-<!-- encouraged to load and test the software's suitability as regards their   -->
-<!-- requirements in conditions enabling the security of their systems and/or  -->
-<!-- data to be ensured and,  more generally, to use and operate it in the     -->
-<!-- same conditions as regards security.                                      -->
-<!--                                                                           -->
-<!-- The fact that you are presently reading this means that you have had      -->
-<!-- knowledge of the CeCILL license and that you accept its terms.            -->
-
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
 	<modelVersion>4.0.0</modelVersion>
+	<parent>
+		<groupId>net.bigeon.config</groupId>
+		<artifactId>ebigeon-config</artifactId>
+		<version>1.8.21</version>
+	</parent>
 
 	<groupId>net.bigeon.gclc</groupId>
 	<artifactId>process</artifactId>
@@ -38,27 +14,12 @@
 	<packaging>jar</packaging>
 
 	<name>process</name>
-  <url>http://maven.apache.org</url>
-
-  <properties>
-    <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-    <copyright.email>emmanuel@bigeon.fr</copyright.email>
-    <license.licenseName>cecill_2.1</license.licenseName>
-  </properties>
-
-  <parent>
-    <groupId>net.bigeon.config</groupId>
-    <artifactId>ebigeon-config</artifactId>
-    <version>1.8.9</version>
-  </parent>
-  <dependencies>
-    <dependency>
-      <groupId>net.bigeon</groupId>
-      <artifactId>gclc</artifactId>
-      <version>2.0.7</version>
-    </dependency>
-  </dependencies>
-
+	<description>A library to handle processes in a generic console application.</description><url>https://bigeon.net/projects/gclc.html</url>
+	<inceptionYear>2017</inceptionYear>
+	<organization>
+		<name>Bigeon</name>
+		<url>https://bigeon.net</url>
+	</organization>
 	<licenses>
 		<license>
 			<distribution>manual</distribution>
@@ -66,6 +27,33 @@
 			<url>https://cecill.info/licences/Licence_CeCILL_V2.1-en.html</url>
 		</license>
 	</licenses>
+
+	<developers>
+		<developer>
+			<email>emmanuel@bigeon.fr</email>
+			<name>Emmanuel Bigeon</name>
+			<url>bigeon.net</url>
+			<roles>
+				<role>PM</role>
+			</roles>
+		</developer>
+	</developers>
+	<scm>
+		<tag>HEAD</tag>
+		<developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
+	</scm>
+	<properties>
+		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
+		<copyright.email>emmanuel@bigeon.fr</copyright.email>
+		<license.licenseName>cecill_2.1</license.licenseName>
+	</properties>
+	<dependencies>
+		<dependency>
+			<groupId>net.bigeon</groupId>
+			<artifactId>gclc</artifactId>
+			<version>2.0.11</version>
+		</dependency>
+	</dependencies>
 	<reporting>
 		<plugins>
 			<plugin>
@@ -80,19 +68,5 @@
 			</plugin>
 		</plugins>
 	</reporting>
-	<developers>
-		<developer>
-			<email>emmanuel@bigeon.fr</email>
-			<name>Emmanuel Bigeon</name>
-			<url>bigeon.net</url>
-			<roles>
-				<role>PM</role>
-			</roles>
-		</developer>
-	</developers>
 	
-  <scm>
-    <tag>process-0.0.1</tag>
-    <developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
-  </scm>
 </project>

gclc-process/src/main/java/net/bigeon/gclc/process/CommandForeground.java

@@ -69,6 +69,8 @@ package net.bigeon.gclc.process;
  * #L%
  */
 import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import net.bigeon.gclc.command.CommandParameters;
 import net.bigeon.gclc.command.ParametrizedCommand;
@@ -92,28 +94,33 @@ import net.bigeon.gclc.manager.ConsoleOutput;
  * @author Emmanuel Bigeon */
 public class CommandForeground extends ParametrizedCommand {
 
+    /** Number of milliseconds in a second. */
+    private static final int    MILLIS_IN_A_SEC = 1000;
+    /** The class logger. */
+    private static final Logger LOGGER          = Logger
+            .getLogger(CommandForeground.class.getName());
+
+    /** The task pool to fetch task from. */
     private final TaskPool pool;
 
     /** Add the fork command.
      *
      * @param name the command name
      * @param pool The pool to get joinable tasks from */
-    public CommandForeground(final String name, TaskPool pool) {
+    public CommandForeground(final String name, final TaskPool pool) {
         super(name, false);
         this.pool = pool;
         addParameters();
     }
 
-    /**
-     *
-     */
+    /** Add the parameters of the command. */
     private void addParameters() {
         try {
             addStringParameter("pid", false);
             addStringParameter("delai", false);
         } catch (final InvalidParameterException e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
+            // Cannot be reached unless GCLC base framework has an error
+            LOGGER.log(Level.SEVERE, "unexpected parameter error!", e);
         }
     }
 
@@ -125,26 +132,34 @@ public class CommandForeground extends ParametrizedCommand {
     protected void doExecute(final ConsoleOutput out, final ConsoleInput in,
             final CommandParameters parameters) throws CommandRunException {
         String string = parameters.get("pid");
-        final List<String> additionals = parameters.getAdditionals();
-        if (string == null && !additionals.isEmpty()) {
-            string = additionals.get(0);
-        }
         if (string == null) {
+            final List<String> additionals = parameters.getAdditionals();
+            if (additionals.isEmpty()) {
                 throw new CommandRunException(CommandRunExceptionType.USAGE,
                         "Missing process id");
             }
+            string = additionals.get(0);
+        }
+        Integer pid;
+        try {
+            pid = Integer.valueOf(string);
+        } catch (final NumberFormatException e) {
+            throw new CommandRunException(CommandRunExceptionType.USAGE,
+                    "PID should be an integer");
+        }
         // Join the command.
-        final Task cmd = pool.get(string);
+        final Task cmd = pool.get(pid);
         if (!(cmd instanceof ForkTask)) {
             throw new CommandRunException("No such forked process");
         }
         long delai = 0;
         final String delaiOpt = parameters.get("delai");
         if (delaiOpt != null) {
-            delai = Long.parseLong(delaiOpt) * 1000;
+            delai = Long.parseLong(delaiOpt) * MILLIS_IN_A_SEC;
         }
         if (delai < 0) {
-            throw new CommandRunException("Join delai cannot be negative");
+            throw new CommandRunException(CommandRunExceptionType.USAGE,
+                    "Join delai cannot be negative");
         }
         ((ForkTask) cmd).join(out, in, delai);
     }

gclc-process/src/main/java/net/bigeon/gclc/process/CommandFork.java

@@ -1,40 +1,10 @@
-/*
- * process, Distribution repositories and basic setup for Emmanuel Bigeon projects
- * Copyright (C) 2014-2018 E. Bigeon
- * mailto:emmanuel@bigeon.fr
- *
- * This software is governed by the CeCILL license under French law and
- * abiding by the rules of distribution of free software.  You can  use,
- * modify and/or redistribute the software under the terms of the CeCILL
- * license as circulated by CEA, CNRS and INRIA at the following URL
- * "http://www.cecill.info".
- *
- * As a counterpart to the access to the source code and  rights to copy,
- * modify and redistribute granted by the license, users are provided only
- * with a limited warranty  and the software's author,  the holder of the
- * economic rights,  and the successive licensors  have only  limited
- * liability.
- *
- * In this respect, the user's attention is drawn to the risks associated
- * with loading,  using,  modifying and/or developing or reproducing the
- * software by the user in light of its specific status of free software,
- * that may mean  that it is complicated to manipulate,  and  that  also
- * therefore means  that it is reserved for developers  and  experienced
- * professionals having in-depth computer knowledge. Users are therefore
- * encouraged to load and test the software's suitability as regards their
- * requirements in conditions enabling the security of their systems and/or
- * data to be ensured and,  more generally, to use and operate it in the
- * same conditions as regards security.
- *
- * The fact that you are presently reading this means that you have had
- * knowledge of the CeCILL license and that you accept its terms.
- */
 /**
  * gclc-process:net.bigeon.gclc.process.CommandFork.java
  * Created on: Nov 13, 2017
  */
 package net.bigeon.gclc.process;
 
+import java.text.MessageFormat;
 /*-
  * #%L
  * process
@@ -74,6 +44,7 @@ import net.bigeon.gclc.command.Command;
 import net.bigeon.gclc.command.ICommand;
 import net.bigeon.gclc.command.ICommandProvider;
 import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
 
@@ -91,15 +62,19 @@ import net.bigeon.gclc.manager.ConsoleOutput;
  * @author Emmanuel Bigeon */
 public class CommandFork extends Command {
 
+    /** The task pool containing the tasks. */
     private final TaskPool         pool;
+    /** The command provider. */
     private final ICommandProvider provider;
+    /** The number of lines stored in the commands. */
     private final int              lines;
 
     /** Add the fork command.
      *
      * @param name the command name
      * @param provider the allowed command collection */
-    public CommandFork(final String name, ICommandProvider provider, TaskPool pool) {
+    public CommandFork(final String name, final ICommandProvider provider,
+                       final TaskPool pool) {
         this(name, provider, pool, -1);
     }
 
@@ -107,24 +82,33 @@ public class CommandFork extends Command {
      *
      * @param name the command name
      * @param provider the allowed command collection */
-    public CommandFork(final String name, ICommandProvider provider, TaskPool pool,
-                       int lines) {
+    public CommandFork(final String name, final ICommandProvider provider,
+                       final TaskPool pool, final int lines) {
         super(name);
         this.provider = provider;
         this.pool = pool;
         this.lines = lines;
     }
 
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.command.ICommand#execute(net.bigeon.gclc.manager.
+     * ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
     @Override
-    public void execute(final ConsoleOutput out, final ConsoleInput in, String... args)
-            throws CommandRunException {
+    public void execute(final ConsoleOutput out, final ConsoleInput in,
+            final String... args) throws CommandRunException {
         if (args.length < 1) {
-            throw new CommandRunException("No command to fork");
+            throw new CommandRunException(CommandRunExceptionType.USAGE,
+                    "No command to fork");
+        }
+        final String string = args[0];
+        final ICommand cmd = provider.get(string);
+        if (cmd == null) {
+            throw new CommandRunException(CommandRunExceptionType.USAGE,
+                    MessageFormat.format("No such command {0}", string));
         }
-        final ICommand cmd = provider.get(args[0]);
         final String[] inner = Arrays.copyOfRange(args, 1, args.length);
         final ForkTask task = new ForkCommandTask(cmd, inner, lines);
-        final Thread th = new Thread(task);
+        final Thread th = new Thread(task, MessageFormat.format("fork [{0}]", string));
         pool.add(task);
         th.start();
     }

gclc-process/src/main/java/net/bigeon/gclc/process/ForkCommandTask.java

@@ -72,10 +72,7 @@ import java.util.Arrays;
 import net.bigeon.gclc.command.ICommand;
 import net.bigeon.gclc.exception.CommandRunException;
 
-/**
- * @author Emmanuel Bigeon
- *
- */
+/** @author Emmanuel Bigeon */
 public class ForkCommandTask extends ForkTask {
     private final ICommand command;
     private final String[] args;
@@ -89,13 +86,6 @@ public class ForkCommandTask extends ForkTask {
         this.args = Arrays.copyOf(args, args.length);
     }
 
-    /* (non-Javadoc)
-     * @see net.bigeon.gclc.process.Task#getName() */
-    @Override
-    public String getName() {
-        return command.getCommandName();
-    }
-
     /* (non-Javadoc)
      * @see net.bigeon.gclc.process.ForkTask#doRun() */
     @Override
@@ -103,4 +93,11 @@ public class ForkCommandTask extends ForkTask {
         command.execute(out, in, args);
     }
 
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.process.Task#getName() */
+    @Override
+    public String getName() {
+        return command.getCommandName();
+    }
+
 }

gclc-process/src/main/java/net/bigeon/gclc/process/ForkTask.java

@@ -77,23 +77,30 @@ import net.bigeon.gclc.manager.ConsoleOutput;
 import net.bigeon.gclc.process.io.ConnectingConsoleInput;
 import net.bigeon.gclc.process.io.ConnectingConsoleOutput;
 
-/**
- * <p>
- * TODO
+/** A task that is made to run into a thread.
  *
  * @author Emmanuel Bigeon */
 public abstract class ForkTask implements Task {
 
+    /** THe listeners. */
     private final Set<InterruptionListener> listeners = new HashSet<>();
-    private boolean                         running   = false;
-
+    /** The running state. */
+    private boolean                         running   = true;
+    /** The running state. */
+    private boolean                         started   = false;
+    /** The connecting input for this task */
     protected final ConnectingConsoleInput  in        = new ConnectingConsoleInput();
+    /** The connecting output for this task */
     protected final ConnectingConsoleOutput out;
+    /** The exception of the run. */
     private CommandRunException             exception;
+    /** The synchronization lock. */
     private final Object                    runLock   = new Object();
 
-    /** @param lines the number of print to store in the output */
-    public ForkTask(int lines) {
+    /** Create the task.
+     * 
+     * @param lines the number of print to store in the output */
+    public ForkTask(final int lines) {
         out = new ConnectingConsoleOutput(ConnectingConsoleOutput.PERSIST, lines);
     }
 
@@ -105,6 +112,20 @@ public abstract class ForkTask implements Task {
         listeners.add(listener);
     }
 
+    /** Actually run the fork. */
+    protected abstract void doRun() throws CommandRunException;
+
+    /** Get the excepion that caused a failure.
+     *
+     * @return the exception */
+    public final CommandRunException getException() {
+        return exception;
+    }
+
+    protected final Object getRunningLock() {
+        return runLock;
+    }
+
     /* (non-Javadoc)
      * @see net.bigeon.gclc.process.Task#isRunning() */
     @Override
@@ -114,10 +135,13 @@ public abstract class ForkTask implements Task {
         }
     }
 
-    /** @param out the console output
+    /** Join the task.
+     * 
+     * @param out the console output
      * @param in the console input
-     * @param timeout the maximal wait (0 for ever) */
-    public final void join(final ConsoleOutput out, final ConsoleInput in, long timeout) {
+     * @param timeout the maximal time to join for (0 for ever) */
+    public final void join(final ConsoleOutput out, final ConsoleInput in,
+            final long timeout) {
         synchronized (runLock) {
             this.out.connect(out);
             this.in.connect(in);
@@ -126,7 +150,6 @@ public abstract class ForkTask implements Task {
                     runLock.wait(timeout);
                 }
             } catch (final InterruptedException e) {
-                // TODO log.
                 Thread.currentThread().interrupt();
             }
             this.out.disconnect();
@@ -148,7 +171,7 @@ public abstract class ForkTask implements Task {
     @Override
     public final void run() {
         synchronized (runLock) {
-            running = true;
+            started = true;
         }
         try {
             doRun();
@@ -162,16 +185,6 @@ public abstract class ForkTask implements Task {
         }
     }
 
-    /** Actually run the fork. */
-    protected abstract void doRun() throws CommandRunException;
-
-    /** Get the excepion that caused a failure.
-     * 
-     * @return the exception */
-    public final CommandRunException getException() {
-        return exception;
-    }
-
     /* (non-Javadoc)
      * @see net.bigeon.gclc.process.Task#setRunning(boolean) */
     @Override
@@ -182,7 +195,10 @@ public abstract class ForkTask implements Task {
         }
     }
 
-    protected final Object getRunningLock() {
-        return runLock;
+    /** @return the started */
+    public boolean isStarted() {
+        synchronized (runLock) {
+            return started;
+        }
     }
 }

gclc-process/src/main/java/net/bigeon/gclc/process/InterruptionListener.java

@@ -68,9 +68,10 @@ package net.bigeon.gclc.process;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-/** A listener for interruption
+/** A listener for interruption.
  *
  * @author Emmanuel Bigeon */
+@FunctionalInterface
 public interface InterruptionListener {
     /** Notification of an interuption of a listened object */
     void interrupted();

gclc-process/src/main/java/net/bigeon/gclc/process/ProcessAttachement.java

@@ -71,10 +71,7 @@ import net.bigeon.gclc.ApplicationAttachement;
 import net.bigeon.gclc.command.ICommandProvider;
 import net.bigeon.gclc.exception.InvalidCommandName;
 
-/**
- * @author Emmanuel Bigeon
- *
- */
+/** @author Emmanuel Bigeon */
 public class ProcessAttachement implements ApplicationAttachement {
 
     private final TaskPool pool;
@@ -86,8 +83,8 @@ public class ProcessAttachement implements ApplicationAttachement {
     }
 
     /* (non-Javadoc)
-     * @see fr.bigeon.gclc.ApplicationAttachement#attach(fr.bigeon.gclc.ConsoleApplication)
-     */
+     * @see fr.bigeon.gclc.ApplicationAttachement#attach(fr.bigeon.gclc.
+     * ConsoleApplication) */
     @Override
     public void attach(ICommandProvider application) throws InvalidCommandName {
         application.add(new ProcessKill("kill", pool));

gclc-process/src/main/java/net/bigeon/gclc/process/ProcessClear.java

@@ -69,7 +69,6 @@ package net.bigeon.gclc.process;
  * #L%
  */
 import net.bigeon.gclc.command.Command;
-import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
 
@@ -89,32 +88,30 @@ public final class ProcessClear extends Command {
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
-     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-     * java.lang.String[]) */
+     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
     @Override
     public void execute(final ConsoleOutput out, final ConsoleInput in,
-                        final String... args) throws CommandRunException {
-        for (final String id : pool.getPIDs()) {
+            final String... args) {
+        for (final Integer id : pool.getPIDs()) {
             if (!pool.get(id).isRunning()) {
                 pool.remove(id);
             }
         }
-        return;
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#tip() */
-    @SuppressWarnings("nls")
     @Override
     public String tip() {
-        return "Request a process to stop (softly)";
+        return "Remove non running processes from pool";
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.Command#usageDetail() */
     @Override
     protected String usageDetail() {
-        return null;
+        return "  All processes in the pool have their running status checked and the "
+                + "non running ones are removed from the list of tasks.";
     }
 
 }

gclc-process/src/main/java/net/bigeon/gclc/process/ProcessKill.java

@@ -70,6 +70,7 @@ package net.bigeon.gclc.process;
  */
 import net.bigeon.gclc.command.Command;
 import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
 
@@ -89,17 +90,19 @@ public final class ProcessKill extends Command {
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
-     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-     * java.lang.String[]) */
+     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
     @Override
     public void execute(final ConsoleOutput out, final ConsoleInput in,
             final String... args) throws CommandRunException {
-        pool.get(args[0]).setRunning(false);
+        if (args.length < 1) {
+            throw new CommandRunException(CommandRunExceptionType.USAGE,
+                    "A pid should be specified.");
+        }
+        pool.get(Integer.parseInt(args[0])).setRunning(false);
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#tip() */
-    @SuppressWarnings("nls")
     @Override
     public String tip() {
         return "Request a process to stop (softly)";
@@ -109,7 +112,13 @@ public final class ProcessKill extends Command {
      * @see fr.bigeon.gclc.command.Command#usageDetail() */
     @Override
     protected String usageDetail() {
-        return null;
+        return "  <pid> is the identification of the process to request the stop to in the pool";
     }
 
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.command.Command#usagePattern() */
+    @Override
+    protected String usagePattern() {
+        return super.usagePattern() + " <pid>";
+    }
 }

gclc-process/src/main/java/net/bigeon/gclc/process/ProcessList.java

@@ -95,20 +95,18 @@ public final class ProcessList extends Command {
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
-     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-     * java.lang.String[]) */
+     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
     @Override
     public void execute(final ConsoleOutput out, final ConsoleInput in,
             final String... args) throws CommandRunException {
-        final ArrayList<String> pids = new ArrayList<>(pool.getPIDs());
+        final ArrayList<Integer> pids = new ArrayList<>(pool.getPIDs());
         Collections.sort(pids);
-        for (final String string : pids) {
+        for (final Integer string : pids) {
             try {
                 out.println(MessageFormat.format("{0}\t{1}", string, //$NON-NLS-1$
                         pool.get(string).getName()));
             } catch (final IOException e) {
-                throw new CommandRunException(
-                        CommandRunExceptionType.INTERACTION,
+                throw new CommandRunException(CommandRunExceptionType.INTERACTION,
                         "Unable to communicate with user", e); //$NON-NLS-1$
             }
         }
@@ -117,7 +115,6 @@ public final class ProcessList extends Command {
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#tip() */
-    @SuppressWarnings("nls")
     @Override
     public String tip() {
         return "List all processes";
@@ -127,7 +124,7 @@ public final class ProcessList extends Command {
      * @see fr.bigeon.gclc.command.Command#usageDetail() */
     @Override
     protected String usageDetail() {
-        return null;
+        return "  No argument is considered by this command";
     }
 
 }

gclc-process/src/main/java/net/bigeon/gclc/process/Task.java

@@ -103,8 +103,8 @@ public interface Task extends Runnable {
     /** Set the running state.
      * <p>
      * This method should be only called by external objects with the false
-     * argument. Calling this method with true has unspecified behavior and
-     * could do nothing as well as restart the command for example.
+     * argument. Calling this method with true has unspecified behavior and could do
+     * nothing as well as restart the command for example.
      *
      * @param running the running state */
     void setRunning(boolean running);

gclc-process/src/main/java/net/bigeon/gclc/process/TaskPool.java

@@ -78,7 +78,7 @@ import java.util.Map;
  * @author Emmanuel Bigeon */
 public final class TaskPool {
     /** The running processes. */
-    private final Map<String, Task> running = new HashMap<>();
+    private final Map<Integer, Task> running = new HashMap<>();
     /** The count for process id. */
     private int                     count   = 0;
     /** The lock for pid attribution synchronization. */
@@ -91,41 +91,28 @@ public final class TaskPool {
     }
 
     /** Default constructor. */
-    public TaskPool(boolean autoClear) {
+    public TaskPool(final boolean autoClear) {
         this.autoClear = autoClear;
     }
 
-    /** Remove a task from the pool
-     * 
-     * @param pid the task id */
-    public void remove(String pid) {
-        synchronized (lock) {
-            running.remove(pid);
-            count = Math.min(count, Integer.parseInt(pid));
-        }
-    }
-
     /** Add a process in the pool.
      *
      * @param cmd the process
      * @return the pid */
-    public String add(final Task cmd) {
+    public int add(final Task cmd) {
         if (cmd == null) {
             throw new IllegalArgumentException("Task cannot be null"); //$NON-NLS-1$
         }
-        final String pid;
+        final int pid;
         synchronized (lock) {
             pid = getPID();
             running.put(pid, cmd);
         }
         if (autoClear) {
             cmd.addInterruptionListener(new InterruptionListener() {
-                @SuppressWarnings("synthetic-access")
                 @Override
                 public void interrupted() {
-                    synchronized (lock) {
                     remove(pid);
-                    }
                     cmd.rmInterruptionListener(this);
                 }
             });
@@ -137,21 +124,21 @@ public final class TaskPool {
      *
      * @param pid the task id
      * @return the task, if any, associated to this id */
-    public Task get(final String pid) {
+    public Task get(final int pid) {
         synchronized (lock) {
-            return running.get(pid);
+            return running.get(Integer.valueOf(pid));
         }
     }
 
     /** Get the next process id.
      *
      * @return the process id */
-    private String getPID() {
+    private int getPID() {
         synchronized (lock) {
-            String pid;
+            int pid;
             do {
-                pid = Integer.toString(count++);
-            } while (running.containsKey(pid));
+                pid = count++;
+            } while (running.containsKey(Integer.valueOf(pid)));
             return pid;
         }
     }
@@ -159,7 +146,7 @@ public final class TaskPool {
     /** Get the running processes' identifiers.
      *
      * @return the pids */
-    public Collection<String> getPIDs() {
+    public Collection<Integer> getPIDs() {
         return new HashSet<>(running.keySet());
     }
 
@@ -168,6 +155,16 @@ public final class TaskPool {
         return autoClear;
     }
 
+    /** Remove a task from the pool
+     *
+     * @param pid the task id */
+    public void remove(final int pid) {
+        synchronized (lock) {
+            running.remove(Integer.valueOf(pid));
+            count = Math.min(count, (pid));
+        }
+    }
+
     /** Request all task to stop running.
      * <p>
      * This call does not guaranty end of execution, it is up to the task

gclc-process/src/main/java/net/bigeon/gclc/process/TaskSpawner.java

@@ -86,7 +86,7 @@ public abstract class TaskSpawner extends Command {
     /** @param name the command name
      * @param pool the pool */
     public TaskSpawner(final String name, final TaskPool pool,
-                       ExecutorService threadPool) {
+                       final ExecutorService threadPool) {
         super(name);
         this.pool = pool;
         this.threadPool = threadPool;
@@ -97,19 +97,17 @@ public abstract class TaskSpawner extends Command {
      * @param args the arguments
      * @return the process to start and add to the pool
      * @throws CommandRunException if the task creation failed */
-    protected abstract Task createTask(ConsoleOutput out, ConsoleInput in,
-            String... args) throws CommandRunException;
+    protected abstract Task createTask(ConsoleOutput out, ConsoleInput in, String... args)
+            throws CommandRunException;
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
-     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-     * java.lang.String[]) */
+     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
     @Override
     public final void execute(final ConsoleOutput out, final ConsoleInput in,
             final String... args) throws CommandRunException {
         final Task task = createTask(out, in, args);
-        final Thread th = new Thread(task);
         pool.add(task);
-        threadPool.execute(th);
+        threadPool.execute(task);
     }
 }

gclc-process/src/main/java/net/bigeon/gclc/process/io/ConnectingConsoleInput.java

@@ -72,28 +72,72 @@ import java.util.logging.Level;
 import java.util.logging.Logger;
 
 import net.bigeon.gclc.manager.ConsoleInput;
+import net.bigeon.gclc.tools.ConstantString;
 import net.bigeon.gclc.tools.StringProvider;
 
-/** @author Emmanuel Bigeon */
+/** A console input that can be connected to and diconnected from.
+ * 
+ * @author Emmanuel Bigeon */
 public final class ConnectingConsoleInput implements ConsoleInput {
 
+    /** The empty string provider. */
+    private static final ConstantString EMPTY_STRING = new ConstantString("");
+    /** The logger. */
     private static final Logger         LOGGER       = Logger
             .getLogger(ConnectingConsoleInput.class.getName());
+
+    /** If the input is closed. */
     private boolean        close          = false;
-    private StringProvider      prompt;
-    private boolean             prompting;
+    /** The prompt string. */
+    private StringProvider prompt         = EMPTY_STRING;
+    /** If the input is currently in prompting state.
+     * <p>
+     * To change it you should be in a promptLock. */
+    private boolean        prompting      = false;
+    /** The synchronization lock for the prompting status. */
     private final Object   promptLock     = new Object();
+    /** The synchronization lock for the connection status. */
     private final Object   connectionLock = new Object();
-    private ConsoleInput        connected;
-    private boolean             disconnection;
+    /** The connected console input.
+     * <p>
+     * To use it, you should be in a promptLock and connectionLock. */
+    private ConsoleInput   connected      = null;
+    /** The connection state.
+     * <p>
+     * To read or modify it, you should be in a connectionLock synchronize block. */
+    private boolean        disconnection  = false;
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#close() */
     @Override
-    public void close() throws IOException {
+    public void close() {
         close = true;
     }
 
+    /** Connect an input.
+     * 
+     * @param input the input to connect */
+    public void connect(final ConsoleInput input) {
+        disconnect();
+        synchronized (promptLock) {
+            connected = input;
+            promptLock.notifyAll();
+        }
+    }
+
+    /** Disconnect the current input. */
+    public void disconnect() {
+        synchronized (promptLock) {
+            synchronized (connectionLock) {
+                if (connected != null) {
+                    disconnection = true;
+                    connected.interruptPrompt();
+                    connected = null;
+                }
+            }
+        }
+    }
+
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#getPrompt() */
     @Override
@@ -128,31 +172,30 @@ public final class ConnectingConsoleInput implements ConsoleInput {
         return prompt(prompt.apply());
     }
 
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleInput#setPrompt(fr.bigeon.gclc.tools.
-     * StringProvider) */
-    @Override
-    public void setPrompt(StringProvider string) {
-        prompt = string;
-    }
-
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(long) */
     @Override
-    public String prompt(long timeout) throws IOException {
+    public String prompt(final long timeout) throws IOException {
         return prompt(prompt.apply(), timeout);
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String) */
     @Override
-    public String prompt(String message) throws IOException {
+    public String prompt(final String message) throws IOException {
         synchronized (promptLock) {
             prompting = true;
         }
-        while (prompting) {
+        while (true) {
             synchronized (promptLock) {
-                if (connected == null) {
+                if (!prompting) {
+                    return null;
+                }
+                boolean connect;
+                synchronized (connectionLock) {
+                    connect = connected != null;
+                }
+                if (!connect) {
                     try {
                         promptLock.wait();
                     } catch (final InterruptedException e) {
@@ -160,24 +203,25 @@ public final class ConnectingConsoleInput implements ConsoleInput {
                         Thread.currentThread().interrupt();
                     }
                 } else {
-                    final String res = connected.prompt(message);
                     synchronized (connectionLock) {
+                        final String res = connected.prompt(message);
                         if (disconnection) {
                             disconnection = false;
                         } else if (prompting) {
                             return res;
+                        } else {
+                            // prompt interrupted, lose the result.
                         }
                     }
                 }
             }
         }
-        return null;
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String, long) */
     @Override
-    public String prompt(String message, long timeout) throws IOException {
+    public String prompt(final String message, final long timeout) throws IOException {
         if (timeout <= 0) {
             return prompt(message);
         }
@@ -185,28 +229,37 @@ public final class ConnectingConsoleInput implements ConsoleInput {
         synchronized (promptLock) {
             prompting = true;
         }
-        while (prompting) {
+        do {
             synchronized (promptLock) {
-                if (connected == null) {
+                if (!prompting) {
+                    return null;
+                }
+                boolean connect;
+                synchronized (connectionLock) {
+                    connect = connected != null;
+                }
+                if (!connect) {
                     try {
-                        promptLock.wait();
+                        promptLock.wait(timeout);
                     } catch (final InterruptedException e) {
                         LOGGER.log(Level.WARNING, "Inerruption of console thread", e);
                         Thread.currentThread().interrupt();
                     }
                 } else {
+                    synchronized (connectionLock) {
                         final String res = connected.prompt(message,
                                 end - System.currentTimeMillis());
-                    synchronized (connectionLock) {
                         if (disconnection) {
                             disconnection = false;
                         } else if (prompting) {
                             return res;
+                        } else {
+                            // prompt interrupted, lose the result.
                         }
                     }
                 }
             }
-        }
+        } while (System.currentTimeMillis() < end);
         return null;
     }
 
@@ -214,31 +267,14 @@ public final class ConnectingConsoleInput implements ConsoleInput {
      * @see fr.bigeon.gclc.manager.ConsoleInput#setPrompt(java.lang.String) */
     @Override
     public void setPrompt(final String prompt) {
-        this.prompt = new StringProvider() {
+        this.prompt = new ConstantString(prompt);
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.manager.ConsoleInput#setPrompt(fr.bigeon.gclc.tools.
+     * StringProvider) */
     @Override
-            public String apply() {
-                return prompt;
-            }
-        };
-    }
-
-    public void connect(ConsoleInput input) {
-        disconnect();
-        synchronized (promptLock) {
-            connected = input;
-            promptLock.notifyAll();
-        }
-    }
-
-    public void disconnect() {
-        synchronized (connectionLock) {
-            if (connected != null) {
-                disconnection = true;
-                synchronized (promptLock) {
-                    connected.interruptPrompt();
-                }
-                connected = null;
-            }
-        }
+    public void setPrompt(final StringProvider string) {
+        prompt = string;
     }
 }

gclc-process/src/main/java/net/bigeon/gclc/process/io/ConnectingConsoleOutput.java

@@ -75,26 +75,37 @@ import java.util.logging.Logger;
 
 import net.bigeon.gclc.manager.ConsoleOutput;
 
-/** @author Emmanuel Bigeon */
+/** A console output that can be connected and disconnected from.
+ * 
+ * @author Emmanuel Bigeon */
 public class ConnectingConsoleOutput implements ConsoleOutput {
 
+    /** The logger. */
     private static final Logger LOGGER  = Logger
             .getLogger(ConnectingConsoleOutput.class.getName());
-    /** If the undelivered message should be stored. */
+    /** The flag indicating that the un-delivered message should be stored. */
     public static final int     QUEUE   = 1;
-    /** If the messages should be stored in all cases. */
+    /** The flag indicating that the messages should be stored in all cases. */
     public static final int     PERSIST = 1 << 1;
+    /** The connected output console */
     private ConsoleOutput       output;
+    /** If the console is closed */
     private boolean             close   = false;
 
+    /** If the messages should be stored. */
     private final boolean       persistent;
+    /** If the message should be stored until delivery. */
     private final boolean       queued;
+    /** The messages. */
     private final Deque<String> messages;
+    /** The number of stored messages. */
     private final int           lines;
 
-    /** @param style the type of redirected output
+    /** Create the console output.
+     * 
+     * @param style the type of redirected output
      * @param lines the number of lines to store */
-    public ConnectingConsoleOutput(int style, int lines) {
+    public ConnectingConsoleOutput(final int style, final int lines) {
         super();
         this.lines = lines;
         queued = (style & QUEUE) != 0;
@@ -106,7 +117,10 @@ public class ConnectingConsoleOutput implements ConsoleOutput {
         }
     }
 
-    private synchronized void addMessage(String text) {
+    /** Add a message.
+     * 
+     * @param text the message */
+    private synchronized void addMessage(final String text) {
         if (persistent || queued && output == null) {
             if (messages.size() == lines) {
                 messages.poll();
@@ -130,35 +144,10 @@ public class ConnectingConsoleOutput implements ConsoleOutput {
         close = true;
     }
 
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleOutput#isClosed() */
-    @Override
-    public boolean isClosed() {
-        return close;
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleOutput#print(java.lang.String) */
-    @Override
-    public void print(String text) {
-        addMessage(text);
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleOutput#println() */
-    @Override
-    public void println() {
-        addMessage(System.lineSeparator());
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleOutput#println(java.lang.String) */
-    @Override
-    public void println(String message) {
-        addMessage(message + System.lineSeparator());
-    }
-
-    public synchronized void connect(ConsoleOutput output) {
+    /** Connect an output.
+     * 
+     * @param output the output */
+    public synchronized void connect(final ConsoleOutput output) {
         this.output = output;
         for (final String string : messages) {
             try {
@@ -173,7 +162,36 @@ public class ConnectingConsoleOutput implements ConsoleOutput {
         }
     }
 
+    /** Disconnect the currently connected output. */
     public synchronized void disconnect() {
         output = null;
     }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.manager.ConsoleOutput#isClosed() */
+    @Override
+    public boolean isClosed() {
+        return close;
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.manager.ConsoleOutput#print(java.lang.String) */
+    @Override
+    public void print(final String text) {
+        addMessage(text);
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.manager.ConsoleOutput#println() */
+    @Override
+    public void println() {
+        addMessage(System.lineSeparator());
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.manager.ConsoleOutput#println(java.lang.String) */
+    @Override
+    public void println(final String message) {
+        addMessage(message + System.lineSeparator());
+    }
 }

gclc-process/src/main/java/net/bigeon/gclc/process/io/package-info.java

@@ -0,0 +1,42 @@
+/** Connecting Input and Output related classes.
+ * <p>
+ * This package groups the Connecting classes that are used to be able to put
+ * the tasks in background and foreground and reattach the actual console input
+ * and outputs to the tasks' ones.
+ * 
+ * @author Emmanuel Bigeon */
+package net.bigeon.gclc.process.io;
+
+/*-
+ * #%L
+ * process
+ * %%
+ * Copyright (C) 2014 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */

gclc-process/src/main/java/net/bigeon/gclc/process/package-info.java

@@ -0,0 +1,48 @@
+/** This package defines elements for processes inside a console application.
+ * <p>
+ * Processes are tasks that are run in background of the application. One may
+ * want to temporarily connect to such a task (through
+ * {@link net.bigeon.gclc.process.CommandForeground}), or verify it's actual
+ * running status (through {@link net.bigeon.gclc.process.ProcessList}. One may
+ * also want to start such a process (through
+ * {@link net.bigeon.gclc.process.ForkCommandTask}) or close it (through
+ * {@link net.bigeon.gclc.process.ProcessKill}).
+ * <p>
+ * The task list can be managed by several commands to list or clear it.
+ * 
+ * @author Emmanuel Bigeon */
+package net.bigeon.gclc.process;
+
+/*-
+ * #%L
+ * process
+ * %%
+ * Copyright (C) 2014 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */

gclc-process/src/test/java/net/bigeon/gclc/process/CommandForegroundTest.java

@@ -0,0 +1,90 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.process;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.fail;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
+import net.bigeon.gclc.process.mocks.ForkTaskMock;
+
+/**
+ * @author Emmanuel Bigeon
+ *
+ */
+public class CommandForegroundTest {
+
+    /**
+     * Test method for {@link net.bigeon.gclc.process.CommandForeground#CommandForeground(java.lang.String, net.bigeon.gclc.process.TaskPool)}.
+     */
+    @Test
+    public void testCommandForeground() {
+        final CommandForeground pl = new CommandForeground("fg", new TaskPool());
+        assertEquals("Command name should be kept as specified", "fg",
+                pl.getCommandName());
+    }
+
+    /** Test method for
+     * {@link net.bigeon.gclc.process.CommandForeground#doExecute(net.bigeon.gclc.manager.ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput, net.bigeon.gclc.command.CommandParameters)}.
+     * 
+     * @throws CommandRunException if a valid command failed */
+    @Test
+    public void testDoExecuteConsoleOutputConsoleInputCommandParameters()
+            throws CommandRunException {
+        final TaskPool pool = new TaskPool(false);
+        final CommandForeground pl = new CommandForeground("fg", pool);
+
+        try {
+            pl.execute(null, null, "-delai", "1");
+            fail("PID should be mandatory");
+        } catch (final CommandRunException e) {
+            assertEquals("No pid specified is a usage error",
+                    CommandRunExceptionType.USAGE, e.getType());
+        }
+        try {
+            pl.execute(null, null, "-pid", "invalid", "-delai", "1");
+            fail("PID should be a number");
+        } catch (final CommandRunException e) {
+            assertEquals("Invalid pid specified is a usage error",
+                    CommandRunExceptionType.USAGE, e.getType());
+        }
+        try {
+            pl.execute(null, null, "-pid", "2", "-delai", "1");
+            fail("PID should exist");
+        } catch (final CommandRunException e) {
+            assertEquals("Inexistent pid specified is a run error",
+                    CommandRunExceptionType.EXECUTION, e.getType());
+        }
+
+        final ForkTaskMock cmd = new ForkTaskMock();
+        final int id = pool.add(cmd);
+        pl.execute(null, null, "-pid", Integer.toString(id), "-delai", "1");
+
+        try {
+            pl.execute(null, null, "-pid", Integer.toString(id), "-delai", "-1");
+            fail("delai should be a posistive number");
+        } catch (final CommandRunException e) {
+            assertEquals("Negative delai specified is a usage error",
+                    CommandRunExceptionType.USAGE, e.getType());
+        }
+
+        assert cmd.getRunCall() == 0;
+    }
+
+    /**
+     * Test method for {@link net.bigeon.gclc.process.CommandForeground#tip()}.
+     */
+    @Test
+    public void testTip() {
+        final CommandForeground pl = new CommandForeground("fg", new TaskPool());
+        assertNotNull("Command tip should be defined", pl.tip());
+        assertNotNull("Command usage should be defined", pl.usagePattern());
+        assertNotNull("Command usage should be defined", pl.usageDetail());
+    }
+
+}

gclc-process/src/test/java/net/bigeon/gclc/process/CommandForkTest.java

@@ -0,0 +1,86 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.process;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.fail;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.command.ICommandProvider;
+import net.bigeon.gclc.command.SubedCommand;
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
+import net.bigeon.gclc.exception.InvalidCommandName;
+import net.bigeon.gclc.process.mocks.CommandMock;
+
+/**
+ * @author Emmanuel Bigeon
+ *
+ */
+public class CommandForkTest {
+
+    /**
+     * Test method for {@link net.bigeon.gclc.process.CommandFork#CommandFork(java.lang.String, net.bigeon.gclc.command.ICommandProvider, net.bigeon.gclc.process.TaskPool)}.
+     */
+    @Test
+    public void testCommandForkStringICommandProviderTaskPool() {
+        final TaskPool pool = new TaskPool();
+        final ICommandProvider provider = new SubedCommand("test");
+        final CommandFork pl = new CommandFork("fork", provider, pool);
+        assertEquals("Command name should be preserved", "fork", pl.getCommandName());
+    }
+
+    /** Test method for
+     * {@link net.bigeon.gclc.process.CommandFork#execute(net.bigeon.gclc.manager.ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput, java.lang.String[])}.
+     * 
+     * @throws InvalidCommandName if the test init failed
+     * @throws CommandRunException */
+    @Test
+    public void testExecute() throws InvalidCommandName, CommandRunException {
+        final TaskPool pool = new TaskPool(false);
+        final ICommandProvider provider = new SubedCommand("test");
+        final CommandFork pl = new CommandFork("fork", provider, pool);
+
+        try {
+            pl.execute(null, null);
+            fail("No command specified should through an exception");
+        } catch (final CommandRunException e) {
+            assertEquals("No command specified is a usage error",
+                    CommandRunExceptionType.USAGE, e.getType());
+        }
+        try {
+            pl.execute(null, null, "invalid");
+            fail("Invalid command specified should through an exception");
+        } catch (final CommandRunException e) {
+            assertEquals("Invalid command specified is a usage error",
+                    CommandRunExceptionType.USAGE, e.getType());
+        }
+
+        final CommandMock mock = new CommandMock();
+        provider.add(mock);
+
+        pl.execute(null, null, mock.getCommandName());
+
+        final ForkTask task = (ForkTask) pool.get(pool.getPIDs().iterator().next());
+        task.join(null, null, 1000);
+
+        assertEquals("Command should be executed when forked", 1, mock.getExecuteCall());
+    }
+
+    /**
+     * Test method for {@link net.bigeon.gclc.process.CommandFork#tip()}.
+     */
+    @Test
+    public void testTip() {
+        final TaskPool pool = new TaskPool();
+        final ICommandProvider provider = new SubedCommand("test");
+        final CommandFork pl = new CommandFork("fork", provider, pool);
+        assertNotNull("Command tip should be defined", pl.tip());
+        assertNotNull("Command usage should be defined", pl.usagePattern());
+        assertNotNull("Command usage should be defined", pl.usageDetail());
+    }
+
+}

gclc-process/src/test/java/net/bigeon/gclc/process/ForkCommandTaskTest.java

@@ -0,0 +1,35 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.process;
+
+import static org.junit.Assert.assertEquals;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.process.mocks.CommandMock;
+
+/**
+ * @author Emmanuel Bigeon
+ *
+ */
+public class ForkCommandTaskTest {
+
+    /** Test method for
+     * {@link net.bigeon.gclc.process.ForkCommandTask#ForkCommandTask(net.bigeon.gclc.command.ICommand, java.lang.String[], int)}.
+     * 
+     * @throws InterruptedException if there is an error in the thread join */
+    @Test
+    public void testForkCommandTask() throws InterruptedException {
+        final CommandMock cmd = new CommandMock();
+        final ForkCommandTask task = new ForkCommandTask(cmd, new String[0], 1);
+
+        final Thread th = new Thread(task);
+        th.start();
+        th.join();
+
+        assertEquals("Task should be executed once", 1, cmd.getExecuteCall());
+        assertEquals("Names should be consistent between task and command",
+                cmd.getCommandName(), task.getName());
+    }
+}

gclc-process/src/test/java/net/bigeon/gclc/process/ForkTaskTest.java

@@ -0,0 +1,62 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.process;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+
+import java.io.IOException;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
+import net.bigeon.gclc.utils.PipedConsoleInput;
+import net.bigeon.gclc.utils.PipedConsoleOutput;
+
+/** @author Emmanuel Bigeon */
+public class ForkTaskTest {
+
+    @Test
+    public void testGenericForkTask() throws IOException {
+        final ForkTask task = new ForkTask(5) {
+
+            @Override
+            public String getName() {
+                return "name";
+            }
+
+            @Override
+            protected void doRun() throws CommandRunException {
+                String msg;
+                try {
+                    msg = in.prompt();
+                } catch (final IOException e) {
+                    throw new CommandRunException(CommandRunExceptionType.INTERACTION,
+                            "Unable to prompt user");
+                }
+                if ("ok".equals(msg)) {
+                    out.println("Message");
+                } else {
+                    out.println("fail");
+                }
+            }
+        };
+
+        final Thread execThread = new Thread(task);
+        execThread.start();
+
+        try (PipedConsoleOutput pco = new PipedConsoleOutput();
+                PipedConsoleInput pci = new PipedConsoleInput(null)) {
+            pci.type("ok");
+            while (!pco.available()) {
+                task.join(pco, pci, 1000);
+            }
+            assertEquals("Execution should work", "Message", pco.readNextLine());
+        }
+        assertFalse("Running state should be updated by task on its completion",
+                task.isRunning());
+    }
+
+}

gclc-process/src/test/java/net/bigeon/gclc/process/ProcessClearTest.java

@@ -0,0 +1,53 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.process;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.process.mocks.TaskMock;
+
+/**
+ * @author Emmanuel Bigeon
+ *
+ */
+public class ProcessClearTest {
+
+    /**
+     * Test method for {@link net.bigeon.gclc.process.ProcessClear#ProcessClear(java.lang.String, net.bigeon.gclc.process.TaskPool)}.
+     */
+    @Test
+    public void testProcessClear() {
+        final ProcessClear pl = new ProcessClear("clear", new TaskPool());
+        assertEquals("Command name should be kept as specified", "clear",
+                pl.getCommandName());
+    }
+
+    /**
+     * Test method for {@link net.bigeon.gclc.process.ProcessClear#execute(net.bigeon.gclc.manager.ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput, java.lang.String[])}.
+     */
+    @Test
+    public void testExecute() {
+        final TaskPool pool = new TaskPool();
+        final TaskMock task = new TaskMock();
+        pool.add(task);
+        final ProcessClear pl = new ProcessClear("clear", pool);
+        pl.execute(null, null);
+        assertTrue("cleared pool should be empty", pool.getPIDs().isEmpty());
+    }
+
+    /**
+     * Test method for {@link net.bigeon.gclc.process.ProcessClear#tip()}.
+     */
+    @Test
+    public void testTip() {
+        final ProcessClear pl = new ProcessClear("clear", new TaskPool());
+        assertNotNull("Command tip should be defined", pl.tip());
+        assertNotNull("Command usage should be defined", pl.usageDetail());
+    }
+
+}

gclc-process/src/test/java/net/bigeon/gclc/process/ProcessKillTest.java

@@ -0,0 +1,65 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.process;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.fail;
+
+import java.io.IOException;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
+import net.bigeon.gclc.process.mocks.TaskMock;
+
+/** @author Emmanuel Bigeon */
+public class ProcessKillTest {
+
+    /** Test method for
+     * {@link net.bigeon.gclc.process.ProcessKill#ProcessKill(java.lang.String, net.bigeon.gclc.process.TaskPool)}. */
+    @Test
+    public void testProcessKill() {
+        final ProcessList pl = new ProcessList("kill", new TaskPool());
+        assertEquals("Command name should be kept as specified", "kill",
+                pl.getCommandName());
+    }
+
+    /** Test method for
+     * {@link net.bigeon.gclc.process.ProcessKill#execute(net.bigeon.gclc.manager.ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput, java.lang.String[])}.
+     * 
+     * @throws CommandRunException if an error occured in test
+     * @throws IOException if an error occured with the console output */
+    @Test
+    public void testExecute() throws CommandRunException {
+        final TaskPool pool = new TaskPool();
+        final TaskMock task = new TaskMock();
+        final int id = pool.add(task);
+        final ProcessKill pl = new ProcessKill("kill", pool);
+        task.reset();
+        try {
+            pl.execute(null, null);
+            fail("Execution with no pid should raise exception");
+        } catch (final CommandRunException e) {
+            assertEquals("Usage exception expected", CommandRunExceptionType.USAGE,
+                    e.getType());
+        }
+        pl.execute(null, null, Integer.toString(id));
+        assertEquals("Task should be shutdown", 1, task.getNumberSetRunning());
+        task.reset();
+        pl.execute(null, null, Integer.toString(id));
+        assertEquals("Task should be shutdown", 1, task.getNumberSetRunning());
+    }
+
+    /** Test method for {@link net.bigeon.gclc.process.ProcessKill#tip()}. */
+    @Test
+    public void testTip() {
+        final ProcessKill pl = new ProcessKill("kill", new TaskPool());
+        assertNotNull("Command tip should be defined", pl.tip());
+        assertNotNull("Command usage should be defined", pl.usagePattern());
+        assertNotNull("Command usage should be defined", pl.usageDetail());
+    }
+
+}

gclc-process/src/test/java/net/bigeon/gclc/process/ProcessListTest.java

@@ -0,0 +1,67 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.process;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
+
+import java.io.IOException;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
+import net.bigeon.gclc.process.mocks.TaskMock;
+import net.bigeon.gclc.utils.PipedConsoleOutput;
+
+/** @author Emmanuel Bigeon */
+public class ProcessListTest {
+
+    /** Test method for
+     * {@link net.bigeon.gclc.process.ProcessList#ProcessList(java.lang.String, net.bigeon.gclc.process.TaskPool)}. */
+    @Test
+    public void testProcessList() {
+        final ProcessList pl = new ProcessList("list", new TaskPool());
+        assertEquals("Command name should be kept as specified", "list",
+                pl.getCommandName());
+    }
+
+    /** Test method for
+     * {@link net.bigeon.gclc.process.ProcessList#execute(net.bigeon.gclc.manager.ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput, java.lang.String[])}.
+     * 
+     * @throws CommandRunException if an error occured
+     * @throws IOException if an error in initialization of console occured */
+    @Test
+    public void testExecute() throws CommandRunException, IOException {
+        final TaskPool pool = new TaskPool();
+        final TaskMock task = new TaskMock();
+        pool.add(task);
+        final ProcessList pl = new ProcessList("list", pool);
+        final PipedConsoleOutput out = new PipedConsoleOutput();
+        pl.execute(out, null);
+
+        assertTrue("Task should be contained in list",
+                out.readNextLine().contains(task.getName()));
+
+        out.close();
+
+        try {
+            pl.execute(out, null);
+            fail("Printing element to closed console should raise interaction exception");
+        } catch (final CommandRunException e) {
+            assertEquals("Console printing error should be an interaction error",
+                    CommandRunExceptionType.INTERACTION, e.getType());
+        }
+    }
+
+    /** Test method for {@link net.bigeon.gclc.process.ProcessList#tip()}. */
+    @Test
+    public void testTip() {
+        final ProcessList pl = new ProcessList("list", new TaskPool());
+        assertNotNull("Command tip should be defined", pl.tip());
+        assertNotNull("Command usage should be defined", pl.usageDetail());
+    }
+}

gclc-process/src/test/java/net/bigeon/gclc/process/TaskPoolTest.java

@@ -0,0 +1,77 @@
+package net.bigeon.gclc.process;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertTrue;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.process.mocks.TaskMock;
+
+public class TaskPoolTest {
+
+    public TaskMock task = new TaskMock();
+    public TaskPool pool = new TaskPool(false);
+
+    @Test
+    public void testTaskPool() {
+        assertFalse("task pool should not auto clear if specified",
+                pool.isAutoClearing());
+        pool = new TaskPool();
+        assertTrue("Default task pool should not auto clear", pool.isAutoClearing());
+    }
+
+    @Test
+    public void testAdd() {
+        final int id = pool.add(task);
+        assertTrue("Added task pid should be in pool", pool.getPIDs().contains(id));
+        assertEquals("Added task should be in pool", task, pool.get(id));
+    }
+
+    @Test
+    public void testGet() {
+        assertNull("Inexistent task id should provide null as task in pool",
+                pool.get(60));
+        final int id = pool.add(task);
+        assertEquals("Added task should be in pool at its id", task, pool.get(id));
+        assertNull("Added task should be in pool at its id", pool.get(id + 1));
+    }
+
+    @Test
+    public void testGetPIDs() {
+        assertTrue("new pool should be empty", pool.getPIDs().isEmpty());
+        final int id = pool.add(task);
+        assertNull("Added task should be in pool at its id", pool.get(id + 1));
+        assertEquals(
+                "Pool task addition should change the pid collection size accordingly", 1,
+                pool.getPIDs().size());
+    }
+
+    @Test
+    public void testRemove() {
+        assertTrue("new pool should be empty", pool.getPIDs().isEmpty());
+        pool.remove(60);
+        assertTrue("removal of inexistent task should be NOP", pool.getPIDs().isEmpty());
+        final int id = pool.add(task);
+        assertEquals(
+                "Pool task addition should change the pid collection size accordingly", 1,
+                pool.getPIDs().size());
+        pool.remove(id + 1);
+        assertEquals("removal of inexistent task should be NOP", 1,
+                pool.getPIDs().size());
+        pool.remove(id);
+        assertTrue("removal of task should reduce the number of tasks",
+                pool.getPIDs().isEmpty());
+    }
+
+    @Test
+    public void testShutdown() {
+        final int id = pool.add(task);
+        task.reset();
+        pool.shutdown();
+        assertEquals("Shutdown should request shut down of tasks, only once", 1,
+                task.getNumberSetRunning());
+    }
+
+}

gclc-process/src/test/java/net/bigeon/gclc/process/io/ConnectingConsoleInputTest.java

@@ -0,0 +1,65 @@
+package net.bigeon.gclc.process.io;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertTrue;
+
+import java.io.IOException;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.tools.ConstantString;
+import net.bigeon.gclc.utils.PipedConsoleInput;
+
+public class ConnectingConsoleInputTest {
+
+    @Test
+    public void testConnect() throws IOException {
+        final ConnectingConsoleInput in = new ConnectingConsoleInput();
+        final PipedConsoleInput pci = new PipedConsoleInput(null);
+        assertNull("Inputs should not transmit before connection", in.prompt(1000));
+        in.connect(pci);
+        pci.type("Some message");
+        assertEquals("Messages should be buffered", "Some message", in.prompt());
+    }
+
+    @Test
+    public void testDisconnect() throws IOException {
+        final ConnectingConsoleInput in = new ConnectingConsoleInput();
+        final PipedConsoleInput pci = new PipedConsoleInput(null);
+        assertNull("Inputs should not transmit before connection", in.prompt(1000));
+        in.connect(pci);
+        pci.type("Some message");
+        assertEquals("Messages should be buffered", "Some message", in.prompt());
+        in.disconnect();
+        assertNull("Disconnected inputs should not transmit anymore", in.prompt(1000));
+    }
+
+    @Test
+    public void testIsClosed() throws IOException {
+        final ConnectingConsoleInput in = new ConnectingConsoleInput();
+        assertFalse("New input should not be closed", in.isClosed());
+        final PipedConsoleInput pci = new PipedConsoleInput(null);
+        in.connect(pci);
+        pci.close();
+        assertFalse("Connected input closing should not close connecting", in.isClosed());
+        final PipedConsoleInput pci2 = new PipedConsoleInput(null);
+        in.connect(pci2);
+        in.close();
+        assertTrue("Input should close on request", in.isClosed());
+        assertFalse("Connected input should not be closed by connecting closure",
+                pci2.isClosed());
+    }
+
+    @Test
+    public void testCoveragePrompt() {
+        final ConnectingConsoleInput in = new ConnectingConsoleInput();
+        in.setPrompt("test");
+        assertEquals("Prompt should be set correctly", "test",
+                ((ConstantString) in.getPrompt()).apply());
+        final ConstantString prompt = new ConstantString("other");
+        in.setPrompt(prompt);
+        assertEquals("Prompt should be set correctly", prompt, in.getPrompt());
+    }
+}

gclc-process/src/test/java/net/bigeon/gclc/process/io/ConnectingConsoleOutputTest.java

@@ -0,0 +1,88 @@
+package net.bigeon.gclc.process.io;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+import java.io.IOException;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.utils.PipedConsoleOutput;
+
+public class ConnectingConsoleOutputTest {
+
+    @Test
+    public void testConnect() throws IOException {
+        final ConnectingConsoleOutput out = new ConnectingConsoleOutput(
+                ConnectingConsoleOutput.QUEUE, 10);
+        final PipedConsoleOutput output = new PipedConsoleOutput();
+        out.println("Some message");
+        assertFalse("Message should not be transmitted", output.available());
+        // Connect
+        out.connect(output);
+
+        // Connected outputs should receive data
+        final String res = output.readNextLine();
+        assertEquals("Message should be correctly transmitted", "Some message", res);
+    }
+
+    @Test
+    public void testDisconnect() throws IOException {
+        final ConnectingConsoleOutput out = new ConnectingConsoleOutput(
+                ConnectingConsoleOutput.QUEUE, 10);
+        final PipedConsoleOutput output = new PipedConsoleOutput();
+        // NOP disconnection when no connection
+        out.disconnect();
+        // Connect
+        out.connect(output);
+
+        // Connected outputs should receive data
+        out.println("Some message");
+        final String res = output.readNextLine();
+        assertEquals("Message should be correctly transmitted", "Some message", res);
+        out.disconnect();
+        out.println("Some message");
+        assertFalse("Message should not be transmitted", output.available());
+    }
+
+    @Test
+    public void testIsClosed() throws IOException {
+        final ConnectingConsoleOutput out = new ConnectingConsoleOutput(
+                ConnectingConsoleOutput.QUEUE, 10);
+        final PipedConsoleOutput output = new PipedConsoleOutput();
+        assertFalse("New output should not be closed", out.isClosed());
+        // Connect
+        out.connect(output);
+
+        output.close();
+
+        assertFalse("Output should not close when connected outputs do", out.isClosed());
+        final PipedConsoleOutput output2 = new PipedConsoleOutput();
+        out.connect(output2);
+        out.close();
+        assertTrue("Output should close on request", out.isClosed());
+        assertFalse("Output should not close connected output when they do",
+                output2.isClosed());
+    }
+
+    @Test
+    public void testPrint() throws IOException {
+        final ConnectingConsoleOutput out = new ConnectingConsoleOutput(
+                ConnectingConsoleOutput.PERSIST, 10);
+
+        out.print("Some text");
+        out.println();
+        final PipedConsoleOutput output = new PipedConsoleOutput();
+        out.connect(output);
+        String res = output.readNextLine();
+        assertEquals("Message should be correctly transmitted", "Some text", res);
+        out.disconnect();
+
+        // Persist should be reprinted on any connect
+        out.connect(output);
+        res = output.readNextLine();
+        assertEquals("Message should be correctly transmitted", "Some text", res);
+        out.disconnect();
+    }
+}

gclc-process/src/test/java/net/bigeon/gclc/process/mocks/CommandMock.java

@@ -0,0 +1,60 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.process.mocks;
+
+import java.io.IOException;
+
+import net.bigeon.gclc.command.ICommand;
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.manager.ConsoleInput;
+import net.bigeon.gclc.manager.ConsoleOutput;
+
+/** @author Emmanuel Bigeon */
+public class CommandMock implements ICommand {
+
+    private int executeCall     = 0;
+    private int tipCall         = 0;
+    private int helpCall        = 0;
+    private int commandNameCall = 0;
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.command.ICommand#execute(net.bigeon.gclc.manager.
+     * ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
+    @Override
+    public void execute(final ConsoleOutput out, final ConsoleInput in,
+            final String... args) throws CommandRunException {
+        executeCall++;
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.command.ICommand#getCommandName() */
+    @Override
+    public String getCommandName() {
+        commandNameCall++;
+        return "name";
+    }
+
+    /* (non-Javadoc)
+     * @see
+     * net.bigeon.gclc.command.ICommand#help(net.bigeon.gclc.manager.ConsoleOutput,
+     * java.lang.String[]) */
+    @Override
+    public void help(final ConsoleOutput output, final String... args)
+            throws IOException {
+        helpCall++;
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.command.ICommand#tip() */
+    @Override
+    public String tip() {
+        tipCall++;
+        return "tip";
+    }
+
+    /** @return the executeCall */
+    public int getExecuteCall() {
+        return executeCall;
+    }
+}

gclc-process/src/test/java/net/bigeon/gclc/process/mocks/ForkTaskMock.java

@@ -0,0 +1,43 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.process.mocks;
+
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.process.ForkTask;
+import net.bigeon.gclc.process.Task;
+
+/**
+ * @author Emmanuel Bigeon
+ *
+ */
+public class ForkTaskMock extends ForkTask implements Task {
+
+    private int runCall = 0;
+
+    /** @param lines */
+    public ForkTaskMock() {
+        super(10);
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.process.Task#getName()
+     */
+    @Override
+    public String getName() {
+        return "name";
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.process.ForkTask#doRun()
+     */
+    @Override
+    protected void doRun() throws CommandRunException {
+        runCall++;
+    }
+
+    /** @return the runCall */
+    public int getRunCall() {
+        return runCall;
+    }
+}

gclc-process/src/test/java/net/bigeon/gclc/process/mocks/TaskMock.java

@@ -0,0 +1,57 @@
+package net.bigeon.gclc.process.mocks;
+
+import net.bigeon.gclc.process.InterruptionListener;
+import net.bigeon.gclc.process.Task;
+
+public class TaskMock implements Task {
+
+    private int runCall        = 0;
+    private int setRunningCall = 0;
+
+    @Override
+    public void run() {
+        runCall++;
+    }
+
+    @Override
+    public void addInterruptionListener(final InterruptionListener listener) {
+        // TODO Auto-generated method stub
+
+    }
+
+    @Override
+    public String getName() {
+        return "name";
+    }
+
+    @Override
+    public boolean isRunning() {
+        // TODO Auto-generated method stub
+        return false;
+    }
+
+    @Override
+    public void rmInterruptionListener(final InterruptionListener listener) {
+        // TODO Auto-generated method stub
+
+    }
+
+    @Override
+    public void setRunning(final boolean running) {
+        setRunningCall++;
+    }
+
+    /**
+     * 
+     */
+    public void reset() {
+        runCall = 0;
+        setRunningCall = 0;
+    }
+
+    /** @return */
+    public Object getNumberSetRunning() {
+        return setRunningCall;
+    }
+
+}

gclc-process/src/test/java/net/bigeon/gclc/process/mocks/package-info.java

@@ -0,0 +1,4 @@
+/** Mock definitions for testing.
+ * 
+ * @author Emmanuel Bigeon */
+package net.bigeon.gclc.process.mocks;

gclc-socket/pom.xml

@@ -1,64 +1,22 @@
-
-<!-- GCLC Socket, Socket implementation of GCLC -->
-<!-- Copyright (C) 2014-2017 E. Bigeon -->
-<!-- mailto:emmanuel@bigeon.fr -->
-<!-- -->
-<!-- This software is governed by the CeCILL license under French law and -->
-<!-- abiding by the rules of distribution of free software. You can use, -->
-<!-- modify and/or redistribute the software under the terms of the CeCILL -->
-<!-- license as circulated by CEA, CNRS and INRIA at the following URL -->
-<!-- "http://www.cecill.info". -->
-<!-- -->
-<!-- As a counterpart to the access to the source code and rights to copy, -->
-<!-- modify and redistribute granted by the license, users are provided only -->
-<!-- with a limited warranty and the software's author, the holder of the -->
-<!-- economic rights, and the successive licensors have only limited -->
-<!-- liability. -->
-<!-- -->
-<!-- In this respect, the user's attention is drawn to the risks associated -->
-<!-- with loading, using, modifying and/or developing or reproducing the -->
-<!-- software by the user in light of its specific status of free software, -->
-<!-- that may mean that it is complicated to manipulate, and that also -->
-<!-- therefore means that it is reserved for developers and experienced -->
-<!-- professionals having in-depth computer knowledge. Users are therefore -->
-<!-- encouraged to load and test the software's suitability as regards their -->
-<!-- requirements in conditions enabling the security of their systems and/or -->
-<!-- data to be ensured and, more generally, to use and operate it in the -->
-<!-- same conditions as regards security. -->
-<!-- -->
-<!-- The fact that you are presently reading this means that you have had -->
-<!-- knowledge of the CeCILL license and that you accept its terms. -->
-
-
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
 	<modelVersion>4.0.0</modelVersion>
-	<artifactId>gclc-socket</artifactId>
-	<version>1.1.11-SNAPSHOT</version>
-	<packaging>jar</packaging>
-	<url>http://www.bigeon.net</url>
-	<properties>
-		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-		<project.scm.id>git.code.bigeon.net</project.scm.id>
-		<license.licenseName>cecill_2.1</license.licenseName>
-		<copyright.email>emmanuel@bigeon.fr</copyright.email>
-	</properties>
-	<dependencies>
-		<dependency>
-			<groupId>net.bigeon</groupId>
-			<artifactId>gclc</artifactId>
-			<version>2.0.7</version>
-		</dependency>
-		<dependency>
-			<groupId>net.bigeon</groupId>
-			<artifactId>smu</artifactId>
-			<version>1.0.0</version>
-		</dependency>
-	</dependencies>
 	<parent>
 		<groupId>net.bigeon.config</groupId>
 		<artifactId>ebigeon-config</artifactId>
-		<version>1.8.9</version>
+		<version>1.8.21</version>
 	</parent>
+	<groupId>net.bigeon.gclc</groupId>
+	<artifactId>socket</artifactId>
+	<version>1.1.13-SNAPSHOT</version>
+	<packaging>jar</packaging>
+	<name>GCLC Socket</name>
+	<description>Socket implementation of GCLC</description>
+	<url>https://bigeon.net/projects/gclc.html</url>
+	<inceptionYear>2016</inceptionYear>
+	<organization>
+		<name>Bigeon</name>
+		<url>https://bigeon.net/</url>
+	</organization>
 	<licenses>
 		<license>
 			<distribution>manual</distribution>
@@ -66,6 +24,43 @@
 			<url>https://cecill.info/licences/Licence_CeCILL_V2.1-en.html</url>
 		</license>
 	</licenses>
+	<developers>
+		<developer>
+			<email>emmanuel@bigeon.fr</email>
+			<name>Emmanuel Bigeon</name>
+			<url>bigeon.net</url>
+			<roles>
+				<role>PM</role>
+			</roles>
+		</developer>
+	</developers>
+	<scm>
+		<developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
+		<tag>HEAD</tag>
+	</scm>
+	<properties>
+		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
+		<project.scm.id>git.code.bigeon.net</project.scm.id>
+		<license.licenseName>cecill_2.1</license.licenseName>
+	</properties>
+	<dependencies>
+		<dependency>
+			<groupId>net.bigeon</groupId>
+			<artifactId>gclc</artifactId>
+			<version>2.0.11</version>
+		</dependency>
+		<dependency>
+			<groupId>net.bigeon</groupId>
+			<artifactId>smu</artifactId>
+			<version>1.0.6</version>
+		</dependency>
+		<dependency>
+			<groupId>net.bigeon.test</groupId>
+			<artifactId>junitmt</artifactId>
+			<version>1.0.0</version>
+			<scope>test</scope>
+		</dependency>
+	</dependencies>
 	<reporting>
 		<plugins>
 			<plugin>
@@ -80,22 +75,4 @@
 			</plugin>
 		</plugins>
 	</reporting>
-	<developers>
-		<developer>
-			<email>emmanuel@bigeon.fr</email>
-			<name>Emmanuel Bigeon</name>
-			<url>bigeon.net</url>
-			<roles>
-				<role>PM</role>
-			</roles>
-		</developer>
-	</developers>
-	<name>GCLC Socket</name>
-	<description>Socket implementation of GCLC</description>
-	<scm>
-		<developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
-		<tag>HEAD</tag>
-	</scm>
-	<groupId>net.bigeon.gclc</groupId>
-	<inceptionYear>2016</inceptionYear>
 </project>

gclc-socket/src/main/java/net/bigeon/gclc/socket/DConnexionManager.java

@@ -61,12 +61,13 @@ public final class DConnexionManager<T> implements ConnexionManager<T> {
     /** The lock for modification of {@link #counters}. */
     private final Object               counterLock = new Object();
     /** The count of connexions. */
-    private int count = 0;
+    private int                        count;
 
     /** Default.constructor. */
     public DConnexionManager() {
         //
     }
+
     /* (non-Javadoc)
      * @see
      * net.bigeon.gclc.socket.ConnexionManager#addConnexion(java.lang.Object) */
@@ -80,11 +81,12 @@ public final class DConnexionManager<T> implements ConnexionManager<T> {
     }
 
     /* (non-Javadoc)
-     * @see
-     * net.bigeon.gclc.socket.ConnexionManager#disconnect(java.lang.String) */
+     * @see net.bigeon.gclc.socket.ConnexionManager#disconnect(java.lang.String) */
     @Override
     public T disconnect(final String id) {
-        if (connecteds.containsKey(id)) {
+        if (!connecteds.containsKey(id)) {
+            return null;
+        }
         final T disc = connecteds.remove(id);
         final Object lock = locks.get(id);
         synchronized (lock) {
@@ -102,8 +104,6 @@ public final class DConnexionManager<T> implements ConnexionManager<T> {
         }
         return disc;
     }
-        return null;
-    }
 
     /* (non-Javadoc)
      * @see net.bigeon.gclc.socket.ConnexionManager#get(java.lang.String) */
@@ -120,8 +120,7 @@ public final class DConnexionManager<T> implements ConnexionManager<T> {
     }
 
     /* (non-Javadoc)
-     * @see
-     * net.bigeon.gclc.socket.ConnexionManager#isConnected(java.lang.String) */
+     * @see net.bigeon.gclc.socket.ConnexionManager#isConnected(java.lang.String) */
     @Override
     public boolean isConnected(final String id) {
         return connecteds.containsKey(id);
@@ -144,7 +143,8 @@ public final class DConnexionManager<T> implements ConnexionManager<T> {
      *
      * @return a new ID */
     private String newID() {
-        return "Client " + count++; //$NON-NLS-1$
+        final int c = count++;
+        return "Client " + c; //$NON-NLS-1$
     }
 
     /* (non-Javadoc)
@@ -154,8 +154,8 @@ public final class DConnexionManager<T> implements ConnexionManager<T> {
     @Override
     public void releaseDisconnexionLock(final String id) {
         synchronized (counterLock) {
-            counters.put(id, Integer
-                    .valueOf(Math.max(counters.get(id).intValue() - 1, 0)));
+            counters.put(id,
+                    Integer.valueOf(Math.max(counters.get(id).intValue() - 1, 0)));
             counterLock.notifyAll();
         }
     }

gclc-socket/src/main/java/net/bigeon/gclc/socket/PluggableConsoleInput.java

@@ -54,9 +54,12 @@ import net.bigeon.gclc.utils.ReadingRunnable;
 /** A console input where the stream can be plugged.
  * <p>
  * This pluggable console input accepts an input and output to be connected to
- * it. The connexion cannot be concurrent, which mean that any connected stream
+ * it. The connection cannot be concurrent, which mean that any connected stream
  * must be disconnected before a new call to
  * {@link #connect(InputStream, PrintStream)} is done.
+ * <p>
+ * On connection, if the prompt is in wait, this console input will provide the
+ * prompting message.
  *
  * @author Emmanuel Bigeon */
 public final class PluggableConsoleInput implements ConsoleInput {
@@ -100,28 +103,26 @@ public final class PluggableConsoleInput implements ConsoleInput {
         closed = true;
     }
 
-    /** Connect the given input stream to the input and output to the hints
-     * writing.
+    /** Connect the given input stream to the input and output to the hints writing.
      *
      * @param stream the input stream
      * @param out the output for hints.
      * @throws IOException if the input is already connected. */
-    public void connect(final InputStream stream,
-                        final PrintStream out) throws IOException {
+    public void connect(final InputStream stream, final PrintStream out)
+            throws IOException {
         synchronized (connexionLock) {
             if (connected) {
-                throw new IOException(
-                        "Input already connected to an input stream"); //$NON-NLS-1$
+                // Cannot be connected to several sources
+                throw new IOException("Input already connected to an input stream"); //$NON-NLS-1$
             }
-
             output = out;
             if (prompting) {
+                // print the hint, to indicate we are waiting for a user input.
                 out.print(hint);
                 out.flush();
             }
-
-            final InputStreamReader streamReader = new InputStreamReader(
-                    stream, StandardCharsets.UTF_8);
+            final InputStreamReader streamReader = new InputStreamReader(stream,
+                    StandardCharsets.UTF_8);
             final BufferedReader reader = new BufferedReader(streamReader);
             connexion = new ReadingRunnable(reader);
             final Thread th = new Thread(connexion, "GCLC Socket - Read input"); //$NON-NLS-1$
@@ -191,19 +192,11 @@ public final class PluggableConsoleInput implements ConsoleInput {
         if (closed) {
             throw new IOException();
         }
-        prompting = true;
-        hint = message;
-        synchronized (connexionLock) {
-            hint = message;
-            if (connected) {
-                output.print(message);
-                output.flush();
-            }
-        }
-
+        preparePrompt(message);
         String res = null;
         while (res == null && !interrupted) {
             try {
+                // Wait for a message or a connection with message
                 res = waitMessageOrConnexion(TIMEOUT, TIMEOUT / TENTH);
             } catch (final InterruptedException e) {
                 LOGGER.log(Level.FINE, "Interruption of thread", e); //$NON-NLS-1$
@@ -217,31 +210,36 @@ public final class PluggableConsoleInput implements ConsoleInput {
         return res;
     }
 
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String,
-     * long) */
-    @Override
-    public String prompt(final String message,
-                         final long timeout) throws IOException {
-        if (closed) {
-            throw new IOException();
-        }
+    /** Prepare the new prompt.
+     * 
+     * @param message the message to indicate request of prompt. */
+    private void preparePrompt(final String message) {
         prompting = true;
+        // hold the message
         synchronized (connexionLock) {
             hint = message;
             if (connected) {
+                // print the message
                 output.print(message);
                 output.flush();
             }
         }
+    }
 
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String, long) */
+    @Override
+    public String prompt(final String message, final long timeout) throws IOException {
+        if (closed) {
+            throw new IOException();
+        }
+        preparePrompt(message);
         String res = null;
         final long tic = System.currentTimeMillis();
         long time = System.currentTimeMillis() - tic;
         while (res == null && !interrupted && time < timeout) {
             try {
-                res = waitMessageOrConnexion(timeout - time,
-                        (timeout - time) / TENTH);
+                res = waitMessageOrConnexion(timeout - time, (timeout - time) / TENTH);
             } catch (final InterruptedException e) {
                 LOGGER.log(Level.FINE, "Interruption of thread", e); //$NON-NLS-1$
                 Thread.currentThread().interrupt();
@@ -255,26 +253,30 @@ public final class PluggableConsoleInput implements ConsoleInput {
         return res;
     }
 
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.manager.ConsoleInput#setPrompt(java.lang.String) */
+    @Override
+    public void setPrompt(final String prompt) {
+        setPrompt(new ConstantString(prompt));
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.manager.ConsoleInput#setPrompt(net.bigeon.gclc.tools.
+     * StringProvider) */
     @Override
     public void setPrompt(final StringProvider prompt) {
         this.prompt = prompt;
     }
 
-    @Override
-    public void setPrompt(String prompt) {
-    	setPrompt(new ConstantString(prompt));
-    }
-
-    /** Wait for a hint or connexion.
+    /** Wait for a hint or connection.
      *
-     * @param messageTimeout the timeout on the current connexion hint waiting
-     * @param connexionTimeout the timeout on the new connexion wait
+     * @param messageTimeout the timeout on the current connection hint waiting
+     * @param connexionTimeout the timeout on the new connection wait
      * @return the hint, or null if not connected or timed out.
      * @throws IOException if the reading failed.
      * @throws InterruptedException if the wait was interrupted */
     private String waitMessageOrConnexion(final long messageTimeout,
-                                          final long connexionTimeout) throws IOException,
-                                                                       InterruptedException {
+            final long connexionTimeout) throws IOException, InterruptedException {
         synchronized (connexionLock) {
             if (connected) {
                 return connexion.getNextMessage(messageTimeout);

gclc-socket/src/main/java/net/bigeon/gclc/socket/PluggableConsoleOutput.java

@@ -60,6 +60,7 @@ public final class PluggableConsoleOutput implements ConsoleOutput {
     public PluggableConsoleOutput() {
         //
     }
+
     /* (non-Javadoc)
      * @see java.lang.AutoCloseable#close() */
     @Override

gclc-socket/src/main/java/net/bigeon/gclc/socket/RemoteDisconnectCommand.java

@@ -65,8 +65,8 @@ public final class RemoteDisconnectCommand<T> extends Command {
      * @param manager the manager
      * @param all if all elements should be disconnected when no argument is
      *        provided */
-    public RemoteDisconnectCommand(final String name,
-            final ConnexionManager<T> manager, final boolean all) {
+    public RemoteDisconnectCommand(final String name, final ConnexionManager<T> manager,
+                                   final boolean all) {
         super(name);
         this.manager = manager;
         this.all = all;
@@ -74,8 +74,7 @@ public final class RemoteDisconnectCommand<T> extends Command {
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
-     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-     * java.lang.String[]) */
+     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
     @Override
     public void execute(final ConsoleOutput out, final ConsoleInput in,
             final String... args) throws CommandRunException {
@@ -89,8 +88,8 @@ public final class RemoteDisconnectCommand<T> extends Command {
             if (manager.isConnected(string)) {
                 manager.disconnect(string);
             } else {
-                print(out, MessageFormat
-                        .format("[WARNING] {0} is not connected", string)); //$NON-NLS-1$
+                print(out,
+                        MessageFormat.format("[WARNING] {0} is not connected", string)); //$NON-NLS-1$
             }
         }
     }
@@ -99,16 +98,14 @@ public final class RemoteDisconnectCommand<T> extends Command {
      *
      * @param out the output
      * @param string the message
-     * @throws CommandRunException if the output exists but cannot be printed
-     *             to */
-    private void print(final ConsoleOutput out,
-                       final String string) throws CommandRunException {
+     * @throws CommandRunException if the output exists but cannot be printed to */
+    private static void print(final ConsoleOutput out, final String string)
+            throws CommandRunException {
         if (out != null) {
             try {
                 out.println(string);
             } catch (final IOException e) {
-                throw new CommandRunException(
-                        CommandRunExceptionType.INTERACTION,
+                throw new CommandRunException(CommandRunExceptionType.INTERACTION,
                         "Unable to print to existing output", e); //$NON-NLS-1$
             }
         }
@@ -123,12 +120,10 @@ public final class RemoteDisconnectCommand<T> extends Command {
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.Command#usageDetail() */
-    @SuppressWarnings("nls")
     @Override
     protected String usageDetail() {
         return MessageFormat.format(
-                "  If arguments are provided the corresponding connexions are closed, " +
-                                    "otherwise{0}{1} are.",
-                System.lineSeparator(), all ? "all connexions" : "none");
+                "  If arguments are provided the corresponding connexions are closed, otherwise\n{0} are.",
+                all ? "all connexions" : "none");
     }
 }

gclc-socket/src/main/java/net/bigeon/gclc/socket/SocketConsoleApplicationShell.java

@@ -82,13 +82,12 @@ public final class SocketConsoleApplicationShell implements Runnable {
     /** THe server address. */
     private final InetAddress addr;
 
-    /** Create a socket application shell which will listen on the given port
-     * and network interface.
+    /** Create a socket application shell which will listen on the given port and
+     * network interface.
      *
      * @param port the part
      * @param addr the inet address */
-    public SocketConsoleApplicationShell(final int port,
-            final InetAddress addr) {
+    public SocketConsoleApplicationShell(final int port, final InetAddress addr) {
         super();
         this.port = port;
         this.addr = addr;
@@ -110,9 +109,9 @@ public final class SocketConsoleApplicationShell implements Runnable {
     }
 
     /** If the port provided was 0, this allows to get the actual port.
+     * 
      * @return the local port
-     * @see java.net.ServerSocket#getLocalPort()
-     */
+     * @see java.net.ServerSocket#getLocalPort() */
     public int getLocalPort() {
         return serverSocket.getLocalPort();
     }
@@ -122,16 +121,14 @@ public final class SocketConsoleApplicationShell implements Runnable {
     @Override
     public void run() {
         // Create the server
-        try (ServerSocket actualServerSocket = new ServerSocket(port, 1,
-                addr)) {
+        try (ServerSocket actualServerSocket = new ServerSocket(port, 1, addr)) {
             serverSocket = actualServerSocket;
             running = true;
             // Create the streams
             runSokectServer();
         } catch (final IOException e) {
             LOGGER.severe("Communication error between client and server"); //$NON-NLS-1$
-            LOGGER.log(Level.FINE,
-                    "Communication error between client and server", e); //$NON-NLS-1$
+            LOGGER.log(Level.FINE, "Communication error between client and server", e); //$NON-NLS-1$
         }
     }
 
@@ -141,7 +138,7 @@ public final class SocketConsoleApplicationShell implements Runnable {
     private void runSokectServer() throws IOException {
         while (running && app.isRunning()) {
             LOGGER.info("Waiting client"); //$NON-NLS-1$
-            try (Socket clientSocket = serverSocket.accept();) {
+            try (Socket clientSocket = serverSocket.accept()) {
 
                 sci.connect(clientSocket);
 

gclc-socket/src/main/java/net/bigeon/gclc/socket/SocketConsoleInterface.java

@@ -68,12 +68,10 @@ public final class SocketConsoleInterface {
      * @param socket the socket
      * @throws IOException if the connection failed */
     public void connect(final Socket socket) throws IOException {
-        final PrintStream printStream = new PrintStream(
-                socket.getOutputStream(), true,
+        final PrintStream printStream = new PrintStream(socket.getOutputStream(), true,
                 StandardCharsets.UTF_8.name());
         output.connect(printStream);
-        input.connect(socket.getInputStream(),
-                printStream);
+        input.connect(socket.getInputStream(), printStream);
     }
 
     /** Disconnect the input and output of the application from the socket's. */

gclc-socket/src/site/site.xml

@@ -0,0 +1,44 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+  #%L
+  Generic Command Ligne console
+  %%
+  Copyright (C) 2014 - 2018 Bigeon
+  %%
+  This software is governed by the CeCILL  license under French law and
+  abiding by the rules of distribution of free software.  You can  use, 
+  modify and/ or redistribute the software under the terms of the CeCILL
+  license as circulated by CEA, CNRS and INRIA at the following URL
+  "http://www.cecill.info". 
+  
+  As a counterpart to the access to the source code and  rights to copy,
+  modify and redistribute granted by the license, users are provided only
+  with a limited warranty  and the software's author,  the holder of the
+  economic rights,  and the successive licensors  have only  limited
+  liability. 
+  
+  In this respect, the user's attention is drawn to the risks associated
+  with loading,  using,  modifying and/or developing or reproducing the
+  software by the user in light of its specific status of free software,
+  that may mean  that it is complicated to manipulate,  and  that  also
+  therefore means  that it is reserved for developers  and  experienced
+  professionals having in-depth computer knowledge. Users are therefore
+  encouraged to load and test the software's suitability as regards their
+  requirements in conditions enabling the security of their systems and/or 
+  data to be ensured and,  more generally, to use and operate it in the 
+  same conditions as regards security. 
+  
+  The fact that you are presently reading this means that you have had
+  knowledge of the CeCILL license and that you accept its terms.
+  #L%
+  -->
+
+<project name="GCLC">
+  <bannerRight>
+    <src>https://bigeon.net/images/logo_48.png</src>
+    <href>https://bigeon.net/</href>
+  </bannerRight>
+  <body>
+    <menu ref="reports"/>
+  </body>
+</project>

gclc-socket/src/test/java/net/bigeon/gclc/socket/ConsoleTestApplication.java

@@ -39,14 +39,12 @@ import java.util.Collection;
 
 import net.bigeon.gclc.ConsoleApplication;
 import net.bigeon.gclc.command.Command;
-import net.bigeon.gclc.command.HelpExecutor;
 import net.bigeon.gclc.command.base.ExitCommand;
+import net.bigeon.gclc.command.base.HelpExecutor;
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.exception.InvalidCommandName;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
-import net.bigeon.gclc.socket.ConnexionManager;
-import net.bigeon.gclc.socket.RemoteDisconnectCommand;
 
 /** A test-purpose application
  *
@@ -64,13 +62,10 @@ public class ConsoleTestApplication {
      * @return create the application */
     @SuppressWarnings("nls")
     public static ConsoleApplication create(final ConsoleOutput output,
-                                            final ConsoleInput input,
-                                            final ConnexionManager<Socket> manager) {
+            final ConsoleInput input, final ConnexionManager<Socket> manager) {
         try {
-            final ConsoleApplication application = new ConsoleApplication(
-                    output, input,
-                    "Welcome to the test application. Type help or test.",
-                    "See you");
+            final ConsoleApplication application = new ConsoleApplication(output, input,
+                    "Welcome to the test application. Type help or test.", "See you");
             application.add(new ExitCommand(EXIT, application) {
                 /* (non-Javadoc)
                  * @see fr.bigeon.gclc.command.ExitCommand#beforeExit() */
@@ -82,8 +77,7 @@ public class ConsoleTestApplication {
                     }
                 }
             });
-            application
-                    .add(new HelpExecutor("help", application.root));
+            application.add(new HelpExecutor("help", application.root));
             application.add(new Command("test") {
 
                 /* (non-Javadoc)
@@ -91,8 +85,7 @@ public class ConsoleTestApplication {
                  * manager.ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
                  * java.lang.String[]) */
                 @Override
-                public void execute(final ConsoleOutput out,
-                                    final ConsoleInput in,
+                public void execute(final ConsoleOutput out, final ConsoleInput in,
                         final String... args) throws CommandRunException {
                     try {
                         output.println("Test command ran fine");
@@ -107,8 +100,7 @@ public class ConsoleTestApplication {
                 }
 
                 /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.Command#usageDetail()
-                 */
+                 * @see fr.bigeon.gclc.command.Command#usageDetail() */
                 @Override
                 protected String usageDetail() {
                     // TODO Auto-generated method stub
@@ -123,11 +115,13 @@ public class ConsoleTestApplication {
                  * manager.ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
                  * java.lang.String[]) */
                 @Override
-                public void execute(final ConsoleOutput out,
-                                    final ConsoleInput in,
+                public void execute(final ConsoleOutput out, final ConsoleInput in,
                         final String... args) throws CommandRunException {
                     try {
-                        Thread.sleep(2000);
+                        Object obj = new Object();
+                        synchronized (obj) {
+                                obj.wait(2000);
+                        }
                         output.println("Test command ran fine");
                     } catch (IOException | InterruptedException e) {
                         throw new CommandRunException("manager closed", e);
@@ -140,8 +134,7 @@ public class ConsoleTestApplication {
                 }
 
                 /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.Command#usageDetail()
-                 */
+                 * @see fr.bigeon.gclc.command.Command#usageDetail() */
                 @Override
                 protected String usageDetail() {
                     // TODO Auto-generated method stub
@@ -149,8 +142,7 @@ public class ConsoleTestApplication {
                     throw new RuntimeException("Not implemented yet");
                 }
             });
-            application.add(
-                    new RemoteDisconnectCommand<>("out", manager, true));
+            application.add(new RemoteDisconnectCommand<>("out", manager, true));
             return application;
         } catch (final InvalidCommandName e) {
             e.printStackTrace();

gclc-socket/src/test/java/net/bigeon/gclc/socket/PluggableConsoleInputTest.java

@@ -39,6 +39,7 @@ package net.bigeon.gclc.socket;
  */
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
 
@@ -46,11 +47,10 @@ import java.io.IOException;
 import java.io.PipedInputStream;
 import java.io.PipedOutputStream;
 import java.io.PrintStream;
+import java.util.concurrent.atomic.AtomicReference;
 
 import org.junit.Test;
 
-import net.bigeon.gclc.socket.PluggableConsoleInput;
-
 /**
  * <p>
  * TODO
@@ -90,7 +90,10 @@ public class PluggableConsoleInputTest {
 
                 }
                 try {
-                    Thread.sleep(100);
+                    Object obj = new Object();
+                    synchronized (obj) {
+                            obj.wait(100);
+                    }
                 } catch (final InterruptedException e) {
                     // TODO Auto-generated catch block
                     e.printStackTrace();
@@ -112,7 +115,10 @@ public class PluggableConsoleInputTest {
             @Override
             public void run() {
                 try {
-                    Thread.sleep(200);
+                    Object obj = new Object();
+                    synchronized (obj) {
+                            obj.wait(200);
+                    }
                 } catch (final InterruptedException e) {
                     // TODO Auto-generated catch block
                     e.printStackTrace();
@@ -162,32 +168,40 @@ public class PluggableConsoleInputTest {
             });
             th.start();
 
+            Object obj = new Object();
+            synchronized (obj) {
                 while (!input.isPrompting()) {
-                Thread.sleep(10);
+                    obj.wait(10);
+                }
             }
 
             input.connect(pis, out);
             testIn.println("tac");
 
+            final AtomicReference<IOException> ref = new AtomicReference<>();
             final Thread th2 = new Thread(new Runnable() {
 
                 @Override
                 public void run() {
                     try {
                         input.prompt("Test", 5000);
-                        fail("Prompt should io");
+// fail("Prompt should io");
                     } catch (final IOException e) {
                         // ok
+                        ref.set(e);
                     }
                 }
             });
             th2.start();
+            synchronized (obj) {
                 while (!input.isPrompting()) {
-                Thread.sleep(10);
+                    obj.wait(10);
+                }
             }
 
             input.close();
-
+            th2.join();
+            assertNotNull("Prompt should io", ref.get());
 
         }
     }

gclc-socket/src/test/java/net/bigeon/gclc/socket/RemoteDisconnectCommandTest.java

@@ -46,8 +46,6 @@ import java.io.IOException;
 import org.junit.Test;
 
 import net.bigeon.gclc.exception.CommandRunException;
-import net.bigeon.gclc.socket.DConnexionManager;
-import net.bigeon.gclc.socket.RemoteDisconnectCommand;
 import net.bigeon.gclc.utils.PipedConsoleOutput;
 
 /**
@@ -65,10 +63,10 @@ public class RemoteDisconnectCommandTest {
     @Test
     public final void testExecute() throws CommandRunException, IOException {
         final DConnexionManager<String> manager = new DConnexionManager<>();
-        final RemoteDisconnectCommand<String> cmd = new RemoteDisconnectCommand<>(
-                "quit", manager, true);
-        final RemoteDisconnectCommand<String> cmd2 = new RemoteDisconnectCommand<>(
-                "quit", manager, false);
+        final RemoteDisconnectCommand<String> cmd = new RemoteDisconnectCommand<>("quit",
+                manager, true);
+        final RemoteDisconnectCommand<String> cmd2 = new RemoteDisconnectCommand<>("quit",
+                manager, false);
         manager.addConnexion("test");
 
         cmd2.execute(null, null);

gclc-socket/src/test/java/net/bigeon/gclc/socket/SocketConsoleApplicationTest.java

@@ -48,12 +48,10 @@ import java.util.logging.Logger;
 
 import org.junit.Test;
 
-import net.bigeon.gclc.socket.SocketConsoleApplicationShell;
-
 /** Test class for {@link SocketConsoleApplicationShell}
  *
  * @author Emmanuel Bigeon */
-@SuppressWarnings({"static-method", "javadoc", "nls"})
+@SuppressWarnings({ "static-method", "javadoc", "nls" })
 public class SocketConsoleApplicationTest {
 
     private static final Logger LOGGER = Logger
@@ -62,12 +60,12 @@ public class SocketConsoleApplicationTest {
     /** @param in the input
      * @return the string
      * @throws IOException if the input reading failed */
-    private String consumeToPrompt(final String server,
-                                   final BufferedReader in) throws IOException {
+    private String consumeToPrompt(final String server, final BufferedReader in)
+            throws IOException {
         String fromServer = server;
         LOGGER.fine("Server: \n" + fromServer);
-        while (fromServer != null && !fromServer.equals("Bye.") &&
-               !fromServer.equals("> ")) {
+        while (fromServer != null && !fromServer.equals("Bye.")
+                && !fromServer.equals("> ")) {
             fromServer = in.readLine();
             LOGGER.fine("Server: \n" + fromServer);
         }
@@ -78,19 +76,17 @@ public class SocketConsoleApplicationTest {
     public void testIntegration() throws IOException, InterruptedException {
         Thread server;
         server = TestServer.getServer();
-        Thread.sleep(1000);
         final String hostName = "127.0.0.1";
         final int portNumber = 3300;
 
         try (Socket kkSocket = new Socket(hostName, portNumber);
-             PrintWriter out = new PrintWriter(kkSocket.getOutputStream(),
-                     true);
+                PrintWriter out = new PrintWriter(kkSocket.getOutputStream(), true);
                 BufferedReader in = new BufferedReader(
                         new InputStreamReader(kkSocket.getInputStream()));) {
 
             String fromServer;
             int i = -1;
-            final String[] cmds = {"help", "toto", "test", "out"};
+            final String[] cmds = { "help", "toto", "test", "out" };
             while ((fromServer = in.readLine()) != null) {
                 i++;
                 fromServer = consumeToPrompt(fromServer, in);
@@ -108,15 +104,13 @@ public class SocketConsoleApplicationTest {
         }
 
         try (Socket kkSocket = new Socket(hostName, portNumber);
-             PrintWriter out = new PrintWriter(kkSocket.getOutputStream(),
-                     true);
+                PrintWriter out = new PrintWriter(kkSocket.getOutputStream(), true);
                 BufferedReader in = new BufferedReader(
                         new InputStreamReader(kkSocket.getInputStream()));) {
 
             String fromServer;
             int i = 0;
-            final String[] cmds = {"help", "toto", "test",
-                    ConsoleTestApplication.EXIT};
+            final String[] cmds = { "help", "toto", "test", ConsoleTestApplication.EXIT };
             while ((fromServer = in.readLine()) != null) {
                 fromServer = consumeToPrompt(fromServer, in);
                 if (fromServer == null || fromServer.equals("Bye.")) {
@@ -131,26 +125,20 @@ public class SocketConsoleApplicationTest {
                 }
                 i++;
             }
-            assertEquals("Application exit command should close connection", 4,
-                    i);
+            assertEquals("Application exit command should close connection", 4, i);
         }
-        Thread.sleep(100);
         TestServer.closeServer();
         server.join();
         server = TestServer.getServer();
-        Thread.sleep(1000);
-
 
         try (Socket kkSocket = new Socket(hostName, portNumber);
-             PrintWriter out = new PrintWriter(kkSocket.getOutputStream(),
-                     true);
+                PrintWriter out = new PrintWriter(kkSocket.getOutputStream(), true);
                 BufferedReader in = new BufferedReader(
                         new InputStreamReader(kkSocket.getInputStream()));) {
 
             String fromServer;
             int i = 0;
-            final String[] cmds = {"help", "toto", "test",
-                    ConsoleTestApplication.EXIT};
+            final String[] cmds = { "help", "toto", "test", ConsoleTestApplication.EXIT };
             while ((fromServer = in.readLine()) != null) {
                 fromServer = consumeToPrompt(fromServer, in);
                 if (fromServer == null || fromServer.equals("Bye.")) {
@@ -164,8 +152,7 @@ public class SocketConsoleApplicationTest {
                 }
                 i++;
             }
-            assertEquals("Application exit command should close connection", 4,
-                    i);
+            assertEquals("Application exit command should close connection", 4, i);
         }
         TestServer.closeServer();
         server.join();

gclc-socket/src/test/java/net/bigeon/gclc/socket/TestConsoleClient.java

@@ -44,9 +44,8 @@ import java.util.Collection;
 import net.bigeon.smu.StringEncoder;
 
 /** TODO Describe TestConsoleClient.java
- * @author Emmanuel Bigeon
  * 
- */
+ * @author Emmanuel Bigeon */
 @SuppressWarnings("nls")
 public class TestConsoleClient {
     @SuppressWarnings("javadoc")
@@ -56,8 +55,7 @@ public class TestConsoleClient {
     }
 
     @SuppressWarnings("javadoc")
-    private static final StringEncoder ENCODER = new StringEncoder("%",
-            TO_ENCODE);
+    private static final StringEncoder ENCODER = new StringEncoder("%", TO_ENCODE);
 
     @SuppressWarnings("javadoc")
     public static void main(String[] args) {
@@ -65,10 +63,9 @@ public class TestConsoleClient {
         final int portNumber = 3300;
 
         try (Socket kkSocket = new Socket(hostName, portNumber);
-                PrintWriter out = new PrintWriter(kkSocket.getOutputStream(),
-                        true);
-                BufferedReader in = new BufferedReader(new InputStreamReader(
-                        kkSocket.getInputStream()));) {
+                PrintWriter out = new PrintWriter(kkSocket.getOutputStream(), true);
+                BufferedReader in = new BufferedReader(
+                        new InputStreamReader(kkSocket.getInputStream()));) {
 
             String fromServer;
             while ((fromServer = in.readLine()) != null) {

gclc-socket/src/test/java/net/bigeon/gclc/socket/TestServer.java

@@ -39,19 +39,14 @@ import java.net.Socket;
 import java.net.UnknownHostException;
 
 import net.bigeon.gclc.ConsoleApplication;
-import net.bigeon.gclc.socket.ConnexionManager;
-import net.bigeon.gclc.socket.DConnexionManager;
-import net.bigeon.gclc.socket.PluggableConsoleInput;
-import net.bigeon.gclc.socket.PluggableConsoleOutput;
-import net.bigeon.gclc.socket.SocketConsoleApplicationShell;
-import net.bigeon.gclc.socket.SocketConsoleInterface;
 
 /** A test server
  *
  * @author Emmanuel Bigeon */
-@SuppressWarnings({"javadoc", "nls"})
+@SuppressWarnings({ "javadoc", "nls" })
 public class TestServer {
 
+    public static final int                      PORT = 3300;
     private static SocketConsoleApplicationShell SHELL;
     private static ConnexionManager<Socket>      manager;
 
@@ -81,10 +76,10 @@ public class TestServer {
             input.setPrompt("> \n");
             output = new PluggableConsoleOutput();
             manager = new DConnexionManager<>();
-            SHELL = new SocketConsoleApplicationShell(3300,
+            SHELL = new SocketConsoleApplicationShell(PORT,
                     InetAddress.getByName("127.0.0.1"));
-            final ConsoleApplication app = ConsoleTestApplication
-                    .create(output, input, manager);
+            final ConsoleApplication app = ConsoleTestApplication.create(output, input,
+                    manager);
             SHELL.setInterface(new SocketConsoleInterface(input, output));
             SHELL.setConnexionManager(manager);
             SHELL.setApplication(app);
@@ -96,6 +91,16 @@ public class TestServer {
                 }
             });
             th.start();
+            try {
+                final Object waiting = new Object();
+                synchronized (waiting) {
+                    while (!input.isPrompting()) {
+                        waiting.wait(50);
+                    }
+                }
+            } catch (final InterruptedException e) {
+                Thread.currentThread().interrupt();
+            }
         }
         return SHELL;
     }

gclc-swt/pom.xml

@@ -1,50 +1,45 @@
-
-<!-- GCLC swt, provide a swt window for console applications                                   -->
-<!-- Copyright (C) 2015-2017 E. Bigeon                      -->
-<!-- mailto:emmanuel@bigeon.fr                                                           -->
-<!--                                                                           -->
-<!-- This software is governed by the CeCILL license under French law and      -->
-<!-- abiding by the rules of distribution of free software.  You can  use,     -->
-<!-- modify and/or redistribute the software under the terms of the CeCILL     -->
-<!-- license as circulated by CEA, CNRS and INRIA at the following URL         -->
-<!-- "http://www.cecill.info".                                                 -->
-<!--                                                                           -->
-<!-- As a counterpart to the access to the source code and  rights to copy,    -->
-<!-- modify and redistribute granted by the license, users are provided only   -->
-<!-- with a limited warranty  and the software's author,  the holder of the    -->
-<!-- economic rights,  and the successive licensors  have only  limited        -->
-<!-- liability.                                                                -->
-<!--                                                                           -->
-<!-- In this respect, the user's attention is drawn to the risks associated    -->
-<!-- with loading,  using,  modifying and/or developing or reproducing the     -->
-<!-- software by the user in light of its specific status of free software,    -->
-<!-- that may mean  that it is complicated to manipulate,  and  that  also     -->
-<!-- therefore means  that it is reserved for developers  and  experienced     -->
-<!-- professionals having in-depth computer knowledge. Users are therefore     -->
-<!-- encouraged to load and test the software's suitability as regards their   -->
-<!-- requirements in conditions enabling the security of their systems and/or  -->
-<!-- data to be ensured and,  more generally, to use and operate it in the     -->
-<!-- same conditions as regards security.                                      -->
-<!--                                                                           -->
-<!-- The fact that you are presently reading this means that you have had      -->
-<!-- knowledge of the CeCILL license and that you accept its terms.            -->
-
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
 	<modelVersion>4.0.0</modelVersion>
-  <groupId>net.bigeon.gclc</groupId>
-  <artifactId>gclc-swt</artifactId>
-  <version>1.1.6-SNAPSHOT</version>
-  <packaging>jar</packaging>
-  <inceptionYear>2015</inceptionYear>
-  <name>GCLC swt</name>
-  <description>A swt window for console applications</description>
-
 	<parent>
 		<groupId>net.bigeon.config</groupId>
 		<artifactId>swt-config</artifactId>
-    <version>1.8.7</version>
+		<version>1.8.11</version>
 	</parent>
-
+	<groupId>net.bigeon.gclc</groupId>
+	<artifactId>swt</artifactId>
+	<version>1.1.6-SNAPSHOT</version>
+	<packaging>jar</packaging>
+	<name>GCLC swt</name>
+	<description>A swt window for console applications</description>
+	<url>https://bigeon.net/projects/gclc.html</url>
+	<inceptionYear>2015</inceptionYear>
+	<organization>
+		<name>Bigeon</name>
+		<url>https://bigeon.net/</url>
+	</organization>
+	<licenses>
+		<license>
+			<distribution>manual</distribution>
+			<name>CeCILL 2.1</name>
+			<url>https://cecill.info/licences/Licence_CeCILL_V2.1-en.html</url>
+		</license>
+	</licenses>
+	<developers>
+		<developer>
+			<email>emmanuel@bigeon.fr</email>
+			<name>Emmanuel Bigeon</name>
+			<url>bigeon.net</url>
+			<roles>
+				<role>PM</role>
+			</roles>
+		</developer>
+	</developers>
+	<scm>
+		<developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
+		<tag>HEAD</tag>
+	</scm>
 	<properties>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 		<copyright.email>emmanuel@bigeon.fr</copyright.email>
@@ -55,22 +50,15 @@
 		<dependency>
 			<groupId>net.bigeon</groupId>
 			<artifactId>gclc</artifactId>
-      <version>2.0.7</version>
+			<version>2.0.11</version>
 		</dependency>
 		<dependency>
 			<groupId>net.bigeon</groupId>
 			<artifactId>collections</artifactId>
-      <version>1.1.1</version>
+			<version>1.1.6</version>
 		</dependency>
 	</dependencies>
 
-	<licenses>
-		<license>
-			<distribution>manual</distribution>
-			<name>CeCILL 2.1</name>
-			<url>https://cecill.info/licences/Licence_CeCILL_V2.1-en.html</url>
-		</license>
-	</licenses>
 	<reporting>
 		<plugins>
 			<plugin>
@@ -85,19 +73,4 @@
 			</plugin>
 		</plugins>
 	</reporting>
-	<developers>
-		<developer>
-			<email>emmanuel@bigeon.fr</email>
-			<name>Emmanuel Bigeon</name>
-			<url>bigeon.net</url>
-			<roles>
-				<role>PM</role>
-			</roles>
-		</developer>
-	</developers>
-
-  <scm>
-    <developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
-    <tag>HEAD</tag>
-  </scm>
 </project>

gclc-swt/src/main/java/net/bigeon/gclc/swt/ConsoleDelayIO.java

@@ -77,6 +77,7 @@ package net.bigeon.gclc.swt;
  * @author Emmanuel Bigeon */
 public interface ConsoleDelayIO {
     /** Get the input text.
+     * 
      * @return the non validated input */
     String getInput();
 

gclc-swt/src/main/java/net/bigeon/gclc/swt/HistoryTextKeyListener.java

@@ -123,8 +123,7 @@ public final class HistoryTextKeyListener extends KeyAdapter {
         }
 
         // Upper arrow retrieves previous commands
-        if (keyCode == SWT.ARROW_UP &&
-            currentIndex < commands.size() - 1) {
+        if (keyCode == SWT.ARROW_UP && currentIndex < commands.size() - 1) {
             currentIndex++;
             final String cmd = commands.get(commands.size() - currentIndex - 1);
             console.setInput(cmd);
@@ -132,12 +131,14 @@ public final class HistoryTextKeyListener extends KeyAdapter {
 
         // Lower arrow retrieves next commands
         if (keyCode == SWT.ARROW_DOWN) {
+            if (currentIndex <= 0) {
                 if (currentIndex == 0) {
-                currentIndex--;
                     console.setInput(EMPTY);
-            } else if (currentIndex > 0) {
-                final String cmd = commands
-                        .get(commands.size() - (--currentIndex) - 1);
+                }
+                currentIndex = -1;
+            } else {
+                currentIndex--;
+                final String cmd = commands.get(commands.size() - currentIndex - 1);
                 console.setInput(cmd);
             }
         }

gclc-swt/src/main/java/net/bigeon/gclc/swt/SWTConsole.java

@@ -92,6 +92,8 @@ import net.bigeon.gclc.tools.StringProvider;
  * @author Emmanuel Bigeon */
 public final class SWTConsole extends Composite
         implements ConsoleDelayIO, ConsoleInput, ConsoleOutput {
+    private static final int    TWO                  = 2;
+    private static final String ERROR_SYNCHRO_PROMPT = "Error in synchronization of prompting";
     /** The number of columns of the layout. */
     private static final int    LAYOUT_NB_COLUMNS    = 2;
     /** The cmd prefix in the output console. */
@@ -101,6 +103,7 @@ public final class SWTConsole extends Composite
             .getLogger(SWTConsole.class.getName());
     /** The empty string constant. */
     private static final String EMPTY                = "";                                     //$NON-NLS-1$
+
     /** The console output text field. */
     private final Text     consoleOutput;
     /** The console input text field. */
@@ -140,6 +143,8 @@ public final class SWTConsole extends Composite
 
     }
 
+    /* (non-Javadoc)
+     * @see org.eclipse.swt.widgets.Composite#checkSubclass() */
     @Override
     protected void checkSubclass() {
         // Disable the check that prevents subclassing of SWT components
@@ -250,29 +255,14 @@ public final class SWTConsole extends Composite
     @Override
     public String prompt() throws IOException {
         synchronized (promptLock) {
-            if (isDisposed()) {
-                throw new IOException();
-            }
-            try {
-                Display.getDefault().syncExec(new Runnable() {
-                    @Override
-                    public void run() {
-                        if (!consoleInput.isDisposed()) {
-                            consoleInput.setEnabled(true);
-                            lblPromptlabel.setText(prompt.apply());
-                            // relayout
-                            SWTConsole.this.layout();
-                            consoleInput.setFocus();
-                        }
-                    }
-                });
-                prompting = true;
+            initPrompt(prompt.apply());
             promptLock.notifyAll();
+            try {
                 while (prompting) {
                     promptLock.wait();
                 }
             } catch (final InterruptedException e) {
-                LOGGER.log(Level.WARNING, "Error in synchronization of prompting", e); //$NON-NLS-1$
+                LOGGER.log(Level.WARNING, ERROR_SYNCHRO_PROMPT, e);
                 command = null;
                 Thread.currentThread().interrupt();
             }
@@ -283,57 +273,14 @@ public final class SWTConsole extends Composite
         return command;
     }
 
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(long) */
-    @Override
-    public String prompt(final long timeout) throws IOException {
-        synchronized (promptLock) {
+    /** Initialize the prompting.
+     * 
+     * @param message the prompt message
+     * @throws IOException if the console is disposed */
+    private void initPrompt(final String message) throws IOException {
         if (isDisposed()) {
             throw new IOException();
         }
-            try {
-                Display.getDefault().syncExec(new Runnable() {
-                    @Override
-                    public void run() {
-                        if (!consoleInput.isDisposed()) {
-                            consoleInput.setEnabled(true);
-                            lblPromptlabel.setText(prompt.apply());
-                            // relayout
-                            SWTConsole.this.layout();
-                            consoleInput.setFocus();
-                        }
-                    }
-                });
-                prompting = true;
-                command = null;
-                promptLock.notifyAll();
-                final long start = System.currentTimeMillis();
-                long cur = start;
-                while (prompting && start + timeout>cur) {
-                    promptLock.wait((cur-start-timeout)/2);
-                    cur = System.currentTimeMillis();
-                }
-            } catch (final InterruptedException e) {
-                LOGGER.log(Level.WARNING, "Error in synchronization of prompting", e); //$NON-NLS-1$
-                command = null;
-                Thread.currentThread().interrupt();
-            }
-        }
-        if (isDisposed()) {
-            throw new IOException("Input closed"); //$NON-NLS-1$
-        }
-        return command;
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.ConsoleManager#prompt(java.lang.String) */
-    @Override
-    public String prompt(final String message) throws IOException {
-        synchronized (promptLock) {
-            if (isDisposed()) {
-                throw new IOException();
-            }
-            try {
         Display.getDefault().syncExec(new Runnable() {
             @SuppressWarnings("synthetic-access")
             @Override
@@ -348,15 +295,92 @@ public final class SWTConsole extends Composite
             }
         });
         prompting = true;
+        command = null;
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(long) */
+    @Override
+    public String prompt(final long timeout) throws IOException {
+        synchronized (promptLock) {
+            initPrompt(prompt.apply());
+            promptLock.notifyAll();
+            try {
+                waitPromptResolution(timeout);
+            } catch (final InterruptedException e) {
+                LOGGER.log(Level.WARNING, ERROR_SYNCHRO_PROMPT, e);
+                command = null;
+                Thread.currentThread().interrupt();
+            }
+        }
+        return command;
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.ConsoleManager#prompt(java.lang.String) */
+    @Override
+    public String prompt(final String message) throws IOException {
+        synchronized (promptLock) {
+            initPrompt(message);
+            promptLock.notifyAll();
+            try {
+                while (prompting) {
                     promptLock.wait();
+                }
                 if (isDisposed()) {
                     throw new IOException();
                 }
             } catch (final InterruptedException e) {
-                LOGGER.log(Level.WARNING, "Error in synchronization of prompting", e); //$NON-NLS-1$
+                LOGGER.log(Level.WARNING, ERROR_SYNCHRO_PROMPT, e);
                 command = null;
                 Thread.currentThread().interrupt();
             } finally {
+                resetPrompt();
+            }
+        }
+        return command;
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String, long) */
+    @Override
+    public String prompt(final String message, final long timeout) throws IOException {
+        synchronized (promptLock) {
+            initPrompt(message);
+            promptLock.notifyAll();
+            try {
+                waitPromptResolution(timeout);
+            } catch (final InterruptedException e) {
+                LOGGER.log(Level.WARNING, ERROR_SYNCHRO_PROMPT, e);
+                command = null;
+                Thread.currentThread().interrupt();
+            } finally {
+                resetPrompt();
+            }
+        }
+        return command;
+    }
+
+    /** Wait the prompt resolution by user.
+     * 
+     * @param timeout the timeout for the wait
+     * @throws InterruptedException if the thread is interrupted
+     * @throws IOException if the console is disposed. */
+    private void waitPromptResolution(final long timeout)
+            throws InterruptedException, IOException {
+        final long start = System.currentTimeMillis();
+        long cur = start;
+        while (prompting && start + timeout > cur) {
+            promptLock.wait((cur - start - timeout) / TWO);
+            cur = System.currentTimeMillis();
+        }
+        if (isDisposed()) {
+            throw new IOException();
+        }
+    }
+
+    /** Reset the prompt message. */
+    private void resetPrompt() {
         Display.getDefault().syncExec(new Runnable() {
             @SuppressWarnings("synthetic-access")
             @Override
@@ -369,18 +393,6 @@ public final class SWTConsole extends Composite
             }
         });
     }
-        }
-        return command;
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String, long) */
-    @Override
-    public String prompt(final String message, final long timeout) throws IOException {
-        // TODO Auto-generated method stub
-        // return null;
-        throw new RuntimeException("Not implemented yet");
-    }
 
     /* (non-Javadoc)
      * @see org.eclipse.swt.widgets.Composite#setFocus() */
@@ -397,6 +409,13 @@ public final class SWTConsole extends Composite
         consoleInput.setSelection(input.length());
     }
 
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.manager.ConsoleInput#setPrompt(java.lang.String) */
+    @Override
+    public void setPrompt(final String prompt) {
+        setPrompt(new ConstantString(prompt));
+    }
+
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.ConsoleManager#setPrompt(java.lang.String) */
     @Override
@@ -415,26 +434,15 @@ public final class SWTConsole extends Composite
         });
     }
 
-    @Override
-    public void setPrompt(String prompt) {
-        setPrompt(new ConstantString(prompt));
-    }
-
-    /** @param string the text */
+    /** Set the input text.
+     * 
+     * @param string the text */
     public void setText(final String string) {
         consoleInput.setText(string);
     }
 
-    /**
-     *
-     */
-    public void validateCommand() {
-        validateInput();
-    }
-
-    /**
-     *
-     */
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.swt.ConsoleDelayIO#validateInput() */
     @Override
     public void validateInput() {
         Display.getDefault().syncExec(new Runnable() {
@@ -450,6 +458,7 @@ public final class SWTConsole extends Composite
                     promptLock.wait();
                 } catch (final InterruptedException e) {
                     LOGGER.log(Level.SEVERE, "Interruption while waiting prompt", e); //$NON-NLS-1$
+                    Thread.currentThread().interrupt();
                 }
             }
             Display.getDefault().syncExec(new Runnable() {

gclc-swt/src/main/java/net/bigeon/gclc/swt/SWTConsoleShell.java

@@ -85,8 +85,8 @@ public final class SWTConsoleShell extends Shell {
     /** Create the shell.
      *
      * @param display the display */
-    public SWTConsoleShell(final Display display) {
-        super(display, SWT.SHELL_TRIM);
+    public SWTConsoleShell(final Display display, final int style) {
+        super(display, style);
         setLayout(new FillLayout(SWT.HORIZONTAL));
 
         createContents();
@@ -98,7 +98,7 @@ public final class SWTConsoleShell extends Shell {
     }
 
     /** Create contents of the shell. */
-    protected void createContents() {
+    private void createContents() {
         console = new SWTConsole(this, SWT.NONE);
         setText("Console Application"); //$NON-NLS-1$
     }

gclc-swt/src/main/java/net/bigeon/gclc/swt/SWTConsoleView.java

@@ -92,8 +92,7 @@ public final class SWTConsoleView extends Composite implements ConsoleDelayIO {
     /** The local implementation of the forwarding runnable
      *
      * @author Emmanuel Bigeon */
-    private final class ToSWTConsoleForwardRunnable
-            extends AOutputForwardRunnable {
+    private final class ToSWTConsoleForwardRunnable extends AOutputForwardRunnable {
         /** The running status */
         private boolean running = true;
 
@@ -104,7 +103,7 @@ public final class SWTConsoleView extends Composite implements ConsoleDelayIO {
 
         @Override
         protected void forwardLine(final String m) {
-            appendConsoleOutput(m);
+            appendConsoleOutput(System.lineSeparator() + m);
         }
 
         @Override
@@ -139,25 +138,23 @@ public final class SWTConsoleView extends Composite implements ConsoleDelayIO {
 
         setLayout(new GridLayout(1, false));
 
-        consoleOutput = new Text(this, SWT.BORDER | SWT.READ_ONLY | SWT.WRAP |
-                                       SWT.V_SCROLL | SWT.MULTI);
-        consoleOutput.setLayoutData(new GridData(SWT.FILL, SWT.FILL, true, true,
-                1, 1));
+        consoleOutput = new Text(this,
+                SWT.BORDER | SWT.READ_ONLY | SWT.WRAP | SWT.V_SCROLL | SWT.MULTI);
+        consoleOutput.setLayoutData(new GridData(SWT.FILL, SWT.FILL, true, true, 1, 1));
         consoleOutput.setRedraw(true);
 
         consoleInput = new Text(this, SWT.BORDER);
-        consoleInput.setLayoutData(
-                new GridData(SWT.FILL, SWT.CENTER, true, false, 1, 1));
+        consoleInput.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false, 1, 1));
         consoleInput.addKeyListener(new HistoryTextKeyListener(this));
     }
 
     /** @param next the next message */
-    protected void appendConsoleOutput(final String next) {
+    public void appendConsoleOutput(final String next) {
         Display.getDefault().syncExec(new Runnable() {
             @SuppressWarnings("synthetic-access")
             @Override
             public void run() {
-                consoleOutput.append(System.lineSeparator() + next);
+                consoleOutput.append(next);
             }
         });
     }

gclc-swt/src/site/site.xml

@@ -0,0 +1,44 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+  #%L
+  Generic Command Ligne console
+  %%
+  Copyright (C) 2014 - 2018 Bigeon
+  %%
+  This software is governed by the CeCILL  license under French law and
+  abiding by the rules of distribution of free software.  You can  use, 
+  modify and/ or redistribute the software under the terms of the CeCILL
+  license as circulated by CEA, CNRS and INRIA at the following URL
+  "http://www.cecill.info". 
+  
+  As a counterpart to the access to the source code and  rights to copy,
+  modify and redistribute granted by the license, users are provided only
+  with a limited warranty  and the software's author,  the holder of the
+  economic rights,  and the successive licensors  have only  limited
+  liability. 
+  
+  In this respect, the user's attention is drawn to the risks associated
+  with loading,  using,  modifying and/or developing or reproducing the
+  software by the user in light of its specific status of free software,
+  that may mean  that it is complicated to manipulate,  and  that  also
+  therefore means  that it is reserved for developers  and  experienced
+  professionals having in-depth computer knowledge. Users are therefore
+  encouraged to load and test the software's suitability as regards their
+  requirements in conditions enabling the security of their systems and/or 
+  data to be ensured and,  more generally, to use and operate it in the 
+  same conditions as regards security. 
+  
+  The fact that you are presently reading this means that you have had
+  knowledge of the CeCILL license and that you accept its terms.
+  #L%
+  -->
+
+<project name="GCLC">
+  <bannerRight>
+    <src>https://bigeon.net/images/logo_48.png</src>
+    <href>https://bigeon.net/</href>
+  </bannerRight>
+  <body>
+    <menu ref="reports"/>
+  </body>
+</project>

gclc-swt/src/test/java/net/bigeon/gclc/swt/HistoryTextKeyListenerTest.java

@@ -73,7 +73,8 @@ import static org.junit.Assert.assertEquals;
 import org.eclipse.swt.SWT;
 import org.junit.Test;
 
-/** <p>
+/**
+ * <p>
  * TODO
  *
  * @author Emmanuel Bigeon */
@@ -87,8 +88,8 @@ public class HistoryTextKeyListenerTest {
             private String input = "";
 
             @Override
-            public void validateInput() {
-                input = "";
+            public String getInput() {
+                return input;
             }
 
             @Override
@@ -97,8 +98,8 @@ public class HistoryTextKeyListenerTest {
             }
 
             @Override
-            public String getInput() {
-                return input;
+            public void validateInput() {
+                input = "";
             }
         };
         HistoryTextKeyListener listener = new HistoryTextKeyListener(io);

gclc-swt/src/test/java/net/bigeon/gclc/swt/SWTConsoleShellTest.java

@@ -75,6 +75,7 @@ import static org.junit.Assert.fail;
 
 import java.io.IOException;
 
+import org.eclipse.swt.SWT;
 import org.eclipse.swt.widgets.Display;
 import org.junit.Test;
 
@@ -91,19 +92,19 @@ import net.bigeon.gclc.manager.ConsoleOutput;
  * TODO
  *
  * @author Emmanuel Bigeon */
-@SuppressWarnings({"javadoc", "static-method", "nls", "deprecation"})
+@SuppressWarnings({ "javadoc", "static-method", "nls", "deprecation" })
 public class SWTConsoleShellTest {
 
     protected static final long  TWO_SECONDS = 2000;
     private static final Display DISPLAY     = Display.getDefault();
 
     @Test
-    public void test() {
-        final SWTConsoleShell shell = new SWTConsoleShell(DISPLAY);
+    public void testGeneric() {
+        final SWTConsoleShell shell = new SWTConsoleShell(DISPLAY, SWT.SHELL_TRIM);
         final SWTConsole swtConsole = shell.getManager();
         try {
-            final ConsoleApplication appl = new ConsoleApplication(swtConsole,
-                    swtConsole, "Hello", "See you");
+            final ConsoleApplication appl = new ConsoleApplication(swtConsole, swtConsole,
+                    "Hello", "See you");
             appl.add(new ExitCommand("exit", appl));
             appl.add(new Command("long") {
 
@@ -112,8 +113,7 @@ public class SWTConsoleShellTest {
                  * manager.ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
                  * java.lang.String[]) */
                 @Override
-                public void execute(final ConsoleOutput out,
-                                    final ConsoleInput in,
+                public void execute(final ConsoleOutput out, final ConsoleInput in,
                         final String... args) throws CommandRunException {
                     try {
                         Thread.sleep(TWO_SECONDS);
@@ -139,6 +139,7 @@ public class SWTConsoleShellTest {
             });
 // shell.pack();
             shell.open();
+            shell.setVisible(false);
             final Thread applThread = new Thread(new Runnable() {
 
                 @Override
@@ -160,7 +161,7 @@ public class SWTConsoleShellTest {
                         @Override
                         public void run() {
                             swtConsole.setText("test"); //$NON-NLS-1$
-                            swtConsole.validateCommand();
+                            swtConsole.validateInput();
                         }
                     });
                     try {
@@ -199,13 +200,13 @@ public class SWTConsoleShellTest {
 
     @Test
     public void testConsoleClose() {
-        final SWTConsoleShell shell = new SWTConsoleShell(DISPLAY);
+        final SWTConsoleShell shell = new SWTConsoleShell(DISPLAY, SWT.SHELL_TRIM);
         final SWTConsole swtConsole = shell.getManager();
         swtConsole.close();
         swtConsole.setPrompt(":");
         try {
-            final ConsoleApplication appl = new ConsoleApplication(swtConsole,
-                    swtConsole, "Hello", "See you");
+            final ConsoleApplication appl = new ConsoleApplication(swtConsole, swtConsole,
+                    "Hello", "See you");
             appl.add(new ExitCommand("exit", appl));
             appl.add(new Command("long") {
                 /* (non-Javadoc)
@@ -213,8 +214,7 @@ public class SWTConsoleShellTest {
                  * manager.ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
                  * java.lang.String[]) */
                 @Override
-                public void execute(final ConsoleOutput out,
-                                    final ConsoleInput in,
+                public void execute(final ConsoleOutput out, final ConsoleInput in,
                         final String... args) throws CommandRunException {
                     try {
                         Thread.sleep(TWO_SECONDS);
@@ -244,8 +244,7 @@ public class SWTConsoleShellTest {
                  * manager.ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
                  * java.lang.String[]) */
                 @Override
-                public void execute(final ConsoleOutput out,
-                                    final ConsoleInput in,
+                public void execute(final ConsoleOutput out, final ConsoleInput in,
                         final String... args) throws CommandRunException {
                     try {
                         swtConsole.prompt("Test");
@@ -270,6 +269,7 @@ public class SWTConsoleShellTest {
             });
 // shell.pack();
             shell.open();
+            shell.setVisible(false);
             final Thread applThread = new Thread(new Runnable() {
 
                 @Override
@@ -291,7 +291,7 @@ public class SWTConsoleShellTest {
                         @Override
                         public void run() {
                             swtConsole.setText("test"); //$NON-NLS-1$
-                            swtConsole.validateCommand();
+                            swtConsole.validateInput();
                         }
                     });
                     Display.getDefault().syncExec(new Runnable() {
@@ -300,28 +300,28 @@ public class SWTConsoleShellTest {
                             swtConsole.setText("ok"); //$NON-NLS-1$
                         }
                     });
-                    swtConsole.validateCommand();
+                    swtConsole.validateInput();
                     Display.getDefault().syncExec(new Runnable() {
                         @Override
                         public void run() {
                             swtConsole.setText("long"); //$NON-NLS-1$
                         }
                     });
-                    swtConsole.validateCommand();
+                    swtConsole.validateInput();
                     Display.getDefault().syncExec(new Runnable() {
                         @Override
                         public void run() {
                             swtConsole.setText("test"); //$NON-NLS-1$
                         }
                     });
-                    swtConsole.validateCommand();
+                    swtConsole.validateInput();
                     Display.getDefault().syncExec(new Runnable() {
                         @Override
                         public void run() {
                             swtConsole.setText("test"); //$NON-NLS-1$
                         }
                     });
-                    swtConsole.validateCommand();
+                    swtConsole.validateInput();
                     try {
                         Thread.sleep(TWO_SECONDS);
                     } catch (final InterruptedException e) {

gclc-swt/src/test/java/net/bigeon/gclc/swt/SWTConsoleViewTest.java

@@ -87,18 +87,19 @@ import net.bigeon.gclc.manager.ConsoleOutput;
 import net.bigeon.gclc.utils.PipedConsoleInput;
 import net.bigeon.gclc.utils.PipedConsoleOutput;
 
-/** <p>
+/**
+ * <p>
  * TODO
  *
  * @author Emmanuel Bigeon */
-@SuppressWarnings({"javadoc", "static-method", "nls", "deprecation"})
+@SuppressWarnings({ "javadoc", "static-method", "nls", "deprecation" })
 public class SWTConsoleViewTest {
 
     protected static final long  TWO_SECONDS = 2000;
     private static final Display DISPLAY     = Display.getDefault();
 
     @Test
-    public void test() {
+    public void testGeneric() {
         final Shell shell = new Shell(DISPLAY);
         final SWTConsoleView swtConsole = new SWTConsoleView(shell, SWT.NONE);
         try (PipedConsoleOutput manager = new PipedConsoleOutput();
@@ -110,15 +111,15 @@ public class SWTConsoleViewTest {
         try (PipedConsoleOutput manager = new PipedConsoleOutput();
                 PipedConsoleInput input = new PipedConsoleInput(System.out)) {
             swtConsole.setManager(manager, input);
-            final ConsoleApplication appl = new ConsoleApplication(manager,
-                    input,
+            final ConsoleApplication appl = new ConsoleApplication(manager, input,
                     "Hello", "See you");
             appl.add(new ExitCommand("exit", appl));
             appl.add(new Command("long") {
 
                 /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[])
-                 */
+                 * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
+                 * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
+                 * java.lang.String[]) */
                 @Override
                 public void execute(final ConsoleOutput out, final ConsoleInput in,
                         final String... args) throws CommandRunException {
@@ -146,6 +147,7 @@ public class SWTConsoleViewTest {
             });
 // shell.pack();
             shell.open();
+            shell.setVisible(false);
             final Thread applThread = new Thread(new Runnable() {
 
                 @Override

gclc.system/pom.xml

@@ -1,36 +1,22 @@
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
 	<modelVersion>4.0.0</modelVersion>
+	<parent>
+		<groupId>net.bigeon.config</groupId>
+		<artifactId>ebigeon-config</artifactId>
+		<version>1.8.21</version>
+	</parent>
 	<groupId>net.bigeon.gclc</groupId>
 	<artifactId>system</artifactId>
-  <version>0.0.2-SNAPSHOT</version>
+	<version>0.0.2</version>
 	<packaging>jar</packaging>
-  <url>http://www.bigeon.net</url>
-  <properties>
-    <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-    <copyright.email>emmanuel@bigeon.fr</copyright.email>
-    <license.licenseName>cecill_2.1</license.licenseName>
-  </properties>
-  <build>
-    <plugins>
-    </plugins>
-  </build>
-  <dependencies>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>4.11</version>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>net.bigeon</groupId>
-      <artifactId>gclc</artifactId>
-      <version>2.0.7</version>
-    </dependency>
-  </dependencies>
 	<name>GCLC system command</name>
 	<description>Provide an exec command to execute system commands</description>
+	<url>https://bigeon.net/projects/gclc.html</url>
 	<inceptionYear>2016</inceptionYear>
-  
+	<organization>
+		<name>Bigeon</name>
+		<url>https://bigeon.net</url>
+	</organization>
 	<licenses>
 		<license>
 			<distribution>manual</distribution>
@@ -38,15 +24,6 @@
 			<url>https://cecill.info/licences/Licence_CeCILL_V2.1-en.html</url>
 		</license>
 	</licenses>
-	<reporting>
-		<plugins>
-			<plugin>
-				<groupId>com.github.sevntu-checkstyle</groupId>
-				<artifactId>dsm-maven-plugin</artifactId>
-				<version>2.2.0</version>
-			</plugin>
-		</plugins>
-	</reporting>
 	<developers>
 		<developer>
 			<email>emmanuel@bigeon.fr</email>
@@ -57,14 +34,40 @@
 			</roles>
 		</developer>
 	</developers>
-
 	<scm>
 		<developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
-    <tag>HEAD</tag>
+		<tag>system-0.0.2</tag>
 	</scm>
-  <parent>
-    <groupId>net.bigeon.config</groupId>
-    <artifactId>ebigeon-config</artifactId>
-    <version>1.8.9</version>
-  </parent>
+	<properties>
+		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
+		<copyright.email>emmanuel@bigeon.fr</copyright.email>
+		<license.licenseName>cecill_2.1</license.licenseName>
+	</properties>
+
+	<dependencies>
+		<dependency>
+			<groupId>net.bigeon</groupId>
+			<artifactId>gclc</artifactId>
+			<version>2.0.11</version>
+		</dependency>
+	</dependencies>
+
+	<build>
+		<plugins>
+		</plugins>
+	</build>
+	<reporting>
+		<plugins>
+			<plugin>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-javadoc-plugin</artifactId>
+				<version>2.10.3</version>
+			</plugin>
+			<plugin>
+				<groupId>com.github.sevntu-checkstyle</groupId>
+				<artifactId>dsm-maven-plugin</artifactId>
+				<version>2.2.0</version>
+			</plugin>
+		</plugins>
+	</reporting>
 </project>

gclc.system/src/main/java/net/bigeon/gclc/system/ExecSystemCommand.java

@@ -39,11 +39,8 @@ package net.bigeon.gclc.system;
  */
 import java.io.BufferedWriter;
 import java.io.IOException;
-import java.io.InputStream;
 import java.io.OutputStream;
 import java.io.OutputStreamWriter;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 import net.bigeon.gclc.command.Command;
 import net.bigeon.gclc.exception.CommandRunException;
@@ -51,34 +48,38 @@ import net.bigeon.gclc.exception.CommandRunExceptionType;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
 
-/** A command that will execute a system command.
+/** A command that will execute system commands.
+ * <p>
+ * Note that this class may induce security issues in the code as it can execute
+ * arbitrary system commands. It is recommended that it be used only in private
+ * applications or application used by aware parties.
  *
  * @author Emmanuel Bigeon */
 public class ExecSystemCommand extends Command {
 
+    /** Error message for the manager closed. */
+    private static final String MANAGER_WAS_CLOSED   = "manager was closed";
     /** The command default name */
     private static final String COMMAND_DEFAULT_NAME = "exec";                //$NON-NLS-1$
     /** The end of line separator */
     private static final String EOL                  = System.lineSeparator();
-    /** The class logger */
-    private static final Logger LOGGER = Logger
-            .getLogger(ExecSystemCommand.class.getName());
 
-    /***/
+    /** Create the command with the name 'exec'. */
     public ExecSystemCommand() {
         super(COMMAND_DEFAULT_NAME);
     }
 
-    /** @param name the name of the command (the input from the manager that
-     *            should trigger this command) */
+    /** Create the command with a custom name.
+     * 
+     * @param name the name of the command (the input from the manager that should
+     *        trigger this command) */
     public ExecSystemCommand(final String name) {
         super(name);
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
-     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-     * java.lang.String[]) */
+     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
     @Override
     public void execute(final ConsoleOutput out, final ConsoleInput in,
             final String... args) throws CommandRunException {
@@ -86,68 +87,28 @@ public class ExecSystemCommand extends Command {
         try {
             proc = Runtime.getRuntime().exec(args);
         } catch (final IOException e2) {
-            LOGGER.log(Level.SEVERE, "Unable to run process", e2); //$NON-NLS-1$
-            return;
+            throw new CommandRunException("Unable to run process", e2);
         }
-
-        final InputStream is = proc
-                .getInputStream();
-        final Thread th = new Thread(new Runnable() {
-
-            @SuppressWarnings("synthetic-access")
-            @Override
-            public void run() {
-                try {
-                    readToEnd(out, is);
-                    is.close();
-                } catch (final CommandRunException e) {
-                    LOGGER.log(Level.WARNING,
-                            "Manager was closed in the meantime...", e); //$NON-NLS-1$
-                } catch (final IOException e) {
-                    LOGGER.log(Level.WARNING, "Input stream was closed...", e); //$NON-NLS-1$
-                }
-            }
-        });
-        th.start();
+        // Stream forwarding to the application's console
+        // This is started in a parallel thread.
+        final Thread th = ForwardingRunnable.connect(proc.getInputStream(), out,
+                "Command output");
+        // Set the empty prompt for the processes promptings.
         in.setPrompt(""); //$NON-NLS-1$
+        // Forward console input to the process, whether the process is waiting or not
+        // for input.
         final OutputStream os = proc.getOutputStream();
-        try (BufferedWriter writer = new BufferedWriter(
-                new OutputStreamWriter(os))) {
+        try (BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(os))) {
             while (th.isAlive()) {
-                String user;
-                try {
-                    user = in.prompt();
-                } catch (final IOException e) {
-                    throw new CommandRunException(
-                            CommandRunExceptionType.INTERACTION,
-                            "manager was closed", e); //$NON-NLS-1$
-                }
+                final String user = in.prompt();
+                // Avoid interruption being sent to process.
+                if (!user.isEmpty()) {
                     writer.write(user + EOL);
                 }
+            }
         } catch (final IOException e1) {
             throw new CommandRunException(CommandRunExceptionType.INTERACTION,
-                    "manager was closed", e1); //$NON-NLS-1$
-        }
-    }
-
-    /** @param is the input stream
-     * @throws CommandRunException if the manager was closed while writing the
-     *             stream */
-    protected void readToEnd(final ConsoleOutput out,
-                             final InputStream is) throws CommandRunException {
-        int c;
-        try {
-            while ((c = is.read()) != -1) {
-                try {
-                    out.print(Character.valueOf((char) c).toString());
-                } catch (final IOException e) {
-                    throw new CommandRunException(
-                            CommandRunExceptionType.INTERACTION,
-                            "manager was closed", e); //$NON-NLS-1$
-                }
-            }
-        } catch (final IOException e) {
-            LOGGER.log(Level.INFO, "input stream reading failed", e); //$NON-NLS-1$
+                    MANAGER_WAS_CLOSED, e1); // $NON-NLS-1$
         }
     }
 
@@ -164,10 +125,9 @@ public class ExecSystemCommand extends Command {
     protected String usageDetail() {
         return "  The system command is a system dependend command like sh on linux or" + //$NON-NLS-1$
                 System.lineSeparator() + "powershell on windows." + //$NON-NLS-1$
-               System.lineSeparator() + System.lineSeparator() +
-               "  As an example if you give \"cat /etc/hostname\" as argument, on a linux" + //$NON-NLS-1$
-               System.lineSeparator() +
-               "system, you would get the computer name." + //$NON-NLS-1$
+                System.lineSeparator() + System.lineSeparator()
+                + "  As an example if you give \"cat\", \"/etc/hostname\" as arguments, on a linux" //$NON-NLS-1$
+                + System.lineSeparator() + "system, you would get the computer name." + //$NON-NLS-1$
                 System.lineSeparator();
     }
 
@@ -177,5 +137,4 @@ public class ExecSystemCommand extends Command {
     protected String usagePattern() {
         return "  CMD <system command>"; //$NON-NLS-1$
     }
-
 }

gclc.system/src/main/java/net/bigeon/gclc/system/ForwardingRunnable.java

@@ -0,0 +1,144 @@
+package net.bigeon.gclc.system;
+
+/*-
+ * #%L
+ * GCLC system command
+ * %%
+ * Copyright (C) 2016 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
+import java.io.IOException;
+import java.io.InputStream;
+import java.util.logging.Level;
+import java.util.logging.Logger;
+
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
+import net.bigeon.gclc.manager.ConsoleOutput;
+
+/** A class forwarding the content of a stream into a console output.
+ * <p>
+ * This class is useful for processes that run in a different environment than
+ * the application but return an output. Such processes include command
+ * processes and socket communications for example.
+ * <p>
+ * In the current library is is used to retrieve a system command output.
+ * 
+ * @author Emmanuel Bigeon */
+public final class ForwardingRunnable implements Runnable {
+    /** Error message for the manager closed. */
+    private static final String MANAGER_WAS_CLOSED = "manager was closed";
+    /** The class logger */
+    private static final Logger LOGGER             = Logger
+            .getLogger(ForwardingRunnable.class.getName());
+
+    /** The console output. */
+    private final ConsoleOutput out;
+    /** The input stream. */
+    private final InputStream   is;
+
+    /** Create the runnable.
+     * 
+     * @param out the output console
+     * @param is the input stream */
+    public ForwardingRunnable(final ConsoleOutput out, final InputStream is) {
+        this.out = out;
+        this.is = is;
+    }
+
+    /* (non-Javadoc)
+     * @see java.lang.Runnable#run() */
+    @Override
+    public void run() {
+        try {
+            readToEnd(out, is);
+            is.close();
+        } catch (final CommandRunException e) {
+            LOGGER.log(Level.WARNING, "Manager was closed in the meantime...", e); //$NON-NLS-1$
+        } catch (final IOException e) {
+            LOGGER.log(Level.WARNING, "Input stream was closed...", e); //$NON-NLS-1$
+        }
+    }
+
+    /** Read the input until its end.
+     * 
+     * @param is the input stream
+     * @throws CommandRunException if the manager was closed while writing the
+     *         stream */
+    private static void readToEnd(final ConsoleOutput out, final InputStream is)
+            throws CommandRunException {
+        int c;
+        try {
+            while ((c = is.read()) >= 0) {
+                // Print the character in the output
+                printCharacter(out, (char) c);
+            }
+        } catch (final IOException e) {
+            LOGGER.log(Level.INFO, "input stream reading failed", e); //$NON-NLS-1$
+        }
+    }
+
+    /** Print a character on the console.
+     * 
+     * @param out the console
+     * @param c the character
+     * @throws CommandRunException if the console failed to print the character. */
+    private static void printCharacter(final ConsoleOutput out, final char c)
+            throws CommandRunException {
+        try {
+            out.print(Character.toString(c));
+        } catch (final IOException e) {
+            throw new CommandRunException(CommandRunExceptionType.INTERACTION,
+                    MANAGER_WAS_CLOSED, e); // $NON-NLS-1$
+        }
+    }
+
+    /** Tool method to connect an input stream to a console output.
+     * 
+     * @param stream the stream
+     * @param output the output
+     * @param name the identification of the thread
+     * @return the thread the forwarding runnable runs into. */
+    public static Thread connect(final InputStream stream, final ConsoleOutput output,
+            final String name) {
+        final Thread th = new Thread(new ForwardingRunnable(output, stream), name);
+        th.start();
+        return th;
+    }
+
+    /** Tool method to connect an input stream to a console output.
+     * 
+     * @param stream the stream
+     * @param output the output
+     * @return the thread the forwarding runnable runs into. */
+    public static Thread connect(final InputStream stream, final ConsoleOutput output) {
+        return connect(stream, output, "Forwarding");
+    }
+}

gclc.system/src/main/java/net/bigeon/gclc/system/package-info.java

@@ -0,0 +1,42 @@
+/** This package contains classes for command execution on the system.
+ * <p>
+ * The class {@link net.bigeon.gclc.system.ExecSystemCommand} is, in the current
+ * version (0.0.1), a source of vulnerability and should probably be used only
+ * in private projects where the user is aware of the dangers.
+ * 
+ * @author Emmanuel Bigeon */
+package net.bigeon.gclc.system;
+
+/*-
+ * #%L
+ * GCLC system command
+ * %%
+ * Copyright (C) 2016 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */

gclc.system/src/site/site.xml

@@ -0,0 +1,44 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+  #%L
+  Generic Command Ligne console
+  %%
+  Copyright (C) 2014 - 2018 Bigeon
+  %%
+  This software is governed by the CeCILL  license under French law and
+  abiding by the rules of distribution of free software.  You can  use, 
+  modify and/ or redistribute the software under the terms of the CeCILL
+  license as circulated by CEA, CNRS and INRIA at the following URL
+  "http://www.cecill.info". 
+  
+  As a counterpart to the access to the source code and  rights to copy,
+  modify and redistribute granted by the license, users are provided only
+  with a limited warranty  and the software's author,  the holder of the
+  economic rights,  and the successive licensors  have only  limited
+  liability. 
+  
+  In this respect, the user's attention is drawn to the risks associated
+  with loading,  using,  modifying and/or developing or reproducing the
+  software by the user in light of its specific status of free software,
+  that may mean  that it is complicated to manipulate,  and  that  also
+  therefore means  that it is reserved for developers  and  experienced
+  professionals having in-depth computer knowledge. Users are therefore
+  encouraged to load and test the software's suitability as regards their
+  requirements in conditions enabling the security of their systems and/or 
+  data to be ensured and,  more generally, to use and operate it in the 
+  same conditions as regards security. 
+  
+  The fact that you are presently reading this means that you have had
+  knowledge of the CeCILL license and that you accept its terms.
+  #L%
+  -->
+
+<project name="GCLC">
+  <bannerRight>
+    <src>https://bigeon.net/images/logo_48.png</src>
+    <href>https://bigeon.net/</href>
+  </bannerRight>
+  <body>
+    <menu ref="reports"/>
+  </body>
+</project>

gclc.system/src/test/java/net/bigeon/gclc/system/ExecSystemCommandTest.java

@@ -0,0 +1,82 @@
+package net.bigeon.gclc.system;
+
+/*-
+ * #%L
+ * GCLC system command
+ * %%
+ * Copyright (C) 2016 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.fail;
+
+import org.junit.Test;
+
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.manager.ConsoleInput;
+import net.bigeon.gclc.manager.ConsoleOutput;
+import net.bigeon.gclc.utils.EmptyInput;
+import net.bigeon.gclc.utils.SinkOutput;
+
+public class ExecSystemCommandTest {
+
+    /** Test the help methods of the command. */
+    @Test
+    public void testExecSystemCommand() {
+        final ExecSystemCommand cmd = new ExecSystemCommand("test");
+        assertEquals("Name should be preserved", "test", cmd.getCommandName());
+        assertNotNull("tip should be defined", cmd.tip());
+        assertNotNull("usage should be defined", cmd.usagePattern());
+        assertNotNull("usage should be defined", cmd.usageDetail());
+    }
+
+    /** Test the execution of the command.
+     * 
+     * @throws CommandRunException */
+    @Test
+    public void testExecute() throws CommandRunException {
+        final ConsoleOutput out = SinkOutput.INSTANCE;
+        final ConsoleInput in = EmptyInput.INSTANCE;
+
+        final ExecSystemCommand cmd = new ExecSystemCommand();
+        if (System.getProperty("os.name").contains("indows")) {
+            cmd.execute(out, in, "cmd", "/C", "dir");
+        } else if (System.getProperty("os.name").contains("inux")) {
+            cmd.execute(out, in, "ls");
+        }
+        try {
+            cmd.execute(out, in, "inexistent");
+            fail("Able to execute inexistent command in system");
+        } catch (final CommandRunException e) {
+            // ok
+        }
+    }
+
+}

gclc/pom.xml

@@ -1,10 +1,44 @@
 
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
 	<modelVersion>4.0.0</modelVersion>
+	<parent>
+		<groupId>net.bigeon.config</groupId>
+		<artifactId>ebigeon-config</artifactId>
+		<version>1.8.21</version>
+	</parent>
+	<groupId>net.bigeon</groupId>
 	<artifactId>gclc</artifactId>
-	<version>2.0.8</version>
+	<version>2.0.12-SNAPSHOT</version>
 	<packaging>jar</packaging>
-	<url>http://bigeon.net</url>
+	<name>Generic Command Ligne console</name>
+	<description>A generic framework for console applications, with customized command input and output streams.</description>
+	<url>https://bigeon.net/projects/gclc.html</url>
+	<inceptionYear>2014</inceptionYear>
+	<organization>
+		<name>Bigeon</name>
+		<url>https://bigeon.net/</url>
+	</organization>
+	<licenses>
+		<license>
+			<distribution>manual</distribution>
+			<name>CeCILL 2.1</name>
+			<url>https://cecill.info/licences/Licence_CeCILL_V2.1-en.html</url>
+		</license>
+	</licenses>
+	<developers>
+		<developer>
+			<email>emmanuel@bigeon.fr</email>
+			<name>Emmanuel Bigeon</name>
+			<url>bigeon.net</url>
+			<roles>
+				<role>PM</role>
+			</roles>
+		</developer>
+	</developers>
+	<scm>
+		<developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
+		<tag>HEAD</tag>
+	</scm>
 	<properties>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 		<project.scm.id>git.code.bigeon.net</project.scm.id>
@@ -13,17 +47,12 @@
 	</properties>
 	<dependencies>
 		<dependency>
-			<groupId>junit</groupId>
-			<artifactId>junit</artifactId>
-			<version>4.11</version>
+			<groupId>net.bigeon.test</groupId>
+			<artifactId>junitmt</artifactId>
+			<version>1.0.0</version>
 			<scope>test</scope>
 		</dependency>
 	</dependencies>
-	<parent>
-		<groupId>net.bigeon.config</groupId>
-		<artifactId>ebigeon-config</artifactId>
-		<version>1.8.9</version>
-	</parent>
 	<build>
 		<plugins>
 			<plugin>
@@ -38,43 +67,18 @@
 			</plugin>
 		</plugins>
 	</build>
-	<licenses>
-		<license>
-			<distribution>manual</distribution>
-			<name>CeCILL 2.1</name>
-			<url>https://cecill.info/licences/Licence_CeCILL_V2.1-en.html</url>
-		</license>
-	</licenses>
 	<reporting>
 		<plugins>
-			<plugin>
-				<groupId>org.apache.maven.plugins</groupId>
-				<artifactId>maven-javadoc-plugin</artifactId>
-				<version>2.10.3</version>
-			</plugin>
 			<plugin>
 				<groupId>com.github.sevntu-checkstyle</groupId>
 				<artifactId>dsm-maven-plugin</artifactId>
 				<version>2.2.0</version>
 			</plugin>
+			<plugin>
+				<groupId>org.codehaus.mojo</groupId>
+				<artifactId>jdepend-maven-plugin</artifactId>
+				<version>2.0</version>
+			</plugin>
 		</plugins>
 	</reporting>
-	<developers>
-		<developer>
-			<email>emmanuel@bigeon.fr</email>
-			<name>Emmanuel Bigeon</name>
-			<url>bigeon.net</url>
-			<roles>
-				<role>PM</role>
-			</roles>
-		</developer>
-	</developers>
-	<inceptionYear>2014</inceptionYear>
-	<name>Generic Command Ligne console</name>
-	<description>A generic framework for console applications, with customized command input and output streams.</description>
-	<scm>
-		<developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
-		<tag>gclc-2.0.8</tag>
-	</scm>
-	<groupId>net.bigeon</groupId>
 </project>

gclc/src/main/java/net/bigeon/gclc/ConsoleApplication.java

@@ -71,9 +71,10 @@ package net.bigeon.gclc;
  */
 import java.io.IOException;
 import java.io.InterruptedIOException;
-import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.LinkedHashSet;
 import java.util.List;
+import java.util.Set;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
@@ -88,9 +89,8 @@ import net.bigeon.gclc.i18n.Messages;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
 
-/**
- * A {@link ConsoleApplication} is an application that require the user to input
- * commands.
+/** A {@link ConsoleApplication} is an application that require the user to
+ * input commands.
  * <p>
  * A typical use case is the following:
  *
@@ -126,7 +126,7 @@ public final class ConsoleApplication implements ICommandProvider {
     /** The state of this application. */
     private boolean                            running;
     /** The listeners. */
-    private final List<CommandRequestListener> listeners = new ArrayList<>();
+    private final Set<CommandRequestListener> listeners = new LinkedHashSet<>();
 
     /** Create a console application.
      *
@@ -135,8 +135,7 @@ public final class ConsoleApplication implements ICommandProvider {
      * @param welcome the welcoming message
      * @param goodbye the goodbye message */
     public ConsoleApplication(final ConsoleOutput out, final ConsoleInput in,
-            final String welcome,
-            final String goodbye) {
+                              final String welcome, final String goodbye) {
         header = welcome;
         footer = goodbye;
         this.in = in;
@@ -161,8 +160,7 @@ public final class ConsoleApplication implements ICommandProvider {
      * java.lang.String[]) */
     @Override
     public void executeSub(final ConsoleOutput output, final ConsoleInput input,
-                           final String command,
-                                 final String... args) throws CommandRunException {
+            final String command, final String... args) throws CommandRunException {
         root.executeSub(output, input, command, args);
     }
 
@@ -182,8 +180,8 @@ public final class ConsoleApplication implements ICommandProvider {
 
     /** Interpret a command line.
      * <p>
-     * This method will split the command in its part and execute the command
-     * with {@link #executeSub(ConsoleOutput, ConsoleInput, String, String...)}.
+     * This method will split the command in its part and execute the command with
+     * {@link #executeSub(ConsoleOutput, ConsoleInput, String, String...)}.
      *
      * @param cmd the command
      * @throws IOException if the command could not be parsed */
@@ -191,19 +189,18 @@ public final class ConsoleApplication implements ICommandProvider {
         List<String> args;
         try {
             args = GCLCConstants.splitCommand(cmd);
-        } catch (final CommandParsingException e1) {
+        } catch (final CommandParsingException e) {
             out.println("Command line cannot be parsed"); //$NON-NLS-1$
-            LOGGER.log(Level.FINE, "Invalid user command " + cmd, e1); //$NON-NLS-1$
+            LOGGER.log(Level.FINE, "Invalid user command " + cmd, e); //$NON-NLS-1$
             return;
         }
         if (!args.isEmpty()) {
             try {
-                executeSub(out, in, args.get(0), Arrays.copyOfRange(
-                        args.toArray(new String[0]), 1, args.size()));
+                executeSub(out, in, args.get(0),
+                        Arrays.copyOfRange(args.toArray(new String[0]), 1, args.size()));
             } catch (final CommandRunException e) {
                 LOGGER.log(Level.FINE, "Command failed: " + cmd, e); //$NON-NLS-1$
-                out.println(Messages
-                        .getString("ConsoleApplication.cmd.failed", cmd)); //$NON-NLS-1$
+                out.println(Messages.getString("ConsoleApplication.cmd.failed", cmd)); //$NON-NLS-1$
                 out.println(e.getLocalizedMessage());
                 if (e.getType() == CommandRunExceptionType.USAGE) {
                     get(args.get(0)).help(out);
@@ -228,8 +225,8 @@ public final class ConsoleApplication implements ICommandProvider {
 
     /** The running loop content.
      * <p>
-     * This consisting in getting the command, executing it and exiting
-     * (restarting the loop). */
+     * This consisting in getting the command, executing it and exiting (restarting
+     * the loop). */
     private void runLoop() {
         try {
             final String cmd = in.prompt();
@@ -241,8 +238,7 @@ public final class ConsoleApplication implements ICommandProvider {
             }
             interpretCommand(cmd);
         } catch (final InterruptedIOException e) {
-            LOGGER.info(
-                    "Prompt interrupted. It is likely the application is closing."); //$NON-NLS-1$
+            LOGGER.info("Prompt interrupted. It is likely the application is closing."); //$NON-NLS-1$
             LOGGER.log(Level.FINER, "Interruption of the prompt.", //$NON-NLS-1$
                     e);
         } catch (final IOException e) {
@@ -250,8 +246,7 @@ public final class ConsoleApplication implements ICommandProvider {
             running = false;
             LOGGER.warning(
                     "The console manager was closed. Closing the application as no one can reach it."); //$NON-NLS-1$
-            LOGGER.log(Level.FINE,
-                    "An exception caused the closing of the application", //$NON-NLS-1$
+            LOGGER.log(Level.FINE, "An exception caused the closing of the application", //$NON-NLS-1$
                     e);
         }
     }
@@ -268,8 +263,7 @@ public final class ConsoleApplication implements ICommandProvider {
             running = false;
             LOGGER.warning(
                     "The console manager was closed. Closing the application as no one can reach it."); //$NON-NLS-1$
-            LOGGER.log(Level.FINE,
-                    "An exception caused the closing of the application", //$NON-NLS-1$
+            LOGGER.log(Level.FINE, "An exception caused the closing of the application", //$NON-NLS-1$
                     e);
             return;
         }

gclc/src/main/java/net/bigeon/gclc/GCLCConstants.java

@@ -85,7 +85,7 @@ import net.bigeon.gclc.exception.CommandParsingException;
 public final class GCLCConstants {
 
     /** The escaping character. */
-    private static final char ESCAPING_CHAR = getSystemEscapingChar();
+    private static final char ESCAPING_CHAR = '\\';
 
     /** Hide utility class constructor. */
     private GCLCConstants() {
@@ -108,13 +108,6 @@ public final class GCLCConstants {
         return cmd.substring(startIndex + 1, index - 1);
     }
 
-    /** Get the excaping character.
-     *
-     * @return the escaping character */
-    private static char getSystemEscapingChar() {
-        return '\\';
-    }
-
     /** Remove escaping characters from the string.
      *
      * @param arg the string to remove excaping character from
@@ -137,7 +130,8 @@ public final class GCLCConstants {
      * @param cmd the command to split in its parts
      * @return the list of argument preceded by the command name
      * @throws CommandParsingException if the parsing of the command failed */
-    public static List<String> splitCommand(final String cmd) throws CommandParsingException {
+    public static List<String> splitCommand(final String cmd)
+            throws CommandParsingException {
         final List<String> args = new ArrayList<>();
         // parse the string to separate arguments
         int index = 0;
@@ -164,6 +158,8 @@ public final class GCLCConstants {
                     startIndex = index;
                 }
                 inString = startIndex == index - 1;
+            } else {
+                // No special processing
             }
         }
         if (startIndex < cmd.length()) {

gclc/src/main/java/net/bigeon/gclc/command/Command.java

@@ -80,8 +80,8 @@ public abstract class Command implements ICommand {
 
     /** Get the brief part of the command help.
      * <p>
-     * This method may be overriden by implementations to improve the help
-     * content. The default behavior is to print the tip.
+     * This method may be overriden by implementations to improve the help content.
+     * The default behavior is to print the tip.
      *
      * @return a brief description of the command
      * @see Command#help(ConsoleOutput, String...) */
@@ -112,8 +112,8 @@ public abstract class Command implements ICommand {
      *
      * @see net.bigeon.gclc.command.ICommand#help(ConsoleOutput, String...) */
     @Override
-    public final void help(final ConsoleOutput manager,
-                           final String... args) throws IOException {
+    public final void help(final ConsoleOutput manager, final String... args)
+            throws IOException {
         manager.println(getCommandName());
         manager.println(brief());
         manager.println();
@@ -123,8 +123,8 @@ public abstract class Command implements ICommand {
         final String details = usageDetail();
         if (details != null && !details.isEmpty()) {
             manager.print(details);
-            if (!(details.endsWith(EOL_LINUX) ||
-                  details.endsWith(System.lineSeparator()))) {
+            if (!(details.endsWith(EOL_LINUX)
+                    || details.endsWith(System.lineSeparator()))) {
                 manager.println();
             }
         }

gclc/src/main/java/net/bigeon/gclc/command/CommandParameters.java

@@ -43,6 +43,7 @@ import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
+import java.util.concurrent.ConcurrentHashMap;
 
 import net.bigeon.gclc.exception.CommandParsingException;
 
@@ -55,9 +56,9 @@ public final class CommandParameters {
     /** Number of element for a string argument. */
     private static final int           STRINGARG_NUMBER_OF_ELEMENTS = 2;
     /** Boolean arguments. */
-    private final Map<String, Boolean> booleanArguments = new HashMap<>();
+    private final Map<String, Boolean> booleanArguments;
     /** String arguments. */
-    private final Map<String, String> stringArguments = new HashMap<>();
+    private final Map<String, String>  stringArguments;
     /** Arguments restriction on the named ones. */
     private final boolean              strict;
     /** additional (unnamed) parameters. */
@@ -70,9 +71,12 @@ public final class CommandParameters {
      * @param strict if the argument are restricted to the declared ones */
     public CommandParameters(final Set<String> bools, final Set<String> strings,
                              final boolean strict) {
+        booleanArguments = new ConcurrentHashMap<>(bools.size());
         for (final String string : bools) {
             booleanArguments.put(string, Boolean.FALSE);
         }
+        // Cannot use concurrent because of the null values.
+        stringArguments = new HashMap<>(strings.size());
         for (final String string : strings) {
             stringArguments.put(string, null);
         }
@@ -98,9 +102,9 @@ public final class CommandParameters {
      *
      * @param key the key
      * @return if the key was specified */
-    public boolean getBool(final String key) {
-        return booleanArguments.containsKey(key) &&
-               booleanArguments.get(key).booleanValue();
+    public boolean isActive(final String key) {
+        Boolean val = booleanArguments.get(key);
+        return val != null && val.booleanValue();
     }
 
     /** Get the boolean arguments.
@@ -122,14 +126,13 @@ public final class CommandParameters {
      * @param key the key
      * @return if the key is present in string arguments or boolean ones. */
     public boolean hasArgument(final String key) {
-        return stringArguments.containsKey(key) ||
-               booleanArguments.containsKey(key);
+        return stringArguments.containsKey(key) || booleanArguments.containsKey(key);
     }
 
     /** Attempt to parse an argument.
      * <p>
-     * This method return 0 if the parsing was incorrect, or the number of
-     * parsed elements.
+     * This method return 0 if the parsing was incorrect, or the number of parsed
+     * elements.
      *
      * @param arg the argument
      * @param next the next element
@@ -170,8 +173,7 @@ public final class CommandParameters {
             }
             final int p = parseArg(args[i], next);
             if (p == 0) {
-                throw new CommandParsingException(
-                        "Invalid parameter " + args[i]); //$NON-NLS-1$
+                throw new CommandParsingException("Invalid parameter " + args[i]); //$NON-NLS-1$
             }
             i += p;
         }

gclc/src/main/java/net/bigeon/gclc/command/CommandProvider.java

@@ -35,8 +35,8 @@ package net.bigeon.gclc.command;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import java.util.ArrayList;
-import java.util.List;
+import java.util.LinkedHashSet;
+import java.util.Set;
 
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.exception.InvalidCommandName;
@@ -52,24 +52,17 @@ public class CommandProvider implements ICommandProvider {
     private static final String MINUS = "-"; //$NON-NLS-1$
     /** The space character. */
     private static final String SPACE = " "; //$NON-NLS-1$
-    /** The commands map. */
-    protected final List<ICommand> commands;
+
+    /** The commands set.
+     * <p>
+     * The insertion order is conserved through the use of a
+     * {@link LinkedHashSet}. */
+    protected final Set<ICommand> commands;
 
     /** Create a command provider. */
     public CommandProvider() {
         super();
-        commands = new ArrayList<>();
-    }
-
-    /** Test the command name validity.
-     *
-     * @param name the command name
-     * @throws InvalidCommandName if the name is invalid */
-    private static void testCommandName(final String name) throws InvalidCommandName {
-        if (name == null || name.isEmpty() || name.startsWith(MINUS)
-                || name.contains(SPACE)) {
-            throw new InvalidCommandName();
-        }
+        commands = new LinkedHashSet<>();
     }
 
     /* (non-Javadoc)
@@ -118,4 +111,15 @@ public class CommandProvider implements ICommandProvider {
         }
         return null;
     }
+
+    /** Test the command name validity.
+     *
+     * @param name the command name
+     * @throws InvalidCommandName if the name is invalid */
+    private static void testCommandName(final String name) throws InvalidCommandName {
+        if (name == null || name.isEmpty() || name.startsWith(MINUS)
+                || name.contains(SPACE)) {
+            throw new InvalidCommandName();
+        }
+    }
 }

gclc/src/main/java/net/bigeon/gclc/command/ICommand.java

@@ -90,8 +90,8 @@ public interface ICommand {
      * @param in the input
      * @param args the arguments
      * @throws CommandRunException if the command failed */
-    void execute(ConsoleOutput out, ConsoleInput in,
-                 String... args) throws CommandRunException;
+    void execute(ConsoleOutput out, ConsoleInput in, String... args)
+            throws CommandRunException;
 
     /** Get teh command name.
      *

gclc/src/main/java/net/bigeon/gclc/command/ICommandProvider.java

@@ -81,8 +81,8 @@ import net.bigeon.gclc.manager.ConsoleOutput;
  * @author Emmanuel BIGEON */
 public interface ICommandProvider {
 
-    /** Adds a command to this provider, if no command was associated with the
-     * given key.
+    /** Adds a command to this provider, if no command was associated with the given
+     * key.
      *
      * @param value the command to execute
      * @return if the command was added
@@ -91,25 +91,25 @@ public interface ICommandProvider {
 
     /** Execute the command with the given name.
      * <p>
-     * If no command with this name is found, an error command is usually
-     * executed. If there are several commands with the same name, the behavior
-     * is unspecified. Depending on the implementation, it may run an error
-     * command or prompt the user for a choice.
+     * If no command with this name is found, an error command is usually executed.
+     * If there are several commands with the same name, the behavior is
+     * unspecified. Depending on the implementation, it may run an error command or
+     * prompt the user for a choice.
      *
      * @param out the output
      * @param in the input
      * @param command the name of the command the user wishes to execute
      * @param args the arguments for the command
      * @throws CommandRunException if the command failed to run */
-    void executeSub(ConsoleOutput out, ConsoleInput in, String command,
-                    String... args) throws CommandRunException;
+    void executeSub(ConsoleOutput out, ConsoleInput in, String command, String... args)
+            throws CommandRunException;
 
     /** Get the command with the given name.
      * <p>
-     * If no command with this name is found, an error command is usually
-     * returned. If there are several commands with the same name, the behavior
-     * is unspecified. Depending on the implementation, it may return an error
-     * command or the first command with this name found.
+     * If no command with this name is found, an error command is usually returned.
+     * If there are several commands with the same name, the behavior is
+     * unspecified. Depending on the implementation, it may return an error command
+     * or the first command with this name found.
      *
      * @param command the name of the command the user wishes to execute
      * @return the command to execute */

gclc/src/main/java/net/bigeon/gclc/command/ParametrizedCommand.java

@@ -89,9 +89,8 @@ public abstract class ParametrizedCommand extends Command {
 
     /** Create a parametrized command.
      * <p>
-     * Implementation are supposed to call the
-     * {@link #addBooleanParameter(String)} and
-     * {@link #addStringParameter(String, boolean)} method to set the
+     * Implementation are supposed to call the {@link #addBooleanParameter(String)}
+     * and {@link #addStringParameter(String, boolean)} method to set the
      * parameters.
      *
      * @param name the name */
@@ -101,9 +100,8 @@ public abstract class ParametrizedCommand extends Command {
 
     /** Create a parametrized command.
      * <p>
-     * Implementation are supposed to call the
-     * {@link #addBooleanParameter(String)} and
-     * {@link #addStringParameter(String, boolean)} method to set the
+     * Implementation are supposed to call the {@link #addBooleanParameter(String)}
+     * and {@link #addStringParameter(String, boolean)} method to set the
      * parameters.
      *
      * @param name the name
@@ -116,9 +114,10 @@ public abstract class ParametrizedCommand extends Command {
     /** Add a boolean parameter to defined parmaters.
      *
      * @param flag the boolean flag
-     * @throws InvalidParameterException if the parameter is already defined as
-     *             a string parameter */
-    protected final void addBooleanParameter(final String flag) throws InvalidParameterException {
+     * @throws InvalidParameterException if the parameter is already defined as a
+     *         string parameter */
+    protected final void addBooleanParameter(final String flag)
+            throws InvalidParameterException {
         data.addBooleanParameter(flag);
     }
 
@@ -126,10 +125,10 @@ public abstract class ParametrizedCommand extends Command {
      *
      * @param flag the parameter flag
      * @param needed if the parameter's absence should cause an exception
-     * @throws InvalidParameterException if the parameter is already defined as
-     *             a boolean parameter */
-    protected final void addStringParameter(final String flag,
-                                            final boolean needed) throws InvalidParameterException {
+     * @throws InvalidParameterException if the parameter is already defined as a
+     *         boolean parameter */
+    protected final void addStringParameter(final String flag, final boolean needed)
+            throws InvalidParameterException {
         data.addStringParameter(flag, needed);
     }
 
@@ -145,8 +144,7 @@ public abstract class ParametrizedCommand extends Command {
     /* (non-Javadoc)
      * @see net.bigeon.gclc.command.Command#execute(java.lang.String[]) */
     @Override
-    public final void execute(final ConsoleOutput output,
-                              final ConsoleInput input,
+    public final void execute(final ConsoleOutput output, final ConsoleInput input,
             final String... args) throws CommandRunException {
         try {
             doExecute(output, input, data.getParameters(input, args));

gclc/src/main/java/net/bigeon/gclc/command/ParametrizedCommandData.java

@@ -75,18 +75,17 @@ import java.io.IOException;
 import java.text.MessageFormat;
 import java.util.ArrayList;
 import java.util.Collections;
-import java.util.HashMap;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
+import java.util.Set;
+import java.util.concurrent.ConcurrentHashMap;
 
 import net.bigeon.gclc.exception.CommandParsingException;
 import net.bigeon.gclc.exception.InvalidParameterException;
 import net.bigeon.gclc.manager.ConsoleInput;
-import net.bigeon.gclc.manager.EmptyInput;
-
-import java.util.Set;
+import net.bigeon.gclc.utils.EmptyInput;
 
 /** An object to handle standardized command parameters.
  *
@@ -95,10 +94,11 @@ public final class ParametrizedCommandData {
 
     /** The boolean parameters mandatory status. */
     private final Set<String> boolParams = new HashSet<>();
+
     /** The string parameters mandatory status. */
-    private final Map<String, Boolean> stringParams = new HashMap<>();
+    private final Map<String, Boolean> stringParams = new ConcurrentHashMap<>();
     /** The parameters mandatory status. */
-    private final Map<String, Boolean> params       = new HashMap<>();
+    private final Map<String, Boolean> params       = new ConcurrentHashMap<>();
     /** The restriction of provided parameters on execution to declared paramters in
      * the status maps. */
     private final boolean              strict;
@@ -120,8 +120,7 @@ public final class ParametrizedCommandData {
      * @param flag the boolean flag
      * @throws InvalidParameterException if the parameter is already defined as a
      *         string parameter */
-    public final void addBooleanParameter(final String flag)
-            throws InvalidParameterException {
+    public void addBooleanParameter(final String flag) throws InvalidParameterException {
         if (params.containsKey(flag) && stringParams.containsKey(flag)) {
             throw new InvalidParameterException("Parameter is already defined as string"); //$NON-NLS-1$
         }
@@ -135,7 +134,7 @@ public final class ParametrizedCommandData {
      * @param needed if the parameter's absence should cause an exception
      * @throws InvalidParameterException if the parameter is already defined as a
      *         boolean parameter */
-    public final void addStringParameter(final String flag, final boolean needed)
+    public void addStringParameter(final String flag, final boolean needed)
             throws InvalidParameterException {
         if (params.containsKey(flag)) {
             checkParam(flag, needed);
@@ -152,9 +151,9 @@ public final class ParametrizedCommandData {
      * @throws InvalidParameterException if the new definition is invalid */
     private void checkParam(final String param, final boolean needed)
             throws InvalidParameterException {
-        if (stringParams.containsKey(param)) {
-            final Boolean need = Boolean
-                    .valueOf(needed || stringParams.get(param).booleanValue());
+        Boolean val = stringParams.get(param);
+        if (val != null) {
+            final Boolean need = Boolean.valueOf(needed || val.booleanValue());
             stringParams.put(param, need);
             params.put(param, need);
             return;
@@ -162,14 +161,28 @@ public final class ParametrizedCommandData {
         throw new InvalidParameterException("Parameter is already defined as boolean"); //$NON-NLS-1$
     }
 
+    /** Retrieve the boolean parameters (aka flags).
+     *
+     * @return the set of boolean parameters */
+    public Set<String> getBooleanParameters() {
+        return Collections.unmodifiableSet(boolParams);
+    }
+
+    /** Retrieve the parameter names.
+     *
+     * @return the stringParams */
+    public Set<String> getParameters() {
+        return params.keySet();
+    }
+
     /** Get the parameters from an input.
      *
      * @param input the input
      * @param args the command arguments
      * @return the command object
      * @throws IOException if the command could not be filled. */
-    public final CommandParameters getParameters(final ConsoleInput input,
-            final String... args) throws IOException {
+    public CommandParameters getParameters(final ConsoleInput input, final String... args)
+            throws IOException {
         final CommandParameters parameters = new CommandParameters(boolParams,
                 stringParams.keySet(), strict);
         try {
@@ -192,6 +205,29 @@ public final class ParametrizedCommandData {
         return parameters;
     }
 
+    /** Get the string parameters names.
+     *
+     * @return the stringParams */
+    public Set<String> getStringParameters() {
+        return stringParams.keySet();
+    }
+
+    /** Test if a parameter is needed.
+     *
+     * @param param the parameter name
+     * @return if the parameter is needed */
+    public boolean isNeeded(final String param) {
+        Boolean val = params.get(param);
+        return val != null && val.booleanValue();
+    }
+
+    /** If the command refuse unrecognized parameters.
+     *
+     * @return the strict */
+    public boolean isStrict() {
+        return strict;
+    }
+
     /** Fill the undefined parameters.
      * <p>
      * This method prompts the user to fill the needed parameters.
@@ -200,7 +236,7 @@ public final class ParametrizedCommandData {
      * @param parameters the parameter list to complete
      * @param toProvide the parameters to ask for
      * @throws IOException if the manager was closed */
-    private final static void fillParameters(final ConsoleInput input,
+    private static void fillParameters(final ConsoleInput input,
             final List<String> toProvide, final CommandParameters parameters)
             throws IOException {
         for (final String string : toProvide) {
@@ -214,40 +250,4 @@ public final class ParametrizedCommandData {
             parameters.set(string, value);
         }
     }
-
-    /** Retrieve the boolean parameters (aka flags).
-     *
-     * @return the set of boolean parameters */
-    public final Set<String> getBooleanParameters() {
-        return Collections.unmodifiableSet(boolParams);
-    }
-
-    /** Retrieve the parameter names.
-     *
-     * @return the stringParams */
-    public final Set<String> getParameters() {
-        return params.keySet();
-    }
-
-    /** Get the string parameters names.
-     *
-     * @return the stringParams */
-    public final Set<String> getStringParameters() {
-        return stringParams.keySet();
-    }
-
-    /** Test if a parameter is needed.
-     *
-     * @param param the parameter name
-     * @return if the parameter is needed */
-    public final boolean isNeeded(final String param) {
-        return params.containsKey(param) && params.get(param).booleanValue();
-    }
-
-    /** If the command refuse unrecognized parameters.
-     *
-     * @return the strict */
-    public final boolean isStrict() {
-        return strict;
-    }
 }

gclc/src/main/java/net/bigeon/gclc/command/SubedCommand.java

@@ -87,8 +87,7 @@ public final class SubedCommand extends CommandProvider implements ICommand {
 
     /** The tab character. */
     private static final String TAB = "\t";  //$NON-NLS-1$
-    /** The command to execute when this command is called with no sub
-     * arguments.
+    /** The command to execute when this command is called with no sub arguments.
      * <p>
      * This may be null, in which case the command should have arguments. */
     private final ICommand      noArgCommand;
@@ -176,15 +175,15 @@ public final class SubedCommand extends CommandProvider implements ICommand {
     /* (non-Javadoc)
      * @see net.bigeon.gclc.command.Command#help() */
     @Override
-    public void help(final ConsoleOutput manager,
-                     final String... args) throws IOException {
+    public void help(final ConsoleOutput manager, final String... args)
+            throws IOException {
         if (args.length != 0 && !args[0].startsWith("-")) { //$NON-NLS-1$
             // Specific
             final ICommand c = get(args[0]);
             if (c != null) {
                 c.help(manager, Arrays.copyOfRange(args, 1, args.length));
             } else {
-              manager.println("No command "+Arrays.toString(args));
+                manager.println("No command " + Arrays.toString(args));
             }
         } else {
             // Generic

gclc/src/main/java/net/bigeon/gclc/command/base/EchoCommand.java

@@ -0,0 +1,100 @@
+/*
+ * ralph.runner, Runs RALPH (belongs to IREQ) in sequence for several days
+ * Copyright (C) 2016-2018 E. Bigeon
+ * mailto:emmanuel@bigeon.fr
+ *
+ * This software is governed by the CeCILL license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ *
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ *
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ *
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ */
+/**
+ * 
+ */
+package net.bigeon.gclc.command.base;
+
+import java.io.IOException;
+import java.text.MessageFormat;
+import java.util.Arrays;
+
+import net.bigeon.gclc.command.Command;
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.manager.ConsoleInput;
+import net.bigeon.gclc.manager.ConsoleOutput;
+
+/** A command printing test to the console.
+ * <p>
+ * This command mimics the effect of the echo command in linux system. The
+ * difference is this command accepts java like formatting.
+ * 
+ * <pre>
+ * echo.execute(out, in, "My Formatted {0}, with {1}", "message", "short arguments");
+ * </pre>
+ * 
+ * @author Emmanuel Bigeon */
+public class EchoCommand extends Command {
+
+    /** Create the command.
+     * 
+     * @param name the command name */
+    public EchoCommand(final String name) {
+        super(name);
+    }
+
+    /* (non-Javadoc)
+     * @see
+     * fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.ConsoleOutput,
+     * fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
+    @Override
+    public void execute(final ConsoleOutput out, final ConsoleInput in,
+            final String... args) throws CommandRunException {
+        try {
+            if (args.length == 1) {
+                out.print(args[0]);
+            }
+            if (args.length > 1) {
+                out.print(MessageFormat.format(args[0],
+                        (Object[]) Arrays.copyOfRange(args, 1, args.length)));
+            }
+            out.println();
+        } catch (final IOException e) {
+            throw new CommandRunException("Printing failed", e);
+        }
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.command.ICommand#tip() */
+    @Override
+    public String tip() {
+        return "Print the message";
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.command.Command#usageDetail() */
+    @Override
+    protected String usageDetail() {
+        return "A formatted message can be used";
+    }
+
+}

gclc/src/main/java/net/bigeon/gclc/command/base/ExitCommand.java

@@ -109,8 +109,8 @@ public class ExitCommand implements ICommand {
     }
 
     @Override
-    public final void execute(final ConsoleOutput output,
-                              final ConsoleInput input, final String... args) {
+    public final void execute(final ConsoleOutput output, final ConsoleInput input,
+            final String... args) {
         beforeExit();
         app.exit();
     }
@@ -123,11 +123,11 @@ public class ExitCommand implements ICommand {
     }
 
     @Override
-    public final void help(final ConsoleOutput manager,
-                     final String... args) throws IOException {
-        manager.println(
-                CLIPrompterMessages.getString(EXIT_MAN, (Object[]) args));
+    public final void help(final ConsoleOutput manager, final String... args)
+            throws IOException {
+        manager.println(CLIPrompterMessages.getString(EXIT_MAN, (Object[]) args));
     }
+
     @Override
     public final String tip() {
         return CLIPrompterMessages.getString(EXIT);

gclc/src/main/java/net/bigeon/gclc/command/base/HelpExecutor.java

@@ -36,7 +36,7 @@
  * gclc:net.bigeon.gclc.command.HelpExecutor.java
  * Created on: Sep 6, 2014
  */
-package net.bigeon.gclc.command;
+package net.bigeon.gclc.command.base;
 
 /*-
  * #%L
@@ -73,6 +73,9 @@ package net.bigeon.gclc.command;
  */
 import java.io.IOException;
 
+import net.bigeon.gclc.command.Command;
+import net.bigeon.gclc.command.ICommand;
+import net.bigeon.gclc.command.SubedCommand;
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.exception.CommandRunExceptionType;
 import net.bigeon.gclc.manager.ConsoleInput;
@@ -93,8 +96,7 @@ public final class HelpExecutor extends Command {
      *
      * @param cmdName the command name
      * @param cmd the command to execute the help of */
-    public HelpExecutor(final String cmdName,
-            final ICommand cmd) {
+    public HelpExecutor(final String cmdName, final ICommand cmd) {
         super(cmdName);
         this.cmd = cmd;
     }
@@ -131,8 +133,7 @@ public final class HelpExecutor extends Command {
     }
 
     /* (non-Javadoc)
-     * @see net.bigeon.gclc.command.Command#usageDetail()
-     */
+     * @see net.bigeon.gclc.command.Command#usageDetail() */
     @Override
     protected String usageDetail() {
         return null;

gclc/src/main/java/net/bigeon/gclc/command/base/MockCommand.java

@@ -113,8 +113,7 @@ public final class MockCommand implements ICommand {
      * @see net.bigeon.gclc.command.ICommand#help(net.bigeon.gclc.manager.
      * ConsoleManager, java.lang.String[]) */
     @Override
-    public void help(final ConsoleOutput manager,
-                     final String... args) {
+    public void help(final ConsoleOutput manager, final String... args) {
         //
     }
 

gclc/src/main/java/net/bigeon/gclc/command/base/ScriptExecution.java

@@ -72,10 +72,11 @@ package net.bigeon.gclc.command.base;
  * #L%
  */
 import java.io.BufferedReader;
-import java.io.FileInputStream;
 import java.io.IOException;
 import java.io.InputStreamReader;
 import java.nio.charset.Charset;
+import java.nio.file.Files;
+import java.nio.file.Paths;
 import java.text.MessageFormat;
 import java.util.Arrays;
 import java.util.List;
@@ -101,10 +102,13 @@ public final class ScriptExecution extends Command {
     private static final String TAB   = "\t"; //$NON-NLS-1$
     /** the space character. */
     private static final String SPACE = " ";  //$NON-NLS-1$
+
     /** The application. */
     private final ConsoleApplication application;
+
     /** The commenting prefix. */
     private final String commentPrefix;
+
     /** The charset for files. */
     private final Charset charset;
 
@@ -122,20 +126,6 @@ public final class ScriptExecution extends Command {
         this.charset = charset;
     }
 
-    /** Check the arguments.
-     *
-     * @param args the arguments
-     * @throws CommandRunException if the arguments were not the ones
-     *             expected */
-    private static void checkArgs(final String[] args) throws CommandRunException {
-        if (args.length == 0) {
-            throw new CommandRunException(CommandRunExceptionType.USAGE,
-                    "Expecting a file"); //$NON-NLS-1$
-        }
-    }
-
-
-
     /* (non-Javadoc)
      * @see net.bigeon.gclc.command.ICommand#execute(java.lang.String[]) */
     @Override
@@ -147,8 +137,9 @@ public final class ScriptExecution extends Command {
         String cmd;
         int lineNo = -1;
         try (InputStreamReader fReader = new InputStreamReader(
-                new FileInputStream(scriptFile), charset);
+                Files.newInputStream(Paths.get(scriptFile)), charset);
                 BufferedReader reader = new BufferedReader(fReader)) {
+            String[] emptyArray = new String[0];
             while ((cmd = reader.readLine()) != null) {
                 lineNo++;
                 final String cmdLine = readCommandLine(cmd, params);
@@ -157,12 +148,12 @@ public final class ScriptExecution extends Command {
                 }
                 final List<String> ps = GCLCConstants.splitCommand(cmdLine);
                 final String command = ps.remove(0);
-                application.executeSub(out, in, command,
-                        ps.toArray(new String[0]));
+                application.executeSub(out, in, command, ps.toArray(emptyArray));
             }
         } catch (final CommandParsingException e) {
-            throw new CommandRunException(MessageFormat.format(
-                    "Invalid command in script ({0})", e.getLocalizedMessage()), //$NON-NLS-1$
+            throw new CommandRunException(
+                    MessageFormat.format("Invalid command in script ({0})", //$NON-NLS-1$
+                            e.getLocalizedMessage()),
                     e);
         } catch (final IOException e) {
             throw new CommandRunException("Unable to read script", //$NON-NLS-1$
@@ -172,29 +163,14 @@ public final class ScriptExecution extends Command {
         }
     }
 
-    /** This method will create the correct exception. The exception source must
-     * be this command.
-     *
-     * @param e the exception
-     * @param lineNo the line nu;ber
-     * @return the exception to actually throw */
-    private static CommandRunException manageRunException(final CommandRunException e,
-                                                   final int lineNo) {
-        return new CommandRunException(CommandRunExceptionType.EXECUTION,
-                MessageFormat.format(
-                        "The script could not complete due to command failure at line {0} ({1})", //$NON-NLS-1$
-                        Integer.valueOf(lineNo), e.getLocalizedMessage()),
-                e);
-    }
-
     /** Read a line of the script.
      *
      * @param cmd the line
      * @param params the formatting parameters
      * @return the command if it is indeed one, null otherwise
      * @throws CommandRunException if the line stqrted with a space character */
-    private String readCommandLine(final String cmd,
-                                   final Object[] params) throws CommandRunException {
+    private String readCommandLine(final String cmd, final Object[] params)
+            throws CommandRunException {
         if (cmd.startsWith(SPACE) || cmd.startsWith(TAB)) {
             throw new CommandRunException(
                     "Invalid line in script (line starts with space character)");
@@ -225,8 +201,7 @@ public final class ScriptExecution extends Command {
         builder.append(
                 "  The script file must contain one line commands. The lines must never"); //$NON-NLS-1$
         builder.append(System.lineSeparator());
-        builder.append(
-                "start with whitespace characters. The lines starting with"); //$NON-NLS-1$
+        builder.append("start with whitespace characters. The lines starting with"); //$NON-NLS-1$
         builder.append(System.lineSeparator());
         builder.append('"');
         builder.append(commentPrefix);
@@ -243,4 +218,30 @@ public final class ScriptExecution extends Command {
         return super.usagePattern() + " <scriptfile>"; //$NON-NLS-1$
     }
 
+    /** Check the arguments.
+     *
+     * @param args the arguments
+     * @throws CommandRunException if the arguments were not the ones expected */
+    private static void checkArgs(final String[] args) throws CommandRunException {
+        if (args.length == 0) {
+            throw new CommandRunException(CommandRunExceptionType.USAGE,
+                    "Expecting a file"); //$NON-NLS-1$
+        }
+    }
+
+    /** This method will create the correct exception. The exception source must be
+     * this command.
+     *
+     * @param e the exception
+     * @param lineNo the line nu;ber
+     * @return the exception to actually throw */
+    private static CommandRunException manageRunException(final CommandRunException e,
+            final int lineNo) {
+        return new CommandRunException(CommandRunExceptionType.EXECUTION,
+                MessageFormat.format(
+                        "The script could not complete due to command failure at line {0} ({1})", //$NON-NLS-1$
+                        Integer.valueOf(lineNo), e.getLocalizedMessage()),
+                e);
+    }
+
 }

gclc/src/main/java/net/bigeon/gclc/command/base/package-info.java

@@ -1,7 +1,4 @@
-/**
- * @author Emmanuel Bigeon
- *
- */
+/** @author Emmanuel Bigeon */
 package net.bigeon.gclc.command.base;
 
 /*-

gclc/src/main/java/net/bigeon/gclc/command/package-info.java

@@ -4,9 +4,9 @@
  * There are some implementations, such as the
  * {@link net.bigeon.gclc.command.ParametrizedCommand} for commands with a
  * predefined set of flags and option taking a string as value, the
- * {@link net.bigeon.gclc.command.SubedCommand} for a command that is declined in
- * a set of sub commands, the {@link net.bigeon.gclc.command.HelpExecutor} for
- * help display of other commands and the
+ * {@link net.bigeon.gclc.command.SubedCommand} for a command that is declined
+ * in a set of sub commands, the {@link net.bigeon.gclc.command.base.HelpExecutor}
+ * for help display of other commands and the
  *
  * @author Emmanuel BIGEON */
 package net.bigeon.gclc.command;

gclc/src/main/java/net/bigeon/gclc/exception/CommandRunException.java

@@ -87,8 +87,7 @@ public final class CommandRunException extends Exception {
      *
      * @param type the type of exception
      * @param message the message */
-    public CommandRunException(final CommandRunExceptionType type,
-                               final String message) {
+    public CommandRunException(final CommandRunExceptionType type, final String message) {
         super(message);
         this.type = type;
     }
@@ -98,8 +97,8 @@ public final class CommandRunException extends Exception {
      * @param type the type of exception
      * @param message a message
      * @param cause the cause */
-    public CommandRunException(final CommandRunExceptionType type,
-                               final String message, final Throwable cause) {
+    public CommandRunException(final CommandRunExceptionType type, final String message,
+                               final Throwable cause) {
         super(message, cause);
         this.type = type;
     }

gclc/src/main/java/net/bigeon/gclc/i18n/Messages.java

@@ -89,8 +89,7 @@ public final class Messages {
             .getBundle(BUNDLE_NAME);
 
     /** The class logger. */
-    private static final Logger LOGGER = Logger
-            .getLogger(Messages.class.getName());
+    private static final Logger LOGGER = Logger.getLogger(Messages.class.getName());
 
     /** Utility class. */
     private Messages() {

gclc/src/main/java/net/bigeon/gclc/manager/ConsoleOutput.java

@@ -86,14 +86,12 @@ public interface ConsoleOutput extends AutoCloseable {
     /** Print a string.
      *
      * @param text the message to print (without line break at the end).
-     * @throws IOException if the manager is closed or could not read the
-     *             prompt */
+     * @throws IOException if the manager is closed or could not read the prompt */
     void print(String text) throws IOException;
 
     /** Prints an end of line.
      *
-     * @throws IOException if the manager is closed or could not read the
-     *             prompt */
+     * @throws IOException if the manager is closed or could not read the prompt */
     void println() throws IOException;
 
     /** Print a string followed by an end of line.
@@ -102,8 +100,7 @@ public interface ConsoleOutput extends AutoCloseable {
      * {@link #println()}.
      *
      * @param message the message to print
-     * @throws IOException if the manager is closed or could not read the
-     *             prompt */
+     * @throws IOException if the manager is closed or could not read the prompt */
     void println(String message) throws IOException;
 
 }

gclc/src/main/java/net/bigeon/gclc/prompt/CLIPrompter.java

@@ -70,6 +70,7 @@ package net.bigeon.gclc.prompt;
  * #L%
  */
 import java.io.IOException;
+import java.text.MessageFormat;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
@@ -118,8 +119,7 @@ public final class CLIPrompter {
      * @param chs the list of integers
      * @param index the max index of choice
      * @return if the parsing was done correctly */
-    private static boolean addUserChoice(final String val,
-                                         final List<Integer> chs,
+    private static boolean addUserChoice(final String val, final List<Integer> chs,
             final int index) {
         if (val.isEmpty()) {
             return true;
@@ -138,18 +138,18 @@ public final class CLIPrompter {
      * @param output the manager
      * @param choices the choices
      * @param cancel the cancel option if it exists
-     * @return the number of choices plus one (or the number of choices if there
-     *         is a cancel)
+     * @return the index of last choice
      * @throws IOException if the manager was closed */
     private static <U> Integer listChoices(final ConsoleOutput output,
-                                           final List<U> choices,
-                                           final String cancel) throws IOException {
+            final List<U> choices, final String cancel) throws IOException {
         int index = 0;
         for (final U u : choices) {
-            output.println(index++ + ") " + u); //$NON-NLS-1$
+            output.println(index + ") " + u); //$NON-NLS-1$
+            index++;
         }
         if (cancel != null) {
-            output.println(index++ + ") " + cancel); //$NON-NLS-1$
+            output.println(index + ") " + cancel); //$NON-NLS-1$
+            index++;
         }
         return Integer.valueOf(index - 1);
     }
@@ -162,36 +162,28 @@ public final class CLIPrompter {
      * @return the choice
      * @throws IOException if the manager was closed */
     public static boolean promptBoolean(final ConsoleOutput manager,
-                                        final ConsoleInput input,
-                                        final String message) throws IOException {
-        String result = input
-                .prompt(message + CLIPrompterMessages.getString(BOOL_CHOICES));
+            final ConsoleInput input, final String message) throws IOException {
+        String booleanChoices = message + CLIPrompterMessages.getString(BOOL_CHOICES);
+        String result = input.prompt(booleanChoices);
         boolean first = true;
-        final String choices = CLIPrompterMessages
-                .getString(PROMPTBOOL_CHOICES_YES1) + ", " + //$NON-NLS-1$
-                               CLIPrompterMessages
-                                       .getString(PROMPTBOOL_CHOICES_NO1);
-        while (!(result.equalsIgnoreCase(
-                CLIPrompterMessages.getString(PROMPTBOOL_CHOICES_YES1)) ||
-                 CLIPrompterMessages.getString(PROMPTBOOL_CHOICES_NO1)
-                         .equalsIgnoreCase(result) ||
-                 CLIPrompterMessages.getString("promptbool.choices.no2") //$NON-NLS-1$
-                         .equalsIgnoreCase(result) ||
-                 CLIPrompterMessages.getString("promptbool.choices.yes2") //$NON-NLS-1$
-                         .equalsIgnoreCase(result))) {
+        String booleanYesChoice = CLIPrompterMessages.getString(PROMPTBOOL_CHOICES_YES1);
+        String booleanNoChoice = CLIPrompterMessages.getString(PROMPTBOOL_CHOICES_NO1);
+        final String choices = MessageFormat.format("{0}, {1}", booleanYesChoice,
+                booleanNoChoice);
+        String yes2 = CLIPrompterMessages.getString("promptbool.choices.yes2");
+        String no2 = CLIPrompterMessages.getString("promptbool.choices.no2");
+        while (!(result.equalsIgnoreCase(booleanYesChoice)
+                || booleanNoChoice.equalsIgnoreCase(result)
+                || no2.equalsIgnoreCase(result) || yes2.equalsIgnoreCase(result))) {
             if (!first) {
 
                 manager.println(CLIPrompterMessages
                         .getString("promptbool.choices.invalid", choices)); //$NON-NLS-1$
-                result = input.prompt(
-                        message + CLIPrompterMessages.getString(BOOL_CHOICES));
+                result = input.prompt(booleanChoices);
             }
             first = false;
         }
-        return result.equalsIgnoreCase(
-                CLIPrompterMessages.getString(PROMPTBOOL_CHOICES_YES1)) ||
-               result.equalsIgnoreCase(CLIPrompterMessages
-                       .getString("promptbool.choices.yes2")); //$NON-NLS-1$
+        return result.equalsIgnoreCase(booleanYesChoice) || result.equalsIgnoreCase(yes2); // $NON-NLS-1$
     }
 
     /** Prompt for a choice.
@@ -206,13 +198,9 @@ public final class CLIPrompter {
      * @return the choice
      * @throws IOException if the manager was closed */
     public static <U> U promptChoice(final ConsoleOutput manager,
-                                     final ConsoleInput input,
-                                     final List<String> keys,
-                                     final List<U> choices,
-                                     final String message,
-                                     final String cancel) throws IOException {
-        final Integer index = promptChoice(manager, input, keys, message,
-                cancel);
+            final ConsoleInput input, final List<String> keys, final List<U> choices,
+            final String message, final String cancel) throws IOException {
+        final Integer index = promptChoice(manager, input, keys, message, cancel);
         if (index == null) {
             return null;
         }
@@ -232,13 +220,9 @@ public final class CLIPrompter {
      * @return the chosen object
      * @throws IOException if the manager was closed */
     public static <U, T> T promptChoice(final ConsoleOutput manager,
-                                        final ConsoleInput input,
-                                        final List<U> choices,
-                                        final Map<U, T> choicesMap,
-                                        final String message,
-                                        final String cancel) throws IOException {
-        final Integer res = promptChoice(manager, input, choices, message,
-                cancel);
+            final ConsoleInput input, final List<U> choices, final Map<U, T> choicesMap,
+            final String message, final String cancel) throws IOException {
+        final Integer res = promptChoice(manager, input, choices, message, cancel);
         if (res == null) {
             return null;
         }
@@ -256,9 +240,7 @@ public final class CLIPrompter {
      * @return the index of the choice
      * @throws IOException if the manager was closed */
     public static <U> Integer promptChoice(final ConsoleOutput manager,
-                                           final ConsoleInput input,
-                                           final List<U> choices,
-                                           final String message,
+            final ConsoleInput input, final List<U> choices, final String message,
             final String cancel) throws IOException {
         manager.println(message);
         final Integer index = listChoices(manager, choices, cancel);
@@ -271,13 +253,12 @@ public final class CLIPrompter {
                 if (r >= 0 && r <= index.intValue()) {
                     break;
                 }
-                manager.println(CLIPrompterMessages
-                        .getString(PROMPTCHOICE_OUTOFBOUNDS, ZERO, index));
+                manager.println(CLIPrompterMessages.getString(PROMPTCHOICE_OUTOFBOUNDS,
+                        ZERO, index));
             } catch (final NumberFormatException e) {
-                LOGGER.log(Level.FINER,
-                        "Unrecognized number. Prompting user again.", e); //$NON-NLS-1$
-                manager.println(CLIPrompterMessages
-                        .getString(PROMPTCHOICE_FORMATERR, ZERO, index));
+                LOGGER.log(Level.FINER, "Unrecognized number. Prompting user again.", e); //$NON-NLS-1$
+                manager.println(CLIPrompterMessages.getString(PROMPTCHOICE_FORMATERR,
+                        ZERO, index));
             }
             listChoices(manager, choices, cancel);
         }
@@ -299,13 +280,10 @@ public final class CLIPrompter {
      * @return the chosen object
      * @throws IOException if the manager was closed */
     public static <U, T> T promptChoice(final ConsoleOutput manager,
-                                        final ConsoleInput input,
-                                        final Map<U, T> choicesMap,
-                                        final String message,
+            final ConsoleInput input, final Map<U, T> choicesMap, final String message,
             final String cancel) throws IOException {
-        return promptChoice(manager, input,
-                new ArrayList<>(choicesMap.keySet()), choicesMap, message,
-                cancel);
+        return promptChoice(manager, input, new ArrayList<>(choicesMap.keySet()),
+                choicesMap, message, cancel);
     }
 
     /** Prompt for an integer.
@@ -314,8 +292,8 @@ public final class CLIPrompter {
      * @param message the prompt message
      * @return the integer
      * @throws IOException if the manager was closed */
-    public static int promptInteger(final ConsoleInput manager,
-                                    final String message) throws IOException {
+    public static int promptInteger(final ConsoleInput manager, final String message)
+            throws IOException {
         boolean still = true;
         int r = 0;
         while (still) {
@@ -344,8 +322,7 @@ public final class CLIPrompter {
      * @return the list of user inputs
      * @throws IOException if the manager was closed */
     public static List<String> promptList(final ConsoleOutput manager,
-                                          final ConsoleInput input,
-                                          final String message) throws IOException {
+            final ConsoleInput input, final String message) throws IOException {
         return promptList(manager, input, message,
                 CLIPrompterMessages.getString("promptlist.exit.defaultkey")); //$NON-NLS-1$
     }
@@ -359,12 +336,10 @@ public final class CLIPrompter {
      * @return the list of user inputs
      * @throws IOException if the manager was closed */
     public static List<String> promptList(final ConsoleOutput manager,
-                                          final ConsoleInput input,
-                                          final String message,
-                                          final String ender) throws IOException {
+            final ConsoleInput input, final String message, final String ender)
+            throws IOException {
         final List<String> strings = new ArrayList<>();
-        manager.println(
-                message + CLIPrompterMessages.getString(LIST_DISP_KEY, ender));
+        manager.println(message + CLIPrompterMessages.getString(LIST_DISP_KEY, ender));
         String res = null;
         while (!ender.equals(res)) {
             res = input.prompt(CLIPrompterMessages.getString(PROMPT));
@@ -383,10 +358,9 @@ public final class CLIPrompter {
      * @return the text
      * @throws IOException if the manager was closed */
     public static String promptLongText(final ConsoleOutput manager,
-                                        final ConsoleInput input,
-                                        final String message) throws IOException {
-        return promptLongText(manager, input, message, CLIPrompterMessages
-                .getString("promptlongtext.exit.defaultkey")); //$NON-NLS-1$
+            final ConsoleInput input, final String message) throws IOException {
+        return promptLongText(manager, input, message,
+                CLIPrompterMessages.getString("promptlongtext.exit.defaultkey")); //$NON-NLS-1$
     }
 
     /** Prompt for a text with several lines.
@@ -398,11 +372,10 @@ public final class CLIPrompter {
      * @return the text
      * @throws IOException if the manager was closed */
     public static String promptLongText(final ConsoleOutput manager,
-                                        final ConsoleInput input,
-                                        final String message,
-                                        final String ender) throws IOException {
-        manager.println(message + CLIPrompterMessages
-                .getString("promptlongtext.exit.dispkey", ender)); //$NON-NLS-1$
+            final ConsoleInput input, final String message, final String ender)
+            throws IOException {
+        manager.println(message
+                + CLIPrompterMessages.getString("promptlongtext.exit.dispkey", ender)); //$NON-NLS-1$
         final StringBuilder res = new StringBuilder();
         String line;
         do {
@@ -426,13 +399,10 @@ public final class CLIPrompter {
      * @return the choice
      * @throws IOException if the manager was closed */
     public static <U> List<U> promptMultiChoice(final ConsoleOutput manager,
-                                                final ConsoleInput input,
-                                                final List<String> keys,
-                                                final List<U> choices,
+            final ConsoleInput input, final List<String> keys, final List<U> choices,
             final String message) throws IOException {
-        final List<Integer> indices = promptMultiChoice(manager, input, keys,
-                message);
-        final List<U> userChoices = new ArrayList<>();
+        final List<Integer> indices = promptMultiChoice(manager, input, keys, message);
+        final List<U> userChoices = new ArrayList<>(indices.size());
         for (final Integer integer : indices) {
             userChoices.add(choices.get(integer.intValue()));
         }
@@ -451,13 +421,10 @@ public final class CLIPrompter {
      * @return the chosen objects (or an empty list)
      * @throws IOException if the manager was closed */
     public static <U, T> List<T> promptMultiChoice(final ConsoleOutput manager,
-                                                   final ConsoleInput input,
-                                                   final List<U> choices,
-                                                   final Map<U, T> choicesMap,
+            final ConsoleInput input, final List<U> choices, final Map<U, T> choicesMap,
             final String message) throws IOException {
-        final List<Integer> chs = promptMultiChoice(manager, input, choices,
-                message);
-        final List<T> userChoices = new ArrayList<>();
+        final List<Integer> chs = promptMultiChoice(manager, input, choices, message);
+        final List<T> userChoices = new ArrayList<>(chs.size());
         for (final Integer integer : chs) {
             userChoices.add(choicesMap.get(choices.get(integer.intValue())));
         }
@@ -474,9 +441,8 @@ public final class CLIPrompter {
      * @return the indices of the choices
      * @throws IOException if the manager was closed */
     public static <U> List<Integer> promptMultiChoice(final ConsoleOutput manager,
-                                                      final ConsoleInput input,
-                                                      final List<U> choices,
-                                                      final String message) throws IOException {
+            final ConsoleInput input, final List<U> choices, final String message)
+            throws IOException {
         manager.println(message);
         final Integer index = listChoices(manager, choices, null);
         String result;
@@ -492,11 +458,11 @@ public final class CLIPrompter {
                 try {
                     added = addUserChoice(val, chs, index.intValue());
                 } catch (final NumberFormatException e) {
-                    LOGGER.log(Level.FINER,
-                            "Unrecognized number. Prompting user again.", e); //$NON-NLS-1$
+                    LOGGER.log(Level.FINER, "Unrecognized number. Prompting user again.", //$NON-NLS-1$
+                            e);
                     keepOn = true;
-                    manager.println(CLIPrompterMessages
-                            .getString(PROMPTCHOICE_FORMATERR, ZERO, index));
+                    manager.println(CLIPrompterMessages.getString(PROMPTCHOICE_FORMATERR,
+                            ZERO, index));
                     listChoices(manager, choices, null);
                     break;
                 }
@@ -522,11 +488,10 @@ public final class CLIPrompter {
      * @return the chosen objects
      * @throws IOException if the manager was closed */
     public static <U, T> List<T> promptMultiChoice(final ConsoleOutput manager,
-                                                   final ConsoleInput input,
-                                                   final Map<U, T> choicesMap,
-                                                   final String message) throws IOException {
-        return promptMultiChoice(manager, input,
-                new ArrayList<>(choicesMap.keySet()), choicesMap, message);
+            final ConsoleInput input, final Map<U, T> choicesMap, final String message)
+            throws IOException {
+        return promptMultiChoice(manager, input, new ArrayList<>(choicesMap.keySet()),
+                choicesMap, message);
     }
 
     /** Prompt the user for a non empty text.
@@ -536,8 +501,7 @@ public final class CLIPrompter {
      * @param reprompt the prompting message after empty input
      * @return the non empty input
      * @throws IOException if the manager was closed */
-    public static String promptNonEmpty(final ConsoleInput manager,
-                                        final String prompt,
+    public static String promptNonEmpty(final ConsoleInput manager, final String prompt,
             final String reprompt) throws IOException {
         String res = manager.prompt(prompt);
         while (res.isEmpty()) {

gclc/src/main/java/net/bigeon/gclc/tools/ConstantString.java

@@ -74,6 +74,13 @@ package net.bigeon.gclc.tools;
 public class ConstantString implements StringProvider {
     private final String string;
 
+    /** Create a provider for a string.
+     *
+     * @param string the string */
+    public ConstantString(final String string) {
+        this.string = string;
+    }
+
     /* (non-Javadoc)
      * @see net.bigeon.gclc.tools.StringProvider#apply() */
     @Override
@@ -81,10 +88,37 @@ public class ConstantString implements StringProvider {
         return string;
     }
 
-    /** Create a provider for a string.
-     * 
-     * @param string the string */
-    public ConstantString(String string) {
-        this.string = string;
+    /* (non-Javadoc)
+     * @see java.lang.Object#hashCode() */
+    @Override
+    public int hashCode() {
+        final int prime = 31;
+        int result = 1;
+        result = prime * result + ((string == null) ? 0 : string.hashCode());
+        return result;
+    }
+
+    /* (non-Javadoc)
+     * @see java.lang.Object#equals(java.lang.Object) */
+    @Override
+    public boolean equals(final Object obj) {
+        if (this == obj) {
+            return true;
+        }
+        if (obj == null) {
+            return false;
+        }
+        if (getClass() != obj.getClass()) {
+            return false;
+        }
+        final ConstantString other = (ConstantString) obj;
+        if (string == null) {
+            if (other.string != null) {
+                return false;
+            }
+        } else if (!string.equals(other.string)) {
+            return false;
+        }
+        return true;
     }
 }

gclc/src/main/java/net/bigeon/gclc/tools/PrintUtils.java

@@ -91,13 +91,13 @@ public final class PrintUtils {
         // Utility class
     }
 
-    /** Print the text possibly cutting it if it goes over the authorized length
-     * and adding a mark of continuation.
+    /** Print the text possibly cutting it if it goes over the authorized length and
+     * adding a mark of continuation.
      *
      * @param text the text to print
      * @param nbCharacters the number of characters of the resulting text
-     * @param indicateTooLong if an indication shell be given that the text
-     *            didn't fit
+     * @param indicateTooLong if an indication shell be given that the text didn't
+     *        fit
      * @return the text to print (will be of exactly nbCharacters). */
     public static String print(final String text, final int nbCharacters,
             final boolean indicateTooLong) {
@@ -125,8 +125,7 @@ public final class PrintUtils {
      * @param i the length of the wrap
      * @return the list of resulting strings */
     public static List<String> wrap(final String description, final int i) {
-        final String[] originalLines = description
-                .split(System.lineSeparator());
+        final String[] originalLines = description.split(System.lineSeparator());
         final List<String> result = new ArrayList<>();
         for (final String string : originalLines) {
             String toCut = string;

gclc/src/main/java/net/bigeon/gclc/utils/AOutputForwardRunnable.java

@@ -103,19 +103,18 @@ public abstract class AOutputForwardRunnable implements Runnable {
 
     /** Create a forward runnable with the given timeout.
      * <p>
-     * Short timeout will be very responsive to the application actual messages,
-     * but may use computation time if the application is not verbose. Long
-     * timeout will save computation time, but will read batches of messages at
-     * once if the application is verbose. The right length for the timeout is
-     * likely to depend on the application and the use of it.
+     * Short timeout will be very responsive to the application actual messages, but
+     * may use computation time if the application is not verbose. Long timeout will
+     * save computation time, but will read batches of messages at once if the
+     * application is verbose. The right length for the timeout is likely to depend
+     * on the application and the use of it.
      * <p>
      * If you do not know what timeout length to use, please use the
      * {@link #AOutputForwardRunnable(PipedConsoleOutput)} constructor.
      *
      * @param manager the manager
      * @param timeout the timeout between message requests. */
-    public AOutputForwardRunnable(final PipedConsoleOutput manager,
-            final long timeout) {
+    public AOutputForwardRunnable(final PipedConsoleOutput manager, final long timeout) {
         super();
         this.manager = manager;
         this.timeout = timeout;

gclc/src/main/java/net/bigeon/gclc/utils/EmptyInput.java

@@ -36,7 +36,9 @@
  * gclc:net.bigeon.gclc.manager.EmptyInput.java
  * Created on: Nov 13, 2017
  */
-package net.bigeon.gclc.manager;
+package net.bigeon.gclc.utils;
+
+import net.bigeon.gclc.manager.ConsoleInput;
 
 /*-
  * #%L
@@ -80,12 +82,15 @@ import net.bigeon.gclc.tools.StringProvider;
  * @author Emmanuel Bigeon */
 public final class EmptyInput implements ConsoleInput {
 
+    private static final ConstantString CONSTANT_STRING = new ConstantString("");
     /** The empty prompter. */
     public static final ConsoleInput INSTANCE = new EmptyInput();
+
     /** The empty input. */
     private EmptyInput() {
         //
     }
+
     /* (non-Javadoc)
      * @see net.bigeon.gclc.manager.ConsoleInput#close() */
     @Override
@@ -97,7 +102,7 @@ public final class EmptyInput implements ConsoleInput {
      * @see net.bigeon.gclc.manager.ConsoleInput#getPrompt() */
     @Override
     public StringProvider getPrompt() {
-        return new ConstantString(""); //$NON-NLS-1$
+        return CONSTANT_STRING; //$NON-NLS-1$
     }
 
     /* (non-Javadoc)
@@ -136,11 +141,9 @@ public final class EmptyInput implements ConsoleInput {
     }
 
     /* (non-Javadoc)
-     * @see net.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String,
-     * long) */
+     * @see net.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String, long) */
     @Override
-    public String prompt(final String message,
-                         final long timeout) {
+    public String prompt(final String message, final long timeout) {
         return ""; //$NON-NLS-1$
     }
 

gclc/src/main/java/net/bigeon/gclc/utils/PipedConsoleInput.java

@@ -87,8 +87,7 @@ import net.bigeon.gclc.tools.StringProvider;
  * used to test application behavior.
  *
  * @author Emmanuel Bigeon */
-public final class PipedConsoleInput
-        implements ConsoleInput {
+public final class PipedConsoleInput implements ConsoleInput {
 
     /** THe inner manager. */
     private final StreamConsoleInput innerManager;
@@ -106,11 +105,9 @@ public final class PipedConsoleInput
     public PipedConsoleInput(final PrintStream outPrint) throws IOException {
         commandInput = new PipedOutputStream();
         in = new PipedInputStream(commandInput);
-        innerManager = new StreamConsoleInput(outPrint, in,
-                StandardCharsets.UTF_8);
+        innerManager = new StreamConsoleInput(outPrint, in, StandardCharsets.UTF_8);
         writing = new WritingRunnable(commandInput, StandardCharsets.UTF_8);
-        final Thread th = new Thread(writing,
-                "GCLC console piped input stream"); //$NON-NLS-1$
+        final Thread th = new Thread(writing, "GCLC console piped input stream"); //$NON-NLS-1$
         th.start();
     }
 
@@ -145,8 +142,7 @@ public final class PipedConsoleInput
      * @see net.bigeon.gclc.manager.ConsoleInput#prompt() */
     @Override
     public String prompt() throws IOException {
-        return innerManager
-                .prompt(innerManager.getPrompt() + System.lineSeparator());
+        return innerManager.prompt(innerManager.getPrompt() + System.lineSeparator());
     }
 
     /* (non-Javadoc)
@@ -164,8 +160,7 @@ public final class PipedConsoleInput
     }
 
     /* (non-Javadoc)
-     * @see net.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String,
-     * long) */
+     * @see net.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String, long) */
     @Override
     public String prompt(final String message, final long timeout) throws IOException {
         return innerManager.prompt(message + System.lineSeparator(), timeout);

gclc/src/main/java/net/bigeon/gclc/utils/PipedConsoleOutput.java

@@ -87,8 +87,7 @@ import net.bigeon.gclc.manager.ConsoleOutput;
  * used to test application behavior.
  *
  * @author Emmanuel Bigeon */
-public final class PipedConsoleOutput
-        implements ConsoleOutput {
+public final class PipedConsoleOutput implements ConsoleOutput {
 
     /** The encoding between streams. */
     private static final String       UTF_8 = "UTF-8";  //$NON-NLS-1$
@@ -140,14 +139,14 @@ public final class PipedConsoleOutput
 
     /** Wait for a specific message to arrive.
      * <p>
-     * When this method returns, the message was appended to the data, it
-     * <em>may or may not</em> be the next line of data.
+     * When this method returns, the message was appended to the data, it <em>may or
+     * may not</em> be the next line of data.
      *
      * @param message the message
-     * @return the thread to join to wait for message delivery
-     * @see net.bigeon.gclc.utils.ReadingRunnable#getWaitForDelivery(java.lang.String) */
-    public Thread getWaitForDelivery(final String message) {
-        return reading.getWaitForDelivery(message);
+     * @throws InterruptedException if the wait on the message was interrupted
+     * @see net.bigeon.gclc.utils.ReadingRunnable#waitForDelivery(String) */
+    public void waitForDelivery(final String message) throws InterruptedException {
+        reading.waitForDelivery(message);
     }
 
     /* (non-Javadoc)

gclc/src/main/java/net/bigeon/gclc/utils/ReadingRunnable.java

@@ -76,8 +76,8 @@ import java.io.IOException;
 import java.io.InterruptedIOException;
 import java.util.ArrayDeque;
 import java.util.Deque;
-import java.util.HashMap;
 import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
@@ -86,65 +86,6 @@ import java.util.logging.Logger;
  * @author Emmanuel Bigeon */
 public final class ReadingRunnable implements Runnable {
 
-    /** The runnable to wait for arrival of a message in the queue.
-     *
-     * @author Emmanuel Bigeon */
-    private final class ToWaitRunnable implements Runnable {
-        /** The Object. */
-        private final Object obj;
-        /** The locking object. */
-        private final Object start;
-        /** The message. */
-        private final String message;
-        /** The started status. */
-        private boolean started = false;
-
-        /** Create the waiting runnable.
-         *
-         * @param obj the object to lock on
-         * @param start the object to notify when ready to wait
-         * @param message the message to wait for */
-        public ToWaitRunnable(final Object obj, final Object start,
-                final String message) {
-            this.obj = obj;
-            this.start = start;
-            this.message = message;
-        }
-
-        /** Test if the waiting runnable is started.
-         *
-         * @return the started */
-        public boolean isStarted() {
-            synchronized (start) {
-                return started;
-            }
-        }
-
-        /* (non-Javadoc)
-         * @see java.lang.Runnable#run() */
-        @Override
-        public void run() {
-            synchronized (obj) {
-                synchronized (start) {
-                    started = true;
-                    start.notifyAll();
-                }
-                while (isRunning()) {
-                    try {
-                        obj.wait();
-                        if (delivering.equals(message)) {
-                            return;
-                        }
-                    } catch (final InterruptedException e) {
-                        LOGGER.log(Level.SEVERE, THREAD_INTERRUPTION_EXCEPTION,
-                                e);
-                        Thread.currentThread().interrupt();
-                    }
-                }
-            }
-        }
-    }
-
     /** The thread intteruption logging message. */
     private static final String THREAD_INTERRUPTION_EXCEPTION = "Thread interruption exception."; //$NON-NLS-1$
     /** The closed pipe message. */
@@ -154,8 +95,10 @@ public final class ReadingRunnable implements Runnable {
     /** Class logger. */
     private static final Logger LOGGER                        = Logger
             .getLogger(ReadingRunnable.class.getName());
+
     /** Read messages. */
     private final Deque<String> messages = new ArrayDeque<>();
+
     /** the reader. */
     private final BufferedReader      reader;
     /** the state of this runnable. */
@@ -165,12 +108,9 @@ public final class ReadingRunnable implements Runnable {
     /** The waiting status for a message. */
     private boolean                   waiting;
     /** The blocker for a given message. */
-    private final Map<String, Object> messageBlocker = new HashMap<>();
+    private final Map<String, Object> messageBlocker     = new ConcurrentHashMap<>();
     /** The lock. */
     private final Object              messageBlockerLock = new Object();
-    /** The message being delivered. */
-    private String delivering;
-
     /** Create a reading runnable.
      *
      * @param reader the input to read from */
@@ -179,28 +119,17 @@ public final class ReadingRunnable implements Runnable {
         this.reader = reader;
     }
 
-    /** Strip the string from head NULL characters.
+    /** Do wait for a message.
+     * <p>
+     * This method should only be called inside a loop.
      *
-     * @param line the line to strip the null character from
-     * @return the resulting string */
-    private static String stripNull(final String line) {
-        String res = line;
-        while (res.length() > 0 && res.charAt(0) == 0) {
-            LOGGER.severe(
-                    "NULL character heading the result of the read. This is a stream problem..."); //$NON-NLS-1$
-            res = res.substring(1);
-        }
-        return res;
-    }
-
-    /** do wait for a message.
-     * <p>This method should only be called inside a loop.
      * @param timeout the timeout of the wait.
-     * @throws IOException if the runnable was stopped and no essage was found.
-     */
+     * @throws IOException if the runnable was stopped and no essage was found. */
     private void doWaitMessage(final long timeout) throws IOException {
         try {
+            synchronized (lock) {
                 lock.wait(timeout);
+            }
         } catch (final InterruptedException e) {
             LOGGER.log(Level.SEVERE, THREAD_INTERRUPTION_EXCEPTION, e);
             Thread.currentThread().interrupt();
@@ -221,7 +150,7 @@ public final class ReadingRunnable implements Runnable {
             }
             waiting = true;
             waitMessage(TIMEOUT);
-            LOGGER.finest("Polled: " + messages.peek()); //$NON-NLS-1$
+            LOGGER.log(Level.FINEST, "Polled: {0}", messages.peek()); //$NON-NLS-1$
             waiting = false;
             notifyMessage(messages.peek());
             return messages.poll();
@@ -248,34 +177,16 @@ public final class ReadingRunnable implements Runnable {
         }
     }
     
-    /** Get a waiting thread for a specific message delivery.
-     *
-     * @param message the message
-     * @return the thread to join to wait for message delivery */
-    public Thread getWaitForDelivery(final String message) {
+    public void waitForDelivery(final String message) throws InterruptedException {
+        Object mLock;
         synchronized (messageBlockerLock) {
             if (!messageBlocker.containsKey(message)) {
                 messageBlocker.put(message, new Object());
             }
-            final Object obj = messageBlocker.get(message);
-            final Object start = new Object();
-            final ToWaitRunnable waitRunn = new ToWaitRunnable(obj, start,
-                    message);
-            final Thread th = new Thread(waitRunn);
-
-            synchronized (start) {
-                th.start();
-                while (!waitRunn.isStarted()) {
-                    try {
-                        start.wait(TIMEOUT);
-                    } catch (final InterruptedException e) {
-                        LOGGER.log(Level.SEVERE, THREAD_INTERRUPTION_EXCEPTION,
-                                e);
-                        Thread.currentThread().interrupt();
+            mLock = messageBlocker.get(message);
         }
-                }
-            }
-            return th;
+        synchronized (mLock) {
+            mLock.wait();
         }
     }
     
@@ -285,15 +196,15 @@ public final class ReadingRunnable implements Runnable {
      * @throws IOException if the pipe is closed */
     public boolean hasMessage() throws IOException {
         synchronized (lock) {
-            if (!running) {
+            final boolean empty = messages.isEmpty();
+            if (!running && empty) {
                 throw new IOException(CLOSED_PIPE);
             }
-            return !messages.isEmpty();
+            return !empty;
         }
     }
 
-    /** Interrupts the wait on the next message by providing an empty
-     * message. */
+    /** Interrupts the wait on the next message by providing an empty message. */
     public void interrupt() {
         synchronized (lock) {
             if (waiting) {
@@ -317,9 +228,8 @@ public final class ReadingRunnable implements Runnable {
      * @param message the message */
     private void notifyMessage(final String message) {
         synchronized (messageBlockerLock) {
-            delivering = message;
-            if (messageBlocker.containsKey(message)) {
             final Object mLock = messageBlocker.get(message);
+            if (mLock!=null) {
                 synchronized (mLock) {
                     mLock.notifyAll();
                 }
@@ -332,32 +242,41 @@ public final class ReadingRunnable implements Runnable {
      * @see java.lang.Runnable#run() */
     @Override
     public void run() {
-        while (running) {
+        while (true) {
+            synchronized (lock) {
+                if (!running) {
+                    break;
+                }
+            }
             try {
                 String line = reader.readLine();
                 if (line == null) {
                     // Buffer end
-                    running = false;
+                    setRunning(false);
                     return;
                 }
-                LOGGER.finer("Read: " + line); //$NON-NLS-1$
+                LOGGER.log(Level.FINER, "Read: {0}", line); //$NON-NLS-1$
                 line = stripNull(line);
                 synchronized (lock) {
                     messages.add(line);
                     lock.notifyAll();
                 }
             } catch (final InterruptedIOException e) {
+                synchronized (lock) {
                     if (running) {
                         LOGGER.info("Reading interrupted"); //$NON-NLS-1$
                     }
-                LOGGER.log(Level.FINER,
-                        "Read interruption was caused by an exception", e); //$NON-NLS-1$
+                }
+                LOGGER.log(Level.FINER, "Read interruption was caused by an exception", //$NON-NLS-1$
+                        e);
             } catch (final IOException e) {
                 LOGGER.log(Level.FINE, "The stream reading threw an exception", //$NON-NLS-1$
                         e);
+                synchronized (lock) {
                     if (running) {
                         LOGGER.severe("Unable to read from stream"); //$NON-NLS-1$
-                    running = false;
+                        setRunning(false);
+                    }
                 }
                 return;
             }
@@ -373,14 +292,28 @@ public final class ReadingRunnable implements Runnable {
         }
     }
 
-    /** Wait for the next message to be integratted.
+    /** Wait for the next message to be integrated.
      *
      * @param timeout the timeout to wait
-     * @throws IOException if the next message was not delivered and the
-     *             runnable stopped. */
+     * @throws IOException if the next message was not delivered and the runnable
+     *         stopped. */
     private void waitMessage(final long timeout) throws IOException {
         while (messages.isEmpty()) {
             doWaitMessage(timeout);
         }
     }
+
+    /** Strip the string from head NULL characters.
+     *
+     * @param line the line to strip the null character from
+     * @return the resulting string */
+    private static String stripNull(final String line) {
+        String res = line;
+        while (res.length() > 0 && res.charAt(0) == 0) {
+            LOGGER.severe(
+                    "NULL character heading the result of the read. This is a stream problem..."); //$NON-NLS-1$
+            res = res.substring(1);
+        }
+        return res;
+    }
 }

gclc/src/main/java/net/bigeon/gclc/utils/SinkOutput.java

@@ -85,6 +85,7 @@ public final class SinkOutput implements ConsoleOutput {
     private SinkOutput() {
         //
     }
+
     /* (non-Javadoc)
      * @see net.bigeon.gclc.manager.ConsoleOutput#close() */
     @Override
@@ -114,8 +115,7 @@ public final class SinkOutput implements ConsoleOutput {
     }
 
     /* (non-Javadoc)
-     * @see net.bigeon.gclc.manager.ConsoleOutput#println(java.lang.String)
-     */
+     * @see net.bigeon.gclc.manager.ConsoleOutput#println(java.lang.String) */
     @Override
     public void println(final String message) {
         //

gclc/src/main/java/net/bigeon/gclc/utils/StreamConsoleInput.java

@@ -101,7 +101,7 @@ public final class StreamConsoleInput implements ConsoleInput {
     private final BufferedReader in;
 
     /** If the manager is closed. */
-    private boolean closed = false;
+    private boolean closed;
 
     /** The prompting thread. */
     private final Thread promptThread;
@@ -109,8 +109,8 @@ public final class StreamConsoleInput implements ConsoleInput {
     /** The reading runnable. */
     private final ReadingRunnable reading;
 
-    /** This default constructor relies on the system defined standart output
-     * and input stream. */
+    /** This default constructor relies on the system defined standart output and
+     * input stream. */
     public StreamConsoleInput() {
         this(System.out, System.in, Charset.defaultCharset());
     }
@@ -143,7 +143,7 @@ public final class StreamConsoleInput implements ConsoleInput {
     /* (non-Javadoc)
      * @see net.bigeon.gclc.manager.ConsoleManager#close() */
     @Override
-    public void close() throws IOException {
+    public void close() {
         closed = true;
         reading.setRunning(false);
         promptThread.interrupt();
@@ -200,8 +200,7 @@ public final class StreamConsoleInput implements ConsoleInput {
     /* (non-Javadoc)
      * @see net.bigeon.gclc.ConsoleManager#prompt(java.lang.String) */
     @Override
-    public String prompt(final String message,
-                         final long timeout) throws IOException {
+    public String prompt(final String message, final long timeout) throws IOException {
         checkOpen();
         if (out != null) {
             out.print(message);
@@ -219,6 +218,6 @@ public final class StreamConsoleInput implements ConsoleInput {
 
     @Override
     public void setPrompt(StringProvider string) {
-    	this.prompt = string;
+        prompt = string;
     }
 }

gclc/src/main/java/net/bigeon/gclc/utils/StreamConsoleOutput.java

@@ -85,10 +85,10 @@ public final class StreamConsoleOutput implements ConsoleOutput {
     /** The print stream. */
     private final PrintStream out;
     /** If the manager is closed. */
-    private boolean closed = false;
+    private boolean           closed;
 
-    /** This default constructor relies on the system defined standart output
-     * and input stream. */
+    /** This default constructor relies on the system defined standart output and
+     * input stream. */
     public StreamConsoleOutput() {
         this(System.out);
     }

gclc/src/main/java/net/bigeon/gclc/utils/WritingRunnable.java

@@ -143,15 +143,14 @@ public final class WritingRunnable implements Runnable {
     public void run() {
         while (running) {
             synchronized (lock) {
-                while (messages.isEmpty()) {
                 waitNextMessage();
                 if (!running) {
                     return;
                 }
-                }
                 writeMessage();
             }
         }
+
     }
 
     /** Set the running status.
@@ -166,11 +165,16 @@ public final class WritingRunnable implements Runnable {
     /** Wait for next message. */
     private void waitNextMessage() {
         try {
+            synchronized (lock) {
+                while (running && messages.isEmpty()) {
                     lock.wait(TIMEOUT);
+                }
+            }
         } catch (final InterruptedException e) {
+            synchronized (lock) {
                 if (running) {
-                LOGGER.log(Level.SEVERE,
-                        "Thread interruption exception.", e); //$NON-NLS-1$
+                    LOGGER.log(Level.SEVERE, "Thread interruption exception.", e); //$NON-NLS-1$
+                }
             }
             Thread.currentThread().interrupt();
         }
@@ -179,8 +183,7 @@ public final class WritingRunnable implements Runnable {
     /** Write next message to output. */
     private void writeMessage() {
         final String message = messages.poll();
-        final ByteBuffer buff = charset
-                .encode(message + System.lineSeparator());
+        final ByteBuffer buff = charset.encode(message + System.lineSeparator());
         if (buff.hasArray()) {
             try {
                 outPrint.write(buff.array());

gclc/src/main/java/net/bigeon/gclc/utils/package-info.java

@@ -1,7 +1,4 @@
-/**
- * @author Emmanuel Bigeon
- *
- */
+/** @author Emmanuel Bigeon */
 package net.bigeon.gclc.utils;
 
 /*-

gclc/src/test/java/net/bigeon/gclc/CommandTestingApplication.java

@@ -73,7 +73,6 @@ package net.bigeon.gclc;
  */
 import java.io.IOException;
 
-import net.bigeon.gclc.ConsoleApplication;
 import net.bigeon.gclc.command.ICommand;
 import net.bigeon.gclc.exception.InvalidCommandName;
 import net.bigeon.gclc.utils.PipedConsoleInput;
@@ -83,9 +82,7 @@ import net.bigeon.gclc.utils.PipedConsoleOutput;
  * <p>
  * TODO
  *
- * @author Emmanuel Bigeon
- *
- */
+ * @author Emmanuel Bigeon */
 @SuppressWarnings("javadoc")
 public class CommandTestingApplication implements AutoCloseable {
 

gclc/src/test/java/net/bigeon/gclc/ConsoleApplicationTest.java

@@ -87,8 +87,6 @@ import java.nio.charset.StandardCharsets;
 
 import org.junit.Test;
 
-import net.bigeon.gclc.CommandRequestListener;
-import net.bigeon.gclc.ConsoleApplication;
 import net.bigeon.gclc.command.ICommand;
 import net.bigeon.gclc.command.base.ExitCommand;
 import net.bigeon.gclc.exception.CommandRunException;
@@ -103,7 +101,7 @@ import net.bigeon.gclc.utils.PipedConsoleOutput;
 /** Test class for ConsoleApplication
  *
  * @author Emmanuel Bigeon */
-@SuppressWarnings({"javadoc", "nls", "static-method"})
+@SuppressWarnings({ "javadoc", "nls", "static-method" })
 public class ConsoleApplicationTest {
 
     /** 3 seconds in milliseconds */
@@ -117,10 +115,8 @@ public class ConsoleApplicationTest {
                 PipedInputStream pis = new PipedInputStream(pout);
                 BufferedReader buf = new BufferedReader(
                         new InputStreamReader(pis, StandardCharsets.UTF_8));
-             PipedConsoleInput in = new PipedConsoleInput(
-                     new PrintStream(pout))) {
-            final ConsoleApplication app = new ConsoleApplication(null, in,
-                    "", "");
+                PipedConsoleInput in = new PipedConsoleInput(new PrintStream(pout))) {
+            final ConsoleApplication app = new ConsoleApplication(null, in, "", "");
             app.exit();
         } catch (final IOException e) {
             fail("System Console Manager failed");
@@ -129,14 +125,13 @@ public class ConsoleApplicationTest {
     }
 
     @Test
-    public void testExecution() throws IOException, InterruptedException,
-                                InvalidCommandName {
+    public void testExecution()
+            throws IOException, InterruptedException, InvalidCommandName {
         try (CommandTestingApplication application = new CommandTestingApplication()) {
 
             // remove welcome
             assertEquals("Header should be preserved",
-                    application.getApplication().header,
-                    application.readNextLine());
+                    application.getApplication().header, application.readNextLine());
             // Remove first prompt
             application.sendCommand("");
             application.sendCommand("test");
@@ -146,15 +141,13 @@ public class ConsoleApplicationTest {
             assertEquals("Command fail should dispaly appropriate message",
                     Messages.getString("ConsoleApplication.cmd.failed", "toto"),
                     application.readNextLine());
-            assertEquals(
-                    "Unrecognized comment should result in a specific message.",
+            assertEquals("Unrecognized comment should result in a specific message.",
                     Messages.getString("CommandProvider.unrecognized", "toto"),
                     application.readNextLine());
             application.sendCommand("long");
             assertEquals("Before wait should receive message", "Waita minute",
                     application.readNextLine());
-            assertEquals("Unexpected message", "done!",
-                    application.readNextLine());
+            assertEquals("Unexpected message", "done!", application.readNextLine());
 
             final CommandRequestListener crl = new CommandRequestListener() {
 
@@ -184,8 +177,7 @@ public class ConsoleApplicationTest {
 
             application.sendCommand("exit");
             assertEquals("Footer should be preserved",
-                    application.getApplication().footer,
-                    application.readNextLine());
+                    application.getApplication().footer, application.readNextLine());
             assertFalse("Stopped application should not be running",
                     application.getApplication().isRunning());
         }
@@ -196,10 +188,9 @@ public class ConsoleApplicationTest {
                 PipedInputStream pis = new PipedInputStream(pout);
                 BufferedReader buf = new BufferedReader(
                         new InputStreamReader(pis, StandardCharsets.UTF_8));
-             PipedConsoleInput in = new PipedConsoleInput(
-                     new PrintStream(pout))) {
-            final ConsoleApplication app = new ConsoleApplication(manager, in,
-                    null, null);
+                PipedConsoleInput in = new PipedConsoleInput(new PrintStream(pout))) {
+            final ConsoleApplication app = new ConsoleApplication(manager, in, null,
+                    null);
             appli = app;
             app.add(new ExitCommand("exit", app));
 
@@ -219,8 +210,7 @@ public class ConsoleApplicationTest {
                 "Application should still exist even if the console input and output are closed.",
                 appli);
         appli.start();
-        assertFalse(
-                "Application should not start on closed console input and output",
+        assertFalse("Application should not start on closed console input and output",
                 appli.isRunning());
     }
 
@@ -228,8 +218,7 @@ public class ConsoleApplicationTest {
     public void testInterpretCommand() throws InvalidCommandName, IOException {
         try (PipedConsoleInput test = new PipedConsoleInput(null);
                 PipedConsoleOutput out = new PipedConsoleOutput()) {
-            final ConsoleApplication appl = new ConsoleApplication(out, test,
-                    "", "");
+            final ConsoleApplication appl = new ConsoleApplication(out, test, "", "");
 
             appl.interpretCommand("invalid cmd \"due to misplaced\"quote");
             assertEquals("Specific error message expected",
@@ -245,11 +234,9 @@ public class ConsoleApplicationTest {
                  * manager.ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput,
                  * java.lang.String[]) */
                 @Override
-                public void execute(final ConsoleOutput out,
-                                    final ConsoleInput in,
+                public void execute(final ConsoleOutput out, final ConsoleInput in,
                         final String... args) throws CommandRunException {
-                    throw new CommandRunException(CommandRunExceptionType.USAGE,
-                            message);
+                    throw new CommandRunException(CommandRunExceptionType.USAGE, message);
                 }
 
                 @Override
@@ -258,8 +245,8 @@ public class ConsoleApplicationTest {
                 }
 
                 @Override
-                public void help(final ConsoleOutput manager,
-                                 final String... args) throws IOException {
+                public void help(final ConsoleOutput manager, final String... args)
+                        throws IOException {
                     manager.println(message);
                 }
 

gclc/src/test/java/net/bigeon/gclc/ConsoleTestApplication.java

@@ -69,12 +69,10 @@ package net.bigeon.gclc;
  */
 import java.io.IOException;
 
-import net.bigeon.gclc.ApplicationAttachement;
-import net.bigeon.gclc.ConsoleApplication;
 import net.bigeon.gclc.command.Command;
-import net.bigeon.gclc.command.HelpExecutor;
 import net.bigeon.gclc.command.ICommandProvider;
 import net.bigeon.gclc.command.base.ExitCommand;
+import net.bigeon.gclc.command.base.HelpExecutor;
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.exception.InvalidCommandName;
 import net.bigeon.gclc.manager.ConsoleInput;