Don't re-wrap nullsafe/nonnull adapters if they are already one (#909)

* Don't re-wrap nullsafe/nonnull adapters if they are already one

nullSafe() is often called defensively, which can lead to a lot of layers of wrapping and checking. This avoids that overhead where possible.

* Add tests

moshi/src/main/java/com/squareup/moshi/JsonAdapter.java

@@ -130,6 +130,9 @@ public abstract class JsonAdapter<T> {
    * nulls.
    */
   @CheckReturnValue public final JsonAdapter<T> nullSafe() {
+    if (this instanceof NullSafeJsonAdapter) {
+      return this;
+    }
     return new NullSafeJsonAdapter<>(this);
   }
 
@@ -141,6 +144,9 @@ public abstract class JsonAdapter<T> {
    * handled elsewhere. This should only be used to fail on explicit nulls.
    */
   @CheckReturnValue public final JsonAdapter<T> nonNull() {
+    if (this instanceof NonNullJsonAdapter) {
+      return this;
+    }
     return new NonNullJsonAdapter<>(this);
   }
 

moshi/src/test/java/com/squareup/moshi/JsonAdapterTest.java

@@ -285,4 +285,14 @@ public final class JsonAdapterTest {
     }.lenient().serializeNulls();
     assertThat(adapter.fromJson("true true")).isEqualTo(true);
   }
+
+  @Test public void nullSafeDoesntDuplicate() {
+    JsonAdapter<Boolean> adapter = new Moshi.Builder().build().adapter(Boolean.class).nullSafe();
+    assertThat(adapter.nullSafe()).isSameAs(adapter);
+  }
+
+  @Test public void nonNullDoesntDuplicate() {
+    JsonAdapter<Boolean> adapter = new Moshi.Builder().build().adapter(Boolean.class).nonNull();
+    assertThat(adapter.nonNull()).isSameAs(adapter);
+  }
 }