RFR(xs): 8149096: Remove unused code in methodComparator

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Mon Feb 8 19:22:27 UTC 2016


Hi Thomas,

The fix looks good.
The following variable and all its uses are dead too as it never gets 
'true' value.
But it is up to you to fix it or not.

I can sponsor and push the fix.

Thank you for taking care about this.
Serguei


On 2/8/16 06:24, Thomas Stüfe wrote:
> Hi all,
>
> please review and sponsor this little change. It gets rid of two 
> unused methods in methodComparator(cpp/hpp).
>
> We stumbled over this because of warings about unchecked realloc() 
> return values; but instead of fixing the code, we may just remove the 
> offending methods, because they are not used anywhere.
>
> See also mail thread here: 
> http://openjdk.5641.n7.nabble.com/Unused-code-in-prims-methodComparator-td254425.html.
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8149096
>
> webrev: 
> http://cr.openjdk.java.net/~stuefe/webrevs/8149096-remove-unused-code-methodComparator/webrev.00/webrev/ 
> <http://cr.openjdk.java.net/%7Estuefe/webrevs/8149096-remove-unused-code-methodComparator/webrev.00/webrev/> 
>
>
> Kind Regards, Thomas



More information about the hotspot-runtime-dev mailing list