Looks good.
Cheers,
Mikael
On 2014-03-03 23:24, Staffan Larsen wrote:
Missed one more place in CatchPatternTest.sh:
http://cr.openjdk.java.net/~sla/8036132/webrev.02/
/Staffan
On 4 mar 2014, at 08:09, Staffan Larsen <staffan.lar...@oracle.com> wrote:
Thanks for noticing! The same mistake was in all the inlined Java code.
new webrev: http://cr.openjdk.java.net/~sla/8036132/webrev.01/
/Staffan
On 3 mar 2014, at 21:04, Mikael Vidstedt <mikael.vidst...@oracle.com> wrote:
I randomly chose a file and ended up looking at GetLocalVariables4Test.sh where
the indentation of all the lines in main() appear to be off - the tab should
have been translated to 8 spaces in this case, not 4?
Cheers,
Mikael
On 2014-03-03 05:30, Staffan Larsen wrote:
Just changing a lot of tabs into a lot more spaces. Easiest to view in the
patch file.
webrev: http://cr.openjdk.java.net/~sla/8036132/webrev.00/
bug: https://bugs.openjdk.java.net/browse/JDK-8036132
Thanks,
/Staffan
