Re: RFR: 8277606: String(String) constructor could copy hashIsZero

2021-11-30 Thread Roger Riggs 
On Mon, 22 Nov 2021 22:59:21 GMT, PROgrm_JARvis  wrote:

> This is a trivial fix to make `String(String)` constructor copy the value of 
> `hashIsZero` field.

Marked as reviewed by rriggs (Reviewer).

-

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

Re: RFR: 8277606: String(String) constructor could copy hashIsZero

2021-11-30 Thread PROgrm_JARvis 
On Mon, 22 Nov 2021 22:59:21 GMT, PROgrm_JARvis  wrote:

> This is a trivial fix to make `String(String)` constructor copy the value of 
> `hashIsZero` field.

Hi there, could anyone please sponsor this change if it is still applicable?

Thanks in advance!

-

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

Re: RFR: 8277606: String(String) constructor could copy hashIsZero

2021-11-22 Thread Claes Redestad 
On Mon, 22 Nov 2021 22:59:21 GMT, PROgrm_JARvis  wrote:

> This is a trivial fix to make `String(String)` constructor copy the value of 
> `hashIsZero` field.

LGTM

-

Marked as reviewed by redestad (Reviewer).

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

RFR: 8277606: String(String) constructor could copy hashIsZero

2021-11-22 Thread PROgrm_JARvis 
This is a trivial fix to make `String(String)` constructor copy the value of 
`hashIsZero` field.

-

Commit messages:
 - fix: copy `hashIsZero` in `String#String(String)`

Changes: https://git.openjdk.java.net/jdk/pull/6511/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=6511=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8277606
  Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6511.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6511/head:pull/6511

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