RFR: 8211917: (zipfs) Creating or updating a JAR file system should put the MANIFEST.MF at the start

[Lance Andersen]

Hi Jaikiran

> On Mar 4, 2020, at 9:54 AM, Jaikiran Pai <jai.forums2013 at gmail.com> wrote:
> 
> Hello Lance,
> 
> On 28/02/20 2:41 am, Lance Andersen wrote:
>> Hi Christoph,
>> 
>> I have cleaned up, re-vamped and added more coverage to the test.  As part of this I started to lay the foundation for removing some of the duplicate code as part of continued clean up and enhanced coverage going forward
>> 
>> The revised webrev can be found at: http://cr.openjdk.java.net/~lancea/8211917/webrev.00/index.html <http://cr.openjdk.java.net/~lancea/8211917/webrev.00/index.html>This all looks good to me (of course, I'm not an official reviewer). The new base testcase for zipfs related testing is definitely going to help in future fixes/enhancements.
> 
> I just had one question in there:
> 
> +++ new/test/jdk/jdk/nio/zipfs/testng/util/ZipFsBaseTest.java	
> ...
> +                {Map.of("create", "true", "noCompression", "true"),
> +                        ZipEntry.STORED},
> +                {Map.of("create", "true", "noCompression", "false"),
> +                        ZipEntry.DEFLATED}
> 
> From what I had read in the javadoc of the private method getDefaultCompressionMethod in jdk.nio.zipfs.ZipFileSystem,
> the "noCompression" property is only there for backward compatibility reasons and the new way
> of configuring this semantic is the use of "compressionMethod" property (with value of either STORED
> or DEFLATED). Is the use of "noCompression" in this base test class intentional or is it just a personal preference?
> I'm fine either way, but wanted to know if I should stick to this form in any future test cases.

Yes when I added formal support for some of the existing Zip FS properties, we decided to  add compressionMethod as the formal property in the event  additional compression methods were added in the future.  So noCompression will always exist and this DataProvider is used in several tests  so I left it as is.  At some point I might change it, but really does not matter as there are other tests which specifically test the compressionMethod property.

Best
Lance
> 
> -Jaikiran
> 
> 

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <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>



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/nio-dev/attachments/20200304/e3df6faa/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: oracle_sig_logo.gif
Type: image/gif
Size: 658 bytes
Desc: not available
URL: <https://mail.openjdk.java.net/pipermail/nio-dev/attachments/20200304/e3df6faa/oracle_sig_logo.gif>