looking good Brian...
> On Jun 5, 2019, at 5:31 PM, Brian Burkhalter <brian.burkhal...@oracle.com>
> wrote:
>
> Updated patch based on CSR feedback:
>
> --- a/src/java.base/share/classes/java/io/PrintWriter.java
> +++ b/src/java.base/share/classes/java/io/PrintWriter.java
> @@ -704,9 +704,8 @@
>
> /**
> * Terminates the current line by writing the line separator string. The
> - * line separator string is defined by the system property
> - * {@code line.separator}, and is not necessarily a single newline
> - * character ({@code '\n'}).
> + * line separator is {@link System#lineSeparator()} and is not
> necessarily
> + * a single newline character ({@code '\n'}).
> */
> public void println() {
> newLine();
>
> Thanks,
>
> Brian
>
>> On Jun 5, 2019, at 1:21 PM, Brian Burkhalter <brian.burkhal...@oracle.com>
>> wrote:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8219992
>> <https://bugs.openjdk.java.net/browse/JDK-8219992>
>>
>> Proposed change included below (excluding copyright year update); CSR to
>> follow.
>>
>> Thanks,
>>
>> Brian
>>
>> --- a/src/java.base/share/classes/java/io/PrintWriter.java
>> +++ b/src/java.base/share/classes/java/io/PrintWriter.java
>> @@ -704,9 +704,8 @@
>>
>> /**
>> * Terminates the current line by writing the line separator string. The
>> - * line separator string is defined by the system property
>> - * {@code line.separator}, and is not necessarily a single newline
>> - * character ({@code '\n'}).
>> + * line separator string is defined by the {@link
>> System#lineSeparator()}
>> + * method, and is not necessarily a single newline character ({@code
>> '\n'}).
>> */
>> public void println() {
>>
>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen|
Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>
