Re: RFR: 8283774: TestZoneOffset::test_immutable should ignore ZoneOffset::rules [v2]

2022-03-29 Thread Claes Redestad 
> - Add capability to ignore fields explicitly when checking for immutability 
> of classes in java.time
> - Use this to avoid a test failure due the new `rules` cache in `ZoneOffset`
> - Remove `TestZoneOffset` from problem list.

Claes Redestad has updated the pull request incrementally with one additional 
commit since the last revision:

  Copyrights

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7989/files
  - new: https://git.openjdk.java.net/jdk/pull/7989/files/44a8cc6b..4da4429b

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=7989=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=7989=00-01

  Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7989.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7989/head:pull/7989

PR: https://git.openjdk.java.net/jdk/pull/7989

Re: RFR: 8283774: TestZoneOffset::test_immutable should ignore ZoneOffset::rules

2022-03-29 Thread Claes Redestad 
On Mon, 28 Mar 2022 10:35:04 GMT, Claes Redestad  wrote:

> - Add capability to ignore fields explicitly when checking for immutability 
> of classes in java.time
> - Use this to avoid a test failure due the new `rules` cache in `ZoneOffset`
> - Remove `TestZoneOffset` from problem list.

Thanks for reviews!

-

PR: https://git.openjdk.java.net/jdk/pull/7989

Re: RFR: 8283774: TestZoneOffset::test_immutable should ignore ZoneOffset::rules

2022-03-28 Thread Naoto Sato 
On Mon, 28 Mar 2022 10:35:04 GMT, Claes Redestad  wrote:

> - Add capability to ignore fields explicitly when checking for immutability 
> of classes in java.time
> - Use this to avoid a test failure due the new `rules` cache in `ZoneOffset`
> - Remove `TestZoneOffset` from problem list.

Marked as reviewed by naoto (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/7989

Re: RFR: 8283774: TestZoneOffset::test_immutable should ignore ZoneOffset::rules

2022-03-28 Thread Roger Riggs 
On Mon, 28 Mar 2022 10:35:04 GMT, Claes Redestad  wrote:

> - Add capability to ignore fields explicitly when checking for immutability 
> of classes in java.time
> - Use this to avoid a test failure due the new `rules` cache in `ZoneOffset`
> - Remove `TestZoneOffset` from problem list.

Update Copyright dates before push.

-

Marked as reviewed by rriggs (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7989

RFR: 8283774: TestZoneOffset::test_immutable should ignore ZoneOffset::rules

2022-03-28 Thread Claes Redestad 
- Add capability to ignore fields explicitly when checking for immutability of 
classes in java.time
- Use this to avoid a test failure due the new `rules` cache in `ZoneOffset`
- Remove `TestZoneOffset` from problem list.

-

Commit messages:
 - TestZoneOffset::test_immutable should ignore ZoneOffset::rules

Changes: https://git.openjdk.java.net/jdk/pull/7989/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=7989=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8283774
  Stats: 12 lines in 3 files changed: 6 ins; 3 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7989.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7989/head:pull/7989

PR: https://git.openjdk.java.net/jdk/pull/7989