Rid pointless wrapping with BufferedInputStream from class reading

stsypanov · jhoeller · commit bff1a1932f57 · 2020-04-25T16:27:01.000+02:00
diff --git a/spring-core/src/main/java/org/springframework/core/type/classreading/SimpleMetadataReader.java b/spring-core/src/main/java/org/springframework/core/type/classreading/SimpleMetadataReader.java
@@ -16,7 +16,6 @@
 
 package org.springframework.core.type.classreading;
 
-import java.io.BufferedInputStream;
 import java.io.IOException;
 import java.io.InputStream;
 
@@ -53,7 +52,7 @@ final class SimpleMetadataReader implements MetadataReader {
 	}
 
 	private static ClassReader getClassReader(Resource resource) throws IOException {
-		try (InputStream is = new BufferedInputStream(resource.getInputStream())) {
+		try (InputStream is = resource.getInputStream()) {
 			try {
 				return new ClassReader(is);
 			}

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,6 @@`
`16`	`16`
`17`	`17`	`package org.springframework.core.type.classreading;`
`18`	`18`
`19`		`-import java.io.BufferedInputStream;`
`20`	`19`	`import java.io.IOException;`
`21`	`20`	`import java.io.InputStream;`
`22`	`21`
`@@ -53,7 +52,7 @@ final class SimpleMetadataReader implements MetadataReader {`
`53`	`52`	`}`
`54`	`53`
`55`	`54`	`private static ClassReader getClassReader(Resource resource) throws IOException {`
`56`		`- try (InputStream is = new BufferedInputStream(resource.getInputStream())) {`
	`55`	`+ try (InputStream is = resource.getInputStream()) {`
`57`	`56`	`try {`
`58`	`57`	`return new ClassReader(is);`
`59`	`58`	`}`