Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work

John Rose john.r.rose at oracle.com
Thu Feb 21 11:20:23 PST 2013


On Feb 21, 2013, at 11:16 AM, "serguei.spitsyn at oracle.com" <serguei.spitsyn at oracle.com> wrote:

> I think, the John's suggestion for odd strings should work here.
> Something like:
>   if (sym == NULL) {
>     str = "CONSTANT_PLACEHOLDER_42";
>   }
> 
> I will file a bug on this.

Yes, that looks good.  I suggest either the empty string, or something slightly informative such as "patch#42", where 42 is the CP index.   — John
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20130221/2e07b802/attachment.html 


More information about the serviceability-dev mailing list