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
