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

[naoto.sato at oracle.com]

Looks good.

Naoto

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()
>    *                               .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
> 
>