Re: RFR: 8263668: Update java.time to use instanceof pattern variable [v5]

2021-04-20 Thread Patrick Concannon
On Wed, 24 Mar 2021 13:57:35 GMT, Roger Riggs  wrote:

> In addition to the other suggestions, java.time could use a round of cleanup 
> to use switch expressions.

Hi Roger, I plan to introduce switch expressions in a follow up issue/PR.

-

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


Re: RFR: 8263668: Update java.time to use instanceof pattern variable [v5]

2021-04-20 Thread Patrick Concannon
> Hi,
> 
> Could someone please review my code for updating the code in the `java.time` 
> package to make use of the `instanceof` pattern variable?
> 
> Kind regards,
> Patrick

Patrick Concannon has updated the pull request with a new target base due to a 
merge or a rebase. The incremental webrev excludes the unrelated changes 
brought in by the merge/rebase. The pull request contains seven additional 
commits since the last revision:

 - Merge remote-tracking branch 'origin/master' into JDK-8263668
 - Merge remote-tracking branch 'origin/master' into JDK-8263668
 - Merge remote-tracking branch 'origin/master' into JDK-8263668
 - Merge remote-tracking branch 'origin/master' into JDK-8263668
 - Merge remote-tracking branch 'origin/master' into JDK-8263668
 - Merge remote-tracking branch 'origin/master' into JDK-8263668
 - 8263668: Update java.time to use instanceof pattern variable

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3170/files
  - new: https://git.openjdk.java.net/jdk/pull/3170/files/6481346a..2dca4a09

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3170&range=04
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3170&range=03-04

  Stats: 3821 lines in 79 files changed: 2021 ins; 1604 del; 196 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3170.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3170/head:pull/3170

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