RFR (XXS): 8010332: remove unused method ciMethod::uses_monitors

[Christian Thalinger]

[I fixed the subject line.]

On Apr 25, 2013, at 1:54 AM, Albert Noll <albert.noll at oracle.com> wrote:

> On 04/24/2013 09:21 PM, Christian Thalinger wrote:
>> On Apr 24, 2013, at 2:18 AM, Albert Noll <albert.noll at oracle.com> wrote:
>> 
>>> Hi all,
>>> 
>>> please review the patch:
>>> 
>>> 
>>> Problem:
>>> unused method with misleading name
>>> 
>>> Fix:
>>> removed unused method
>> 
>> The change looks good but are you sure you're not mixing up bugs?
>> 
>> -- Chris
>> 
> Hi, thanks Chris. Yes, the link points to the wrong bug. Here is the correct link.
> 
> https://jbs.oracle.com/bugs/browse/JDK-8010332

And the correct link to the webrev:

http://cr.openjdk.java.net/~roland/8010332/webrev.00/

(I hate URLs where the target is actually different than the name; it's hard to spot.)

The change looks good though.

-- Chris

> 
> Sorry for the confusion,
> Albert
>>> 
>>> Testing:
>>> JPRT 'compiler'
>>> 
>>> webrev:  http://cr.openjdk.java.net/~roland/8010332/webrev.00/
>>> jbs: https://jbs.oracle.com/bugs/browse/JDK-8010332
>> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20130425/91fb6c9e/attachment.html