Hi Roger, Looks good overall; total nitpicks here:
FileInputStream (similar story in FileOutputStream) 48-49: “explicitly" is used twice 53: could probably drop “explicitly” here altogether. The “Shd” in a couple of test names is kind of annoying; perhaps s/Shd/Should/ ? Copyright dates are not update in a few places. Thanks, Brian On Sep 29, 2017, at 10:17 AM, Roger Riggs <roger.ri...@oracle.com> wrote: > Comments are appreciated on […] the implementation [1]. > > [1] webrev: http://cr.openjdk.java.net/~rriggs/webrev-fis-cleanup-8080225/
