Looks good for me.
-Dmitry
On 2014-01-03 21:42, Kevin Walls wrote:
Hi,
This problem means you can't use the SA if the target app contains a
symbol which uses a non-ascii character. The SA tool will fail with an
error, the JVM itself and the SA having calculated different hashes for
such
Thanks!
On 17/01/14 14:17, Dmitry Samersoff wrote:
Looks good for me.
-Dmitry
On 2014-01-03 21:42, Kevin Walls wrote:
Hi,
This problem means you can't use the SA if the target app contains a
symbol which uses a non-ascii character. The SA tool will fail with an
error, the JVM itself and
Thanks - just need that second reviewer then...
On 13/01/14 12:38, Staffan Larsen wrote:
Looks good! Thanks for taking the time to re-write the test in Java.
Thanks,
/Staffan
On 8 jan 2014, at 15:59, Kevin Walls kevin.wa...@oracle.com wrote:
Hi Staffan -
Hi Staffan -
http://cr.openjdk.java.net/~kevinw/8028623/webrev.01/
Yes it's better now, getting pid and launching a tool etc have been
previous reasons to use a script, but with this new help it's not too
bad!...
Thanks,
Kevin
On 07/01/14 09:43, Staffan Larsen wrote:
Kevin,
The fix
Kevin,
The fix looks good.
For tests, we are trying to avoid adding new shell-script based tests since
they too often cause problems. Would it be possible to rewrite the test in pure
Java code? There are some helper routines in
test/testlibrary/com/oracle/java/testlibrary/ that could be
Hi,
This problem means you can't use the SA if the target app contains a
symbol which uses a non-ascii character. The SA tool will fail with an
error, the JVM itself and the SA having calculated different hashes for
such Strings.
bug:
https://bugs.openjdk.java.net/browse/JDK-8028623