RFR: JDK-8272984: javadoc support for reproducible builds [v2]
Jonathan Gibbons
jjg at openjdk.java.net
Mon Jan 24 19:11:15 UTC 2022
On Mon, 24 Jan 2022 15:32:46 GMT, Hannes Wallnöfer <hannesw at openjdk.org> wrote:
>> Jonathan Gibbons has updated the pull request incrementally with one additional commit since the last revision:
>>
>> change allowable date range to within 10 years of the time of use
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/BaseOptions.java line 359:
>
>> 357: private static final ZonedDateTime now = ZonedDateTime.now();
>> 358: static final ZonedDateTime DATE_MIN = now.minusYears(10);
>> 359: static final ZonedDateTime DATE_MAX = now.plusYears(10);
>
> What's the purpose of limiting valid dates this way?
Partly because the code on which I based this on (in the jar tool) had a range check, and partly because I think it makes sense to do a sanity check on the provided value. The range in the jar tool was 1980-2099 which (see elsewhere) is an artifact of the zip spec, so I changed the range to a "reasonably broad" range of reasonable values.
Given that (for now) we convert internally to a `Date`, we can't accept years before Unix epoch, 1970, anyway, so that's another reason for the check.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7171
More information about the javadoc-dev
mailing list