RFR: 8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way

[Andreas Lundblad]

On Mon, May 05, 2014 at 01:18:03PM +0200, Fredrik Öhrström wrote:
> The output from stdout and stderr from the background must always be logged
> into a file. The default name for that file is portfile+".stdouterr"
> Thus you could replace stdouterrfile == null with
> 
> assert(stdouterrfile != null && stdouterrfile.length() >= 1)
> 
> And get rid of /dev/null etc.

Fixed. (I went for IllegalArgumentException to avoid nasty exceptions when running without -ea.)

See new revision here:

http://cr.openjdk.java.net/~alundblad/8042088/webrev.01/

-- Andreas