PreferEarlyReturn recipe

[greg-at-moderne]

What's changed?

What's your motivation?

Anything in particular you'd like reviewers to focus on?

Anyone you would like to review specifically?

Have you considered any alternatives or workarounds?

Any additional context

Checklist

• I've added unit tests to cover both positive and negative cases
• I've read and applied the recipe conventions and best practices
• I've used the IntelliJ IDEA auto-formatter on affected files

[github-actions]

Some suggestions could not be made:

• src/main/java/org/openrewrite/staticanalysis/UseDiamondOperator.java
  • lines 276-277
  • lines 290-292
  • lines 309-309

[github-actions]

Suggested change

	public @Nullable J postVisit(@NonNull J tree, ExecutionContext executionContext) {
	J ret = super.postVisit(tree, executionContext);
	if (getCursor().pollMessage("PREFER_EARLY_RETURN") != null) {
	public @Nullable J postVisit(@NonNull J tree, ExecutionContext ctx) {
	J ret = super.postVisit(tree, ctx);
	ret = (J) new UnwrapElseAfterReturn().getVisitor().visit(ret, ctx, getCursor().getParent());