Deprecate redundant extractBytes(:ByteBuffer) method in ByteUtils.

Refactors extractBytes(:ByteBuffer) to call getBytes(:ByteBuffer), thereby avoid the NullPointerException by throwing the more appropriate IllegalArgumentException with a descriptive message instead.

Author: jxblum

src/main/java/org/springframework/data/redis/util/ByteUtils.java

@@ -31,6 +31,7 @@
  * @author Christoph Strobl
  * @author Mark Paluch
  * @author Guy Korland
+ * @author John Blum
  * @since 1.7
  */
 public final class ByteUtils {
@@ -251,13 +252,10 @@ public static ByteBuffer getByteBuffer(String theString, Charset charset) {
 	 * @param buffer must not be {@literal null}.
 	 * @return the extracted bytes.
 	 * @since 2.1
+	 * @deprecated Since 3.2.0. Use {@link #getBytes(ByteBuffer)} instead.
 	 */
+	@Deprecated(since = "3.2.0")
 	public static byte[] extractBytes(ByteBuffer buffer) {
-
-		ByteBuffer duplicate = buffer.duplicate();
-		byte[] bytes = new byte[duplicate.remaining()];
-		duplicate.get(bytes);
-
-		return bytes;
+		return getBytes(buffer);
 	}
 }