Description Christoph Laeubrich 2021-01-26 00:03:57 EST

HyperlinkManager currently enforces that no hyperlinks -> IHyperlinkDetector.detectHyperlinks return null

but this enforcement seems of no value and complicates client code.

The code

> if (hyperlinks == null)
>	continue;
> Assert.isLegal(hyperlinks.length > 0);

can simply be rewritten to 

> if (hyperlinks == null || hyperlinks.length == 0)
>	continue;

This would allow to simply return an array e.g. from a stream operation and is semantically equivalent.