On Wed, 4 Mar 2026 06:22:19 GMT, Sruthy Jayan <[email protected]> wrote:

>> Bug Reference : https://bugs.openjdk.org/browse/JDK-8375045
>> 
>> On AIX, the POSIX TZ environment variable is not interpreted as expected 
>> when it includes DST rules.
>> Example: `TZ=CET-1CEST,M3.5.0,M10.5.0`
>> 
>> The fix includes : 
>> The TZ environment variable is processed by first matching the full value 
>> against tzmappings, then retrying with the value truncated at the first 
>> comma if no match is found, and finally defaulting to GMT if no mapping 
>> exists.
>> 
>> This fixes the regression caused by 
>> https://bugs.openjdk.org/browse/JDK-8292899
>> 
>> Signed-off-by: Sruthy Jayan <[email protected]>
>
> Sruthy Jayan has updated the pull request incrementally with two additional 
> commits since the last revision:
> 
>  - Fix alignment issue
>    
>    Signed-off-by: Sruthy Jayan <[email protected]>
>  - Refactor function name for clarity and fix memory allocation issues
>    
>    Signed-off-by: Sruthy Jayan <[email protected]>

LGTM, assuming you have tested the changes on AIX.
(btw you will need to remove the trailing period from the PR title. Skara is 
complaining about it)

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

Marked as reviewed by naoto (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/29183#pullrequestreview-3890838854

Reply via email to