RFR: 8302516: Do some cleanup of nsk/share/jdi/EventHandler.java

[Chris Plummer]

On Wed, 1 Mar 2023 20:57:24 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

>> How would you recommend doing it? Is there a style guide that covers this (I can never find it when I need it).
>
> Just as it was before. 
> if (event instanceof ExceptionEvent &&
>      defaultExceptionRequest != null &&
>       defaultExceptionRequest.equals(event.request())) {

I find that much less readable since the 2nd and 3rd lines of the `if` expression are indented the same as the first statement that follows. Previously they had added a blank line to resolve this, but I don't like that either. You shouldn't have a blank line at the start of a compound statement block.

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

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