EmptyBlock adds unnecessary `continue` in empty while loops

[greg-at-moderne]

What version of OpenRewrite are you using?

8.53.1

How are you running OpenRewrite?

Moderne

What is the smallest, simplest way to reproduce the problem?

A few examples of OSS repro:

• https://github.com/apache/orc/blob/c73651d22c51cb69ef82992e0bdca61ec932cb71/java/core/src/test/org/apache/orc/impl/TestZlib.java#L78
• https://github.com/apache/james-mime4j/blob/0ee7cc830ed449d8c44c0e5e2398a79fa2f5ff89/core/src/main/java/org/apache/james/mime4j/stream/MimeEntity.java#L409
• https://github.com/spring-projects/spring-batch/blob/08c4cb16b854b773f974eeb2073a04c56a0eb6ab/spring-batch-integration/src/test/java/org/springframework/batch/integration/chunk/ChunkMessageItemWriterIntegrationTests.java#L115

  while (replies.receive(10L) != null) {
+    continue;
  }

The recipe adds a continue statement to an empty while loop body. It's counterproductive. If anything, it should remove continue if it was the only statement.

[juancaCeb]

I'm working on it. I was able to replicate the behavior based on the tests.