RFR: 8271268: Fix Javadoc links for Stream.mapMulti

Mourad Abbay duke at openjdk.org
Fri Sep 22 11:03:12 UTC 2023


On Fri, 22 Sep 2023 08:42:39 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Fix Javadoc links for Stream.mapMulti, Stream.MapMultiInt, Stream.mapMultiToInt, Stream.mapMultiToLong and Stream.mapMultiToDouble.
>
> What is this patch looking to fix? The link is not broken in the first place. This `#mapMulti mapMulti` aims to hide the irrelevant long list of parameter types Javadoc renders, like that seen in [`flatMap`'s](https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/stream/Stream.html#flatMap(java.util.function.Function)) "See Also" section.
> 
> I believe a simple `mapMulti` is better than an overblown `mapMulti(java.util.function.BiConsumer<? super T, ? super java.util.function.Consumer<R>>)`. However, it's always better to unify the rendering in both cases, such as updating the link in `flatMap`'s `@see` to `@see #mapMulti mapMulti`.

@liach here the result of having `@see #mapMulti mapMulti`
<img width="1512" alt="Screenshot 2023-09-22 at 11 57 44 AM" src="https://github.com/openjdk/jdk/assets/41261819/c6a260b9-55dc-4152-9214-632efbfa2f55">

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

PR Comment: https://git.openjdk.org/jdk/pull/15794#issuecomment-1731226664


More information about the core-libs-dev mailing list