RFR 8211919: ZipDirectoryStream should provide a stream of paths that are relative to the directory
lance.andersen at oracle.com
Tue Jan 15 19:00:30 UTC 2019
> On Jan 15, 2019, at 3:10 AM, Langer, Christoph <christoph.langer at sap.com> wrote:
> Hi Lance,
>> Thank you for the feedback, please see below
> Welcome, thanks for addressing my findings
>> line 97: spelling: “filter”
>> fixed though was “DirectoryStream.Filter” which I changed to
>> DirectoryStream filter
> Hm, not quite yet... What I mean is still in line 96:
> * Iterator correctly indicates whether the filer has been matched
> -> I think you mean filter instead of filer here.
>> line 309: you could use an import java.util.zip.ZipException?
>> done, still getting used to Intellij vs Netbeans which I like better for import
> I'm currently using Eclipse though I should try IntelliJ, too. As for imports, I would also think the org.testng.* imports should go after the java.* imports. But not sure if there's some official style guide for that.
Well the ordering was done via Intellij so I am going to leave that for now.
> Overall, looks good to me now. (The latest webrev, that addresses Alan's point, too)
> Best regards
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
More information about the core-libs-dev