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

Neil Richards neil.richards at ngmr.net
Mon Oct 10 14:12:14 UTC 2011


On Mon, 2011-10-10 at 13:58 +0100, Steve Poole wrote:
> 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
> 
> 

Hi Steve,
For ease of review, I've uploaded the suggested patch as a webrev [1].

The change looks good to me.

Regards,
Neil

[1] http://cr.openjdk.java.net/~ngmr/ojdk-228/webrev.00/

-- 
Unless stated above:
IBM email: neil_richards at uk.ibm.com
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU




More information about the core-libs-dev mailing list