Request for review - removal of unused dlinfo variable in java_md.c

Steve Poole spoole at linux.vnet.ibm.com
Mon Oct 10 12:58:13 UTC 2011


hi all,

Please find attached a trivial patch to remove an unused variable in
jdk/src/solaris/bin/java_md.c

The present of the variable causes the AIX build to fail since Dl_info
is not supported on AIX.    The other use of Dl_info in the file is
safely wrapped with an #if defined(__solaris__)  

I have built with the  change successfully on AIX and Linux.

Cheers

Steve


-------------- next part --------------
# HG changeset patch
# User Steve Poole <spoole at linux.vnet.ibm.com>
# Date 1318250829 -3600
# Node ID bf46fe8768c71a47d8fee9b784653b6f0eb971f6
# Parent  f1ec21b8142168ff40f3278d2f6b5fe4bd5f3b26
Remove unused dlinfo variable. (fails to build on AIX)

diff -r f1ec21b81421 -r bf46fe8768c7 src/solaris/bin/java_md.c
--- a/src/solaris/bin/java_md.c	Thu Oct 06 14:01:37 2011 -0700
+++ b/src/solaris/bin/java_md.c	Mon Oct 10 13:47:09 2011 +0100
@@ -820,7 +820,6 @@
 jboolean
 LoadJavaVM(const char *jvmpath, InvocationFunctions *ifn)
 {
-    Dl_info dlinfo;
     void *libjvm;
 
     JLI_TraceLauncher("JVM path is %s\n", jvmpath);


More information about the core-libs-dev mailing list