FeatureFileDumper main method could handle exceptions better

[davidmoten]

The main method in FeatureFileDumper catches exceptions and writes to System.err but does not exit the process with a non-zero exit code nor throw a RuntimeException. One of these should occur.

        try {
            ...
        } catch (IOException ioe) {
            System.err.println("I/O Error " + ioe);
        } catch (PropertyException p) {
            System.err.println("Bad configuration " + p);
        }

[nshmyrev]

Thank you for your report. In processing large databases it's ok to have issues. Stopping the work is not a good idea the, it's better to warn and proceed. The filedumper needs rewokr though.

[davidmoten]

I agree that sometimes the caller would want problems to be ignored but it should be up to the caller to do the ignoring (or not ignore). Putting output to stderr only is not a great way of signaling a problem in my opinion.

My reading indicates that the consensus for main method usage is to call System.exit as the main method is expected to be called from scripts rather than Java code. Scripts can ignore errors if they want to. On 13 Jan 2015 08:14, "Nickolay V. Shmyrev" notifications@github.com wrote:

Thank you for your report. In processing large databases it's ok to have issues. Stopping the work is not a good idea the, it's better to warn and proceed. The filedumper needs rewokr though.

— Reply to this email directly or view it on GitHub https://github.com/cmusphinx/sphinx4/issues/18#issuecomment-69647003.