Re: code review request for more StringTable verification code (8019835)

2013-09-16 Thread Daniel D. Daugherty
Subject: Re: code review request for more StringTable verification code (8019835) Thanks for quick re-review! Dan On 9/16/13 11:43 AM, serguei.spit...@oracle.com wrote: Looks nice. Thanks, Serguei On 9/16/13 10:08 AM, Daniel D. Daugherty wrote: Greetings, I have some new diagnostic code that I

RE: code review request for more StringTable verification code (8019835)

2013-09-16 Thread Ron Durbin
Dan it looks good, Thx Ron > -Original Message- > From: Daniel D. Daugherty > Sent: Monday, September 16, 2013 11:48 AM > To: Serguei Spitsyn > Cc: serviceability-dev@openjdk.java.net; hotspot-runtime-...@openjdk.java.net > Subject: Re: code review request fo

Re: code review request for more StringTable verification code (8019835)

2013-09-16 Thread serguei.spit...@oracle.com
Looks nice. Thanks, Serguei On 9/16/13 10:08 AM, Daniel D. Daugherty wrote: Greetings, I have some new diagnostic code that I would like to add to the system that was developed during my hunt for this bug: 8019835 Strings interned in different threads equal but does not == http://bugs

Re: code review request for more StringTable verification code (8019835)

2013-09-16 Thread Daniel D. Daugherty
Thanks for quick re-review! Dan On 9/16/13 11:43 AM, serguei.spit...@oracle.com wrote: Looks nice. Thanks, Serguei On 9/16/13 10:08 AM, Daniel D. Daugherty wrote: Greetings, I have some new diagnostic code that I would like to add to the system that was developed during my hunt for this bu