Hi, Please review these changes to j.u.s. StreamSupport to make it final and non-instantiable, i also tacked on some JavaDoc broken link fixes made by Henry.
Paul. # HG changeset patch # User psandoz # Date 1369825082 -7200 # Node ID bfdc1ed75460c9e6869827cf9acabd4b1a5e9d29 # Parent 6be3ce51e61dbdab6766c74223c076a7b3472be6 8014731: j.u.stream.StreamSupport class has default constructor generated Summary: This change set also fixes broken links Reviewed-by: Contributed-by: Paul Sandoz <[email protected]>, Henry Jen <[email protected]> diff -r 6be3ce51e61d -r bfdc1ed75460 src/share/classes/java/util/stream/StreamSupport.java --- a/src/share/classes/java/util/stream/StreamSupport.java Tue May 28 15:22:30 2013 +0200 +++ b/src/share/classes/java/util/stream/StreamSupport.java Wed May 29 12:58:02 2013 +0200 @@ -41,7 +41,11 @@ * * @since 1.8 */ -public class StreamSupport { +public final class StreamSupport { + + // Suppresses default constructor, ensuring non-instantiability. + private StreamSupport() {} + /** * Creates a new sequential {@code Stream} from a {@code Spliterator}. * @@ -50,7 +54,7 @@ * * <p>It is strongly recommended the spliterator report a characteristic of * {@code IMMUTABLE} or {@code CONCURRENT}, or be - * <a href="Spliterator.html#binding">late-binding</a>. Otherwise, + * <a href="../Spliterator.html#binding">late-binding</a>. Otherwise, * {@link #stream(Supplier, int)} should be used to * reduce the scope of potential interference with the source. See * <a href="package-summary.html#Non-Interference">Non-Interference</a> for @@ -75,7 +79,7 @@ * * <p>It is strongly recommended the spliterator report a characteristic of * {@code IMMUTABLE} or {@code CONCURRENT}, or be - * <a href="Spliterator.html#binding">late-binding</a>. Otherwise, + * <a href="../Spliterator.html#binding">late-binding</a>. Otherwise, * {@link #stream(Supplier, int)} should be used to * reduce the scope of potential interference with the source. See * <a href="package-summary.html#Non-Interference">Non-Interference</a> for @@ -102,7 +106,7 @@ * * <p>For spliterators that report a characteristic of {@code IMMUTABLE} * or {@code CONCURRENT}, or that are - * <a href="Spliterator.html#binding">late-binding</a>, it is likely + * <a href="../Spliterator.html#binding">late-binding</a>, it is likely * more efficient to use {@link #stream(java.util.Spliterator)} instead. * The use of a {@code Supplier} in this form provides a level of * indirection that reduces the scope of potential interference with the @@ -138,7 +142,7 @@ * * <p>For spliterators that report a characteristic of {@code IMMUTABLE} * or {@code CONCURRENT}, or that are - * <a href="Spliterator.html#binding">late-binding</a>, it is likely + * <a href="../Spliterator.html#binding">late-binding</a>, it is likely * more efficient to use {@link #stream(Spliterator)} instead. * The use of a {@code Supplier} in this form provides a level of * indirection that reduces the scope of potential interference with the @@ -172,7 +176,7 @@ * * <p>It is strongly recommended the spliterator report a characteristic of * {@code IMMUTABLE} or {@code CONCURRENT}, or be - * <a href="Spliterator.html#binding">late-binding</a>. Otherwise, + * <a href="../Spliterator.html#binding">late-binding</a>. Otherwise, * {@link #stream(Supplier, int)}} should be used to * reduce the scope of potential interference with the source. See * <a href="package-summary.html#Non-Interference">Non-Interference</a> for @@ -195,7 +199,7 @@ * * <p>It is strongly recommended the spliterator report a characteristic of * {@code IMMUTABLE} or {@code CONCURRENT}, or be - * <a href="Spliterator.html#binding">late-binding</a>. Otherwise, + * <a href="../Spliterator.html#binding">late-binding</a>. Otherwise, * {@link #stream(Supplier, int)}} should be used to * reduce the scope of potential interference with the source. See * <a href="package-summary.html#Non-Interference">Non-Interference</a> for @@ -220,7 +224,7 @@ * * <p>For spliterators that report a characteristic of {@code IMMUTABLE} * or {@code CONCURRENT}, or that are - * <a href="Spliterator.html#binding">late-binding</a>, it is likely + * <a href="../Spliterator.html#binding">late-binding</a>, it is likely * more efficient to use {@link #intStream(Spliterator.OfInt)} instead. * The use of a {@code Supplier} in this form provides a level of * indirection that reduces the scope of potential interference with the @@ -254,7 +258,7 @@ * * <p>For spliterators that report a characteristic of {@code IMMUTABLE} * or {@code CONCURRENT}, or that are - * <a href="Spliterator.html#binding">late-binding</a>, it is likely + * <a href="../Spliterator.html#binding">late-binding</a>, it is likely * more efficient to use {@link #intStream(Spliterator.OfInt)} instead. * The use of a {@code Supplier} in this form provides a level of * indirection that reduces the scope of potential interference with the @@ -286,7 +290,7 @@ * * <p>It is strongly recommended the spliterator report a characteristic of * {@code IMMUTABLE} or {@code CONCURRENT}, or be - * <a href="Spliterator.html#binding">late-binding</a>. Otherwise, + * <a href="../Spliterator.html#binding">late-binding</a>. Otherwise, * {@link #stream(Supplier, int)} should be used to * reduce the scope of potential interference with the source. See * <a href="package-summary.html#Non-Interference">Non-Interference</a> for @@ -310,7 +314,7 @@ * * <p>It is strongly recommended the spliterator report a characteristic of * {@code IMMUTABLE} or {@code CONCURRENT}, or be - * <a href="Spliterator.html#binding">late-binding</a>. Otherwise, + * <a href="../Spliterator.html#binding">late-binding</a>. Otherwise, * {@link #stream(Supplier, int)} should be used to * reduce the scope of potential interference with the source. See * <a href="package-summary.html#Non-Interference">Non-Interference</a> for @@ -335,7 +339,7 @@ * * <p>For spliterators that report a characteristic of {@code IMMUTABLE} * or {@code CONCURRENT}, or that are - * <a href="Spliterator.html#binding">late-binding</a>, it is likely + * <a href="../Spliterator.html#binding">late-binding</a>, it is likely * more efficient to use {@link #longStream(Spliterator.OfLong)} instead. * The use of a {@code Supplier} in this form provides a level of * indirection that reduces the scope of potential interference with the @@ -369,7 +373,7 @@ * * <p>For spliterators that report a characteristic of {@code IMMUTABLE} * or {@code CONCURRENT}, or that are - * <a href="Spliterator.html#binding">late-binding</a>, it is likely + * <a href="../Spliterator.html#binding">late-binding</a>, it is likely * more efficient to use {@link #longStream(Spliterator.OfLong)} instead. * The use of a {@code Supplier} in this form provides a level of * indirection that reduces the scope of potential interference with the @@ -402,7 +406,7 @@ * * <p>It is strongly recommended the spliterator report a characteristic of * {@code IMMUTABLE} or {@code CONCURRENT}, or be - * <a href="Spliterator.html#binding">late-binding</a>. Otherwise, + * <a href="../Spliterator.html#binding">late-binding</a>. Otherwise, * {@link #stream(Supplier, int)} should be used to * reduce the scope of potential interference with the source. See * <a href="package-summary.html#Non-Interference">Non-Interference</a> for @@ -426,7 +430,7 @@ * * <p>It is strongly recommended the spliterator report a characteristic of * {@code IMMUTABLE} or {@code CONCURRENT}, or be - * <a href="Spliterator.html#binding">late-binding</a>. Otherwise, + * <a href="../Spliterator.html#binding">late-binding</a>. Otherwise, * {@link #stream(Supplier, int)} should be used to * reduce the scope of potential interference with the source. See * <a href="package-summary.html#Non-Interference">Non-Interference</a> for @@ -451,7 +455,7 @@ * <p> * For spliterators that report a characteristic of {@code IMMUTABLE} * or {@code CONCURRENT}, or that are - * <a href="Spliterator.html#binding">late-binding</a>, it is likely + * <a href="../Spliterator.html#binding">late-binding</a>, it is likely * more efficient to use {@link #doubleStream(Spliterator.OfDouble)} instead. * The use of a {@code Supplier} in this form provides a level of * indirection that reduces the scope of potential interference with the @@ -485,7 +489,7 @@ * * <p>For spliterators that report a characteristic of {@code IMMUTABLE} * or {@code CONCURRENT}, or that are - * <a href="Spliterator.html#binding">late-binding</a>, it is likely + * <a href="../Spliterator.html#binding">late-binding</a>, it is likely * more efficient to use {@link #doubleStream(Spliterator.OfDouble)} instead. * The use of a {@code Supplier} in this form provides a level of * indirection that reduces the scope of potential interference with the
