On Sat, 7 Jan 2023 18:03:04 GMT, Alan Bateman <al...@openjdk.org> wrote:

> I don't think the implementation notes should be included as part of the 
> adding the lint warning as I think it creates an attractive nuisance.

I agree with you - not only for that reason, but also because as others have 
pointed out the addition of the warning is really a separate task from the 
evaluation and triage of any existing leaks. Moreover, this latter task is 
really multiple separate tasks that belong to the various module/functional 
groups.

I'll remove all the Javadoc `@implNote`'s from the patch, and 
`src/java.base/share/classes/java/lang/doc-files/ThisEscape.html` as well - it 
will still be available in the git history if someone comes up with a better 
home for it.

This will also further reduce the number of labels and therefore required 
reviewers.

Thanks.

-------------

PR: https://git.openjdk.org/jdk/pull/11874

Reply via email to