RFR: 8337111: Bad HTML checker for generated documentation [v6]

Nizar Benalla nbenalla at openjdk.org
Fri Dec 20 19:34:21 UTC 2024


On Fri, 20 Dec 2024 17:35:28 GMT, Nizar Benalla <nbenalla at openjdk.org> wrote:

>> test/docs/jdk/javadoc/doccheck/doccheckutils/FileProcessor.java line 61:
>> 
>>> 59:             });
>>> 60:         } catch (IOException e) {
>>> 61:             e.printStackTrace();
>> 
>> What's the reason to swallow exceptions here?
>
> I could let the exception propagate, I just didn't think it was interesting.

I think I fixed this in [95f61e6](https://github.com/openjdk/jdk/pull/21879/commits/95f61e688eb19b3696c5b342f34f3ff58d733037) (alongside a couple other things that popped up when I ran a linter on the code).

I know those files exist, so it should really never be thrown.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21879#discussion_r1894315556


More information about the javadoc-dev mailing list