RFR: JDK-8231587: Memory leak in WorkArounds.serializedForms
Hannes Wallnöfer
hannes.wallnoefer at oracle.com
Thu Oct 17 15:44:02 UTC 2019
Looks good.
Hannes
> Am 14.10.2019 um 21:55 schrieb Jonathan Gibbons <jonathan.gibbons at oracle.com>:
>
> Please review a simple fix, and some code cleanup, for a memory leak in javadoc.
>
> The leak was first noted by Florian in the mail thread here:
> https://mail.openjdk.java.net/pipermail/javadoc-dev/2019-September/001142.html
>
> The primary part of the fix is (just) the removal of the unnecessary 'static' modifier.
>
> The remainder of the fix is to introduce a local method to access a NewSerializedForm
> object, creating it if needed, and to remove an unnecessary "utils" parameter from the
> internal methods. There's more cleanup that could be done, but I'm trying to keep this
> patch simple for now. (For example, rename NewSerializedForm to SerializedForm,
> and expose it more directly, instead of wrapping it.)
>
> -- Jon
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8231587
> Webrev: http://cr.openjdk.java.net/~jjg/8231587/webrev.00/index.html
>
More information about the javadoc-dev
mailing list