RFR 8238919: (doc) Broken code snippet in the java.util.stream package documentation

[Mandy Chung]

On 2/12/20 10:52 AM, Roger Riggs wrote:
> Please review a trivial doc fix to remove html markup that should not 
> be inside {@code}.
> And a couple of spacing and punctuation fixes.
>
> diff --git 
> a/src/java.base/share/classes/java/util/stream/package-info.java 
> b/src/java.base/share/classes/java/util/stream/package-info.java
> --- a/src/java.base/share/classes/java/util/stream/package-info.java
> +++ b/src/java.base/share/classes/java/util/stream/package-info.java
> @@ -43,7 +43,7 @@
>   * <p>The key abstraction introduced in this package is 
> <em>stream</em>.  The
>   * classes {@link java.util.stream.Stream}, {@link 
> java.util.stream.IntStream},
>   * {@link java.util.stream.LongStream}, and {@link 
> java.util.stream.DoubleStream}
> - * are streams over objects and the primitive {@code int}, {@code 
> long} and
> + * are streams over objects and the primitive {@code int}, {@code 
> long}, and
>   * {@code double} types.  Streams differ from collections in several 
> ways:
>   *
>   * <ul>
> @@ -176,7 +176,7 @@
>   * do:
>   *
>   * <pre>{@code
> - *     int sumOfWeights = widgets.<b>parallelStream()</b>
> + *     int sumOfWeights = widgets.parallelStream()

It would be good to keep "parallelStream()" bold as the original intent.

Would breaking it into two code block work?  If not, this patch is okay.

Mandy

> *                               .filter(b -> b.getColor() == RED)
>   *                               .mapToInt(b -> b.getWeight())
>   *                               .sum();
> @@ -242,7 +242,7 @@
>   *     String s = sl.collect(joining(" "));
>   * }</pre>
>   *
> - * First a list is created consisting of two strings: "one"; and 
> "two". Then a
> + * First a list is created consisting of two strings: "one" and 
> "two". Then a
>   * stream is created from that list. Next the list is modified by 
> adding a third
>   * string: "three". Finally the elements of the stream are collected 
> and joined
>   * together. Since the list was modified before the terminal {@code 
> collect}
> @@ -344,7 +344,7 @@
>   * parallelization:
>   *
>   * <pre>{@code
> - *     List<String>results =
> + *     List<String> results =
>   *         stream.filter(s -> pattern.matcher(s).matches())
>   *               .collect(Collectors.toList());  // No side-effects!
>   * }</pre>
>
> Issue:
>   https://bugs.openjdk.java.net/browse/JDK-8238919
>
> Thanks, Roger
>
>