diff options
author | Laszlo Csomor <laszlocsomor@google.com> | 2016-08-31 09:25:40 +0000 |
---|---|---|
committer | Klaus Aehlig <aehlig@google.com> | 2016-08-31 14:51:00 +0000 |
commit | a79f832e036ec9de5dd23d38acbc3d9d22c0ed4d (patch) | |
tree | 5b9b6b2eb9742f7ed1e5ea19e84f058c20070d08 /src/main/java/com/google/devtools/build/lib/windows | |
parent | 27f9661e50af0f6a947f36e307d31b52500eff23 (diff) |
Windows, JNI: do not load windows_jni.dll twice.
WindowsProcesses.ensureJni() was duplicating the work of
WindowsJniLoader.loadJni(); this change removes the former and
replaces the only call site with the latter.
--
MOS_MIGRATED_REVID=131806917
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/windows')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/windows/WindowsProcesses.java | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/windows/WindowsProcesses.java b/src/main/java/com/google/devtools/build/lib/windows/WindowsProcesses.java index 5774f0f26b..67d6362afd 100644 --- a/src/main/java/com/google/devtools/build/lib/windows/WindowsProcesses.java +++ b/src/main/java/com/google/devtools/build/lib/windows/WindowsProcesses.java @@ -22,7 +22,6 @@ import java.util.List; public class WindowsProcesses { public static final long INVALID = -1; - private static boolean jniLoaded = false; private WindowsProcesses() { // Prevent construction } @@ -132,19 +131,10 @@ public class WindowsProcesses { static native String nativeStreamGetLastError(long process); public static int getpid() { - ensureJni(); + WindowsJniLoader.loadJni(); return nativeGetpid(); } - private static synchronized void ensureJni() { - if (jniLoaded) { - return; - } - - System.loadLibrary("windows_jni"); - jniLoaded = true; - } - static String quoteCommandLine(List<String> argv) { StringBuilder result = new StringBuilder(); for (int iArg = 0; iArg < argv.size(); iArg++) { |