Unused code in prims/methodComparator

[David Holmes]

HI Thomas,

Ultimately this is part of JVMTI so adding the serviceability folk.

On 20/01/2016 6:31 PM, Thomas Stüfe wrote:
> Hi all,
>
> when investigating a warning about unchecked realloc() usage in
> methodComparator.hpp, I found that the offending function
> (BciMap::store_fragment_location) is not callable anymore.
>
> BciMap::store_fragment_location is called
> by MethodComparator::methods_switchable, which is not called anywhere.
>
> Is there any reason we should keep this?

Certainly seems to be dead code. Unclear when/why redefinition dropped 
the notion of "switchable".

Thanks,
David

> Regards, Thomas
>