Hi Kumar,

The launcher changes looks fine, but I recommend generally documenting the null-handling protocol in these sort of functions. We generally don't include bug ids in the next of the test, but I see the test file does that in other areas.

Cheers,

-Joe

On 04/24/2012 08:24 AM, Kumar Srinivasan wrote:
Hi,

Could you please review this, actually the launcher is a 2 line change,
but when I was testing/experimenting with  this, it was hard to inspect
the .jtr file for test failure, the test framework is cleanup to help the
reader quickly identify the failing test cases.

http://cr.openjdk.java.net/~ksrini/7151434

Thanks
Kumar



Reply via email to