Re: RFR: 8274003: ProcessHandleImpl.Info toString has an if check which is always true

2021-09-21 Thread Iris Clark 
On Tue, 21 Sep 2021 18:14:17 GMT, Roger Riggs  wrote:

> Correct the check if any field has been appended to the StringBuilder in 
> ProcessHandleImpl.Info.toString().

Marked as reviewed by iris (Reviewer).

-

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

Re: RFR: 8274003: ProcessHandleImpl.Info toString has an if check which is always true

2021-09-21 Thread Naoto Sato 
On Tue, 21 Sep 2021 18:14:17 GMT, Roger Riggs  wrote:

> Correct the check if any field has been appended to the StringBuilder in 
> ProcessHandleImpl.Info.toString().

Marked as reviewed by naoto (Reviewer).

-

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

Re: RFR: 8274003: ProcessHandleImpl.Info toString has an if check which is always true

2021-09-21 Thread Brian Burkhalter 
On Tue, 21 Sep 2021 18:14:17 GMT, Roger Riggs  wrote:

> Correct the check if any field has been appended to the StringBuilder in 
> ProcessHandleImpl.Info.toString().

Marked as reviewed by bpb (Reviewer).

-

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

RFR: 8274003: ProcessHandleImpl.Info toString has an if check which is always true

2021-09-21 Thread Roger Riggs 
Correct the check if any field has been appended to the StringBuilder in 
ProcessHandleImpl.Info.toString().

-

Commit messages:
 - 8274003: ProcessHandleImpl.Info toString has an if check which is always true

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

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