Re: RFR: 8015831: Add lint check for calling overridable methods from a constructor

2023-01-05 Thread Alan Bateman
On Fri, 6 Jan 2023 02:20:53 GMT, Archie L. Cobbs wrote: > This PR adds a new lint warning category `this-escape`. > > It also adds `@SuppressWarnings` annotations as needed to the JDK itself to > allow the JDK to continue to compile with `-Xlint:all`. > > A 'this' escape warning is generated f

Re: RFR: 8015831: Add lint check for calling overridable methods from a constructor

2023-01-05 Thread David Holmes
On Fri, 6 Jan 2023 02:20:53 GMT, Archie L. Cobbs wrote: > This PR adds a new lint warning category `this-escape`. > > It also adds `@SuppressWarnings` annotations as needed to the JDK itself to > allow the JDK to continue to compile with `-Xlint:all`. > > A 'this' escape warning is generated f

RFR: 8015831: Add lint check for calling overridable methods from a constructor

2023-01-05 Thread Archie L . Cobbs
This PR adds a new lint warning category `this-escape`. It also adds `@SuppressWarnings` annotations as needed to the JDK itself to allow the JDK to continue to compile with `-Xlint:all`. A 'this' escape warning is generated for a constructor `A()` in a class `A` when the compiler detects that

Re: RFR: 8298872: Update CheckStatus.java for changes to TLS implementation [v2]

2023-01-05 Thread Matthew Donovan
> Updated the test for TLS 1.2 and removed from ProblemList.txt Matthew Donovan has updated the pull request incrementally with two additional commits since the last revision: - updated copyright to correct year - updated copyright year - Changes: - all: https://git.openjdk.org

RFR: 8298872: Update CheckStatus.java for changes to TLS implementation

2023-01-05 Thread Matthew Donovan
Updated the test for TLS 1.2 and removed from ProblemList.txt - Commit messages: - removed extra whitespace - 8298872: Update CheckStatus.java for changes to TLS implementation Changes: https://git.openjdk.org/jdk/pull/11857/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=1