RFR: 8035648: Don't use Handle in java_lang_String::print
Stefan Karlsson
stefan.karlsson at oracle.com
Wed Feb 26 12:12:33 PST 2014
On 2014-02-26 20:37, Coleen Phillimore wrote:
>
> This looks fine. It's a leaf function and it's pretty clear that
> there are no safepoints In the past, we converted all the calls to
> take Handles whenever possible because it saved a lot of
> re-handleizing and bugs with unhandled oops, and we prefer that unless
> there's a good reason. Here there is a good reason and it's
> obviously a leaf function, so this is fine. I don't think you should
> add No_Safepoint_Verifier.
Thanks for reviewing!
StefanK
>
> Thanks,
> Coleen
>
> On 2/24/2014 4:44 AM, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please, review this change to let java_lang_String::print take a raw
>> String oop instead of a Handle to a String oop. This change makes
>> java_lang_String::print callable from code that has no HandleMark
>> setup. E.g. when calling from a debugger.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~stefank/8035648/webrev.00/
>>
>> RFE:
>> https://bugs.openjdk.java.net/browse/JDK-8035648
>>
>> thanks,
>> StefanK
>
More information about the hotspot-dev
mailing list