JDK 8 code review request for 7082231 "Put a @since 1.7 on System.lineSeparator"

Eamonn McManus eamonn.mcmanus at oracle.com
Wed Aug 24 07:15:59 UTC 2011


Looks good. But I am thinking that this shows we need a regression test 
that ensures that there are no missing @since tags, given that JDK 7 
went out without this one. I am imagining two doclets, one of which 
creates a file with a list of the signatures of every public element 
(class, method, constructor, or field), and the other of which checks 
that every public element is either listed in the file or has a @since 
tag. So we would have run the first doclet on JDK 6 and then used its 
output with the second doclet to check JDK 7. I'm probably stating the 
obvious.

Éamonn


On 24/8/11 12:43 AM, Joe Darcy wrote:
> Hello.
>
> Please review this simple patch to fix 7082231 "Put a @since 1.7 on 
> System.lineSeparator", which will correct an @since tag omitted by the 
> JDK 7 fix for 6900043 "Add method to return line.separator property."
>
> diff --git a/src/share/classes/java/lang/System.java 
> b/src/share/classes/java/lang/System.java
> --- a/src/share/classes/java/lang/System.java
> +++ b/src/share/classes/java/lang/System.java
> @@ -632,6 +632,7 @@
>      *
>      * <p>On UNIX systems, it returns {@code "\n"}; on Microsoft
>      * Windows systems it returns {@code "\r\n"}.
> +     * @since 1.7
>      */
>     public static String lineSeparator() {
>         return lineSeparator;
>
>
> Thanks,
>
> -Joe



More information about the core-libs-dev mailing list