Exception when using SeparateJvmsExecutor with verbose key set to false.

[kjanosz]

(I'm rewriting the last comment from #135)

I've encountered into some strange behaviour when spawning new JVM with default jvmflags ("-Xmx2048m -Xms2048m -XX:CompileThreshold=100")

Error: Could not find or load main class  -Xmx2048m -Xms2048m -XX:CompileThreshold=100

Note the extra space before -Xmx2048m.

Changing line 27 in SeparateJvmsExecutor from:
val flags = s"${if (currentContext(Key.verbose)) "-verbose:gc " else ""}" + existingFlags
to:
val flags = s"${if (currentContext(Key.verbose)) "-verbose:gc" else ""} " + existingFlags
changes only error msg:

Invalid maximum heap size: -Xmx2048m -Xms2048m -XX:CompileThreshold=100
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit

[kjanosz]

@axel22: I've done some quickfix and changing jvmflags key type from String to List[String] works. I could make a PR if you want.

Alternatively splitting over jvmflags before creation of commands sequence in JvmRunner.runJvm should work too, but won't it break usages when any of flags contain whitepsace? (don't know if this is allowed behaviour)

[axel22]

It sounds as if the multiple command-line arguments were merged into a single argument. This could be related to the recent whitespace-related refactoring.

About List[String] - yes, please! :)