> This is a REDO of the Fix that was incompletely implemented earlier: > [8285838: Fix for TZ environment variable DST > rules](https://github.com/openjdk/jdk/pull/8660) > > Offset calculation now accounts all the way upto year in order to avoid > cross-day miscalculations as well as to calculate always in the correct > direction for offset. In situations where there may be multiple days, the > excess days of offset will be shaved off by applying mod to `seconds_per_day` > , which will remove the excessive days that might be included in the offset > calculation for special scenarios like a leap year / February months and > variances between 30 and 31 days. > > I have tested this solution with the cases where this fix had failed last > time as well, and confirmed it works: > _(where 7200 represents 7200 seconds -> +2 hour offset)_ > Sample output: > 
Gaurav Chaudhari has updated the pull request incrementally with one additional commit since the last revision: 8288377: Simplified TZ offset calc and consolidate MACOS impl. ------------- Changes: - all: https://git.openjdk.org/jdk/pull/9312/files - new: https://git.openjdk.org/jdk/pull/9312/files/f9230150..7e462d95 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=9312&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9312&range=00-01 Stats: 81 lines in 1 file changed: 1 ins; 75 del; 5 mod Patch: https://git.openjdk.org/jdk/pull/9312.diff Fetch: git fetch https://git.openjdk.org/jdk pull/9312/head:pull/9312 PR: https://git.openjdk.org/jdk/pull/9312