RFR: 8321274: Rename ZipEntry.extraAttributes to ZipEntry.externalAttributes [v2]
Eirik Bjørsnøs
eirbjo at openjdk.org
Fri Feb 2 20:22:03 UTC 2024
On Fri, 2 Feb 2024 19:55:51 GMT, Lance Andersen <lancea at openjdk.org> wrote:
> I think the change is OK, any reason to not make it `externalFileAttributes`?
None other than length / verbosity. But since this refers to the _attributes of the external file_, dropping any name element also loses some semantics, introducing a potential for confusion. Perhaps verbosity is the sensible choice here.
If you agree to the above, I can update the PR to rename to the following names instead:
- `ZipFile.externalFileAttributes`
- `JavaUtilZipFileAccess.java.[set|get]ExternalFileAttributes`
- `JarSigner.externalFileAttributesDetected`
- `jarsigner/Main.externalFileAttributesDetected`
- `external.file.attributes.detected` in `Resources.java`
WDYT?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16952#issuecomment-1924638322
More information about the security-dev
mailing list