8026808: serviceability/sa/jmap-hprof/JMapHProfLargeHeapTest.java failed with unexpected exit value

Fixes a bug with vmArgs when using JDKToolLauncher

Reviewed-by: sla, dholmes
This commit is contained in:
Fredrik Arvidsson 2013-10-23 10:24:28 +02:00
parent 286c6a3d50
commit e731a6d078

View file

@ -100,7 +100,7 @@ public class JDKToolLauncher {
* @return The JDKToolLauncher instance
*/
public JDKToolLauncher addVMArg(String arg) {
vmArgs.add("-J" + arg);
vmArgs.add(arg);
return this;
}
@ -124,7 +124,10 @@ public class JDKToolLauncher {
public String[] getCommand() {
List<String> command = new ArrayList<String>();
command.add(executable);
command.addAll(vmArgs);
// Add -J in front of all vmArgs
for (String arg : vmArgs) {
command.add("-J" + arg);
}
command.addAll(toolArgs);
return command.toArray(new String[command.size()]);
}