Review

When a DataOutputStream instance wraps an underlying OutputStream instance,
it is recommended to flush or close the DataOutputStream before operating on the OutputStream.
Although in these cases this is not strictly necessary because the
DataOutputStream's close and flush method has no effects. However, it is a good practice to call
flush/close explicitly as mentioned for example here.
This pull request adds a call to flush.

• missing/call

net/sf/ntru/sign/SignatureParameters.java

writeTo(OutputStream)

class SignatureParameters {
    /**
     * Writes the parameter set to an output stream
     * @param os an output stream
     * @throws IOException
     */
    public void writeTo(OutputStream os) throws IOException {
        DataOutputStream dos = new DataOutputStream(os);
        dos.writeInt(N);
        dos.writeInt(q);
        dos.writeInt(d);
        dos.writeInt(d1);
        dos.writeInt(d2);
        dos.writeInt(d3);
        dos.writeInt(B);
        dos.writeInt(basisType.ordinal());
        dos.writeFloat(beta);
        dos.writeFloat(normBound);
        dos.writeFloat(keyNormBound);
        dos.writeInt(signFailTolerance);
        dos.writeBoolean(primeCheck);
        dos.writeBoolean(sparse);
        dos.writeInt(bitsF);
        dos.write(keyGenAlg.ordinal());
        dos.writeUTF(hashAlg);
        dos.write(polyType.ordinal());
    }

}