RFR: 8294226: Document missing UnsupportedTemporalTypeException

2025-06-16 Thread Gautham Krishnan
Some methods in the java.time.chrono interfaces—ChronoLocalDate, ChronoLocalDateTime, and ChronoZonedDateTime—override methods from the java.time.temporal.Temporal interface that are documented to throw UnsupportedTemporalTypeException when given unsupported fields or units. These overridden me

Integrated: 8334742: Change java.time month/day field types to 'byte'

2025-05-28 Thread Gautham Krishnan
On Wed, 30 Apr 2025 20:30:59 GMT, Gautham Krishnan wrote: > In the following classes, month and day values are stored in fields of type > 'int' or 'short'. The range of allowed values is small enough that the type > can be 'byte' instead. >

Re: RFR: 8334742: Change java.time month/day field types to 'byte' [v2]

2025-04-30 Thread Gautham Krishnan
On Wed, 30 Apr 2025 21:11:51 GMT, Roger Riggs wrote: >> Gautham Krishnan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Updating copyright header > > Its recommended to set the assignee of a issue before s

Re: RFR: 8334742: Change java.time month/day field types to 'byte' [v2]

2025-04-30 Thread Gautham Krishnan
On Wed, 30 Apr 2025 20:50:34 GMT, Naoto Sato wrote: >> Gautham Krishnan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Updating copyright header > > Since they are serialized fields (except Hijrah ones), I

Re: RFR: 8334742: Change java.time month/day field types to 'byte' [v2]

2025-04-30 Thread Gautham Krishnan
lDate > java.time.chono.HijrahDate > > Refactoring the type will give the JVM a little more layout flexibility, and > will be especially useful when these classes become value classes. Gautham Krishnan has updated the pull request incrementally with one additional commit since the last

RFR: 8334742: Change java.time month/day field types to 'byte'

2025-04-30 Thread Gautham Krishnan
In the following classes, month and day values are stored in fields of type 'int' or 'short'. The range of allowed values is small enough that the type can be 'byte' instead. java.time.YearMonth java.time.MonthDay java.time.LocalDate java.time.chono.HijrahDate Refactoring the type will give the

Integrated: 8342886: Update MET timezone in TimeZoneNames files

2025-04-30 Thread Gautham Krishnan
On Fri, 25 Apr 2025 09:57:38 GMT, Gautham Krishnan wrote: > MET timezone entry in TimeZoneNames.java and TimeZoneNames_*.java needs to be > updated as MET is alias to Europe/Brussels as per 2024b tzdata changes. > > Also Bug4848242.java needs to be removed as the test expects all

Re: RFR: 8342886: Update MET timezone in TimeZoneNames files [v5]

2025-04-29 Thread Gautham Krishnan
> MET timezone entry in TimeZoneNames.java and TimeZoneNames_*.java needs to be > updated as MET is alias to Europe/Brussels as per 2024b tzdata changes. > > Also Bug4848242.java needs to be removed as the test expects all euro locale > time zones should have the same short

Re: RFR: 8342886: Update MET timezone in TimeZoneNames files [v3]

2025-04-29 Thread Gautham Krishnan
On Tue, 29 Apr 2025 17:36:26 GMT, Naoto Sato wrote: >> Gautham Krishnan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Updating comment in Bug4848242.java > > You'll need to append "8342886&quo

Re: RFR: 8342886: Update MET timezone in TimeZoneNames files [v4]

2025-04-29 Thread Gautham Krishnan
> MET timezone entry in TimeZoneNames.java and TimeZoneNames_*.java needs to be > updated as MET is alias to Europe/Brussels as per 2024b tzdata changes. > > Also Bug4848242.java needs to be removed as the test expects all euro locale > time zones should have the same short

Re: RFR: 8342886: Update MET timezone in TimeZoneNames files

2025-04-29 Thread Gautham Krishnan
On Fri, 25 Apr 2025 18:12:32 GMT, Naoto Sato wrote: >> MET timezone entry in TimeZoneNames.java and TimeZoneNames_*.java needs to >> be updated as MET is alias to Europe/Brussels as per 2024b tzdata changes. >> >> Also Bug4848242.java needs to be removed as the test expects all euro locale >>

Re: RFR: 8342886: Update MET timezone in TimeZoneNames files [v3]

2025-04-29 Thread Gautham Krishnan
> MET timezone entry in TimeZoneNames.java and TimeZoneNames_*.java needs to be > updated as MET is alias to Europe/Brussels as per 2024b tzdata changes. > > Also Bug4848242.java needs to be removed as the test expects all euro locale > time zones should have the same short

Re: RFR: 8342886: Update MET timezone in TimeZoneNames files

2025-04-29 Thread Gautham Krishnan
On Fri, 25 Apr 2025 18:12:32 GMT, Naoto Sato wrote: >> MET timezone entry in TimeZoneNames.java and TimeZoneNames_*.java needs to >> be updated as MET is alias to Europe/Brussels as per 2024b tzdata changes. >> >> Also Bug4848242.java needs to be removed as the test expects all euro locale >>

Re: RFR: 8342886: Update MET timezone in TimeZoneNames files [v2]

2025-04-29 Thread Gautham Krishnan
> MET timezone entry in TimeZoneNames.java and TimeZoneNames_*.java needs to be > updated as MET is alias to Europe/Brussels as per 2024b tzdata changes. > > Also Bug4848242.java needs to be removed as the test expects all euro locale > time zones should have the same short

RFR: 8342886: Update MET timezone in TimeZoneNames files

2025-04-25 Thread Gautham Krishnan
MET timezone entry in TimeZoneNames.java and TimeZoneNames_*.java needs to be updated as MET is alias to Europe/Brussels as per 2024b tzdata changes. Also Bug4848242.java needs to be removed as the test expects all euro locale time zones should have the same short names. - Commit m