EmptyBlock adds unnecessary continue in empty while loops #570
Description
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.