[jdk17] RFR: JDK-8268972: Add default impl for recent new Reporter.print method

Jonathan Gibbons jjg at openjdk.java.net
Thu Jun 17 18:09:27 UTC 2021


On Thu, 17 Jun 2021 17:50:31 GMT, Pavel Rappo <prappo at openjdk.org> wrote:

>> Please review a simple fix to add a default implementation for a recent new method on the Reporter interface. (It was an oversight that the default implementation was not provided in the original work.)
>
> src/jdk.javadoc/share/classes/jdk/javadoc/doclet/Reporter.java line 101:
> 
>> 99:      * The implementation provided by the {@code javadoc} tool to
>> 100:      * {@link Doclet#init(Locale, Reporter) initialize} a doclet
>> 101:      * overrides this implementation.
> 
> Overrides how?

...per the documented spec.   I tried writing `The implementation .... always generates an appropriate diagnostic` but that's not true if an exception is thrown, and writing `The implementation ... does not throw UOE` seems weird.

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

PR: https://git.openjdk.java.net/jdk17/pull/92


More information about the javadoc-dev mailing list