RFR: 8356645: Javac should utilize new ZIP file system read-only access mode [v3]

[David Beaumont]

On Wed, 18 Jun 2025 20:20:19 GMT, David Beaumont <duke at openjdk.org> wrote:

>> We typically don't use `assert` in javac anymore, as it can be disabled. Either the condition is important, then we typically use `com.sun.tools.javac.util.Assert.check`, or we don't do the check. I think using `Assert` here would be appropriate.
>
> Hmmm .... question:
> 
> The other place using FSInfo static method was the cySym verifier. And this is the cySym code. So it feels like it would be good to be consistent between both these cases. So is this a place for FSInfo as well, or not?
> 
> The advantage of FSInfo is that you can get the JAR provider (which we know is the ZIP provider too) and not have to worry about mismatches between given file and "discovered" file system type. You just only use the JAR provider and you're done.

I removed the FSInfo use in the test, just not worth the dependency.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25639#discussion_r2156481223