RFR: 8277087: ZipException: zip END header not found at ZipFile#Source.findEND [v2]
Lance Andersen
lancea at openjdk.java.net
Wed Nov 17 20:02:46 UTC 2021
On Wed, 17 Nov 2021 19:25:33 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
> > Sorry if my point was not clear. I would prefer to have 1 test to exercise a Zip file comment vs have tests in multiple areas. Expanding the existing test in this case keeps the primary coverage in one location and makes it easier for future maintainers.
>
> I have preferred to have separate tests for the separate use cases, other than one big testcases which cover all possible combinations of exceptions and parameters.
This is really a style choice so I guess we can agree to disagree. Either way it would be preferable to have both tests in the same location.
>
> > I am not suggesting to not make your change, I am suggesting to include this change as well.
>
> I understood that you suggest adding this additional change as well, and I pointed out why it is not necessary, if that code will be executed with the long comment it will break the specification.
The additional change simply helps provide a layer of extra protection from corruption creating the zip file.
Anyways, I will approve the changes as they are.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6380
More information about the core-libs-dev
mailing list