On Tue, 14 Jul 2026 15:30:03 GMT, Kevin Rushforth <[email protected]> wrote:
>> Andy Goryachev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> review comments
>
> modules/jfx.incubator.richtext/src/main/java/jfx/incubator/scene/control/richtext/model/FileListFormatHandler.java
> line 53:
>
>> 51: * Constructor.
>> 52: */
>> 53: public FileListFormatHandler() {
>
> This class is intended to be a singleton. Why does it have a public
> constructor?
singleton to allow reuse of the default implementation, public constructor to
allow subclassing for custom models.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2196#discussion_r3580427711
