On Wed, 6 Oct 2021 16:04:54 GMT, Bradford Wetmore <wetm...@openjdk.org> wrote:
>> 8274809: Update java.base classes to use try-with-resources > > src/java.base/share/classes/sun/net/NetProperties.java line 71: > >> 69: f = new File(f, "net.properties"); >> 70: fname = f.getCanonicalPath(); >> 71: try (FileInputStream fis = new FileInputStream(fname)) { > > Why did the BufferedInputStream get pulled out here? I decieded to remove it because `Properties.load` already buffers inputstream. So usage of BufferedInputStream seems redundant. https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/java/util/Properties.java#L471 ------------- PR: https://git.openjdk.java.net/jdk/pull/5818