On Mon, 22 Aug 2022 17:08:09 GMT, Jayashree Huttanagoudar <d...@openjdk.org> 
wrote:

> So the patch should look like :
> ...

Yes. If you decide to print out `s` no matter if success or not, you can move 
it out of the if block.

> I tried to execute the test case with this change and I don't see anything 
> changed much.

Of course not. It would be useful if the test fails one day. For example, you 
can try to modify the exception message on line 150 of `NTLoginModule` and see 
how the test fails.

-------------

PR: https://git.openjdk.org/jdk/pull/9159

Reply via email to