On Fri, 22 Sep 2023 08:42:39 GMT, Chen Liang <li...@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
