From 12ead865aa40fb97a0d7b3e2565d4f20acbba4fd Mon Sep 17 00:00:00 2001
From: Knut Wannheden <knut@moderne.io>
Date: Wed, 24 Apr 2024 14:06:24 +0200
Subject: [PATCH] Add generic type variable bound to `JsonRightPadded<T>`
 (#4147)

Changes `<T>` to `<T extends Json>` as that is the only way it is currently being used.

Note: In the `J` model the padding constructs are also used to wrap non-`Tree` elements, but not in `Json`.
---
 .../src/main/java/org/openrewrite/json/JsonVisitor.java     | 2 +-
 .../java/org/openrewrite/json/tree/JsonRightPadded.java     | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/rewrite-json/src/main/java/org/openrewrite/json/JsonVisitor.java b/rewrite-json/src/main/java/org/openrewrite/json/JsonVisitor.java
index 51f66622d3b..623b0093acb 100755
--- a/rewrite-json/src/main/java/org/openrewrite/json/JsonVisitor.java
+++ b/rewrite-json/src/main/java/org/openrewrite/json/JsonVisitor.java
@@ -96,7 +96,7 @@ public Space visitSpace(Space space, P p) {
     }
 
     @SuppressWarnings("ConstantConditions")
-    public <T> JsonRightPadded<T> visitRightPadded(@Nullable JsonRightPadded<T> right, P p) {
+    public <T extends Json> JsonRightPadded<T> visitRightPadded(@Nullable JsonRightPadded<T> right, P p) {
         if (right == null) {
             //noinspection ConstantConditions
             return null;
diff --git a/rewrite-json/src/main/java/org/openrewrite/json/tree/JsonRightPadded.java b/rewrite-json/src/main/java/org/openrewrite/json/tree/JsonRightPadded.java
index 7d76a14e97a..1fd89a779a0 100644
--- a/rewrite-json/src/main/java/org/openrewrite/json/tree/JsonRightPadded.java
+++ b/rewrite-json/src/main/java/org/openrewrite/json/tree/JsonRightPadded.java
@@ -38,7 +38,7 @@
 @FieldDefaults(makeFinal = true, level = AccessLevel.PRIVATE)
 @EqualsAndHashCode(callSuper = false, onlyExplicitlyIncluded = true)
 @Data
-public class JsonRightPadded<T> {
+public class JsonRightPadded<T extends Json> {
     @With
     T element;
 
@@ -52,7 +52,7 @@ public JsonRightPadded<T> map(UnaryOperator<T> map) {
         return withElement(map.apply(element));
     }
 
-    public static <T> List<T> getElements(List<JsonRightPadded<T>> ls) {
+    public static <T extends Json> List<T> getElements(List<JsonRightPadded<T>> ls) {
         List<T> list = new ArrayList<>();
         for (JsonRightPadded<T> l : ls) {
             T elem = l.getElement();
@@ -92,7 +92,7 @@ public static <J2 extends Json> List<JsonRightPadded<J2>> withElements(List<Json
         return after;
     }
 
-    public static <T> JsonRightPadded<T> build(T element) {
+    public static <T extends Json> JsonRightPadded<T> build(T element) {
         return new JsonRightPadded<>(element, Space.EMPTY, Markers.EMPTY);
     }