From a9374cc0bcf024330afeb84517a1d3191eb1be5a Mon Sep 17 00:00:00 2001 From: Igor Ignatyev Date: Tue, 11 Apr 2017 21:59:26 -0700 Subject: [PATCH] 8178415: remove ProcessTools::getPlatformSpecificVMArgs from testlibary Reviewed-by: dholmes --- .../jdk/testlibrary/JDKToolLauncher.java | 2 -- .../jdk/testlibrary/ProcessTools.java | 18 ------------------ 2 files changed, 20 deletions(-) diff --git a/jdk/test/lib/testlibrary/jdk/testlibrary/JDKToolLauncher.java b/jdk/test/lib/testlibrary/jdk/testlibrary/JDKToolLauncher.java index cd3b7cea022..c19cb5df63a 100644 --- a/jdk/test/lib/testlibrary/jdk/testlibrary/JDKToolLauncher.java +++ b/jdk/test/lib/testlibrary/jdk/testlibrary/JDKToolLauncher.java @@ -24,7 +24,6 @@ package jdk.testlibrary; import java.util.ArrayList; -import java.util.Arrays; import java.util.List; /** @@ -60,7 +59,6 @@ public class JDKToolLauncher { } else { executable = JDKToolFinder.getTestJDKTool(tool); } - vmArgs.addAll(Arrays.asList(ProcessTools.getPlatformSpecificVMArgs())); } /** diff --git a/jdk/test/lib/testlibrary/jdk/testlibrary/ProcessTools.java b/jdk/test/lib/testlibrary/jdk/testlibrary/ProcessTools.java index 8246c8fd707..cc3a93ffcab 100644 --- a/jdk/test/lib/testlibrary/jdk/testlibrary/ProcessTools.java +++ b/jdk/test/lib/testlibrary/jdk/testlibrary/ProcessTools.java @@ -259,23 +259,6 @@ public final class ProcessTools { return ProcessHandle.current().getPid(); } - /** - * Get platform specific VM arguments (e.g. -d64 on 64bit Solaris) - * - * @return String[] with platform specific arguments, empty if there are - * none - */ - public static String[] getPlatformSpecificVMArgs() { - String osName = System.getProperty("os.name"); - String dataModel = System.getProperty("sun.arch.data.model"); - - if (osName.equals("SunOS") && dataModel.equals("64")) { - return new String[] { "-d64" }; - } - - return new String[] {}; - } - /** * Create ProcessBuilder using the java launcher from the jdk to be tested, * and with any platform specific arguments prepended. @@ -301,7 +284,6 @@ public final class ProcessTools { ArrayList args = new ArrayList<>(); args.add(javapath); - Collections.addAll(args, getPlatformSpecificVMArgs()); if (addTestVmAndJavaOptions) { // -cp is needed to make sure the same classpath is used whether the test is