Breaking Change: Remove unused overload of AbstractMessageLite.addAll that worked on a Collection instead of a List.

protobuf-github-bot · copybara-github · commit 3b9bd6db71d2 · 2024-01-12T09:58:06.000-08:00
PiperOrigin-RevId: 597869849
diff --git a/java/core/src/main/java/com/google/protobuf/AbstractMessageLite.java b/java/core/src/main/java/com/google/protobuf/AbstractMessageLite.java
@@ -113,12 +113,6 @@ protected static void checkByteStringIsUtf8(ByteString byteString)
     }
   }
 
-  // For binary compatibility
-  @Deprecated
-  protected static <T> void addAll(final Iterable<T> values, final Collection<? super T> list) {
-    Builder.addAll(values, (List) list);
-  }
-
   protected static <T> void addAll(final Iterable<T> values, final List<? super T> list) {
     Builder.addAll(values, list);
   }

Original file line number	Diff line number	Diff line change
`@@ -113,12 +113,6 @@ protected static void checkByteStringIsUtf8(ByteString byteString)`
`113`	`113`	`}`
`114`	`114`	`}`
`115`	`115`
`116`		`- // For binary compatibility`
`117`		`- @Deprecated`
`118`		`- protected static <T> void addAll(final Iterable<T> values, final Collection<? super T> list) {`
`119`		`- Builder.addAll(values, (List) list);`
`120`		`- }`
`121`		`-`
`122`	`116`	`protected static <T> void addAll(final Iterable<T> values, final List<? super T> list) {`
`123`	`117`	`Builder.addAll(values, list);`
`124`	`118`	`}`