Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work (original) (raw)
John Rose john.r.rose at oracle.com
Thu Feb 21 11:20:23 PST 2013
- Previous message: Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work
- Next message: Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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 = "CONSTANTPLACEHOLDER42"; }
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
- Previous message: Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work
- Next message: Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]