Thanks Roger. Will apply your suggestions.
> On Jan 7, 2020, at 11:02 AM, Roger Riggs <roger.ri...@oracle.com> wrote:
>
> Hi Jim,
>
> The nature of the problem should be apparent from the summary and problem
> descriptions.
>
> The CSR/diff does not identify the methods modified.
>
> The new 3rd bullet seems more like a consequence of the first 2 actions, not
> an independent step.
> The word 'implies' seems out of place in a specification of behavior.
> I think you can drop 'this/thus implies' without loss of clarity or intent.
>
> Is it significant that the first change spells out \n and \r and the second
> refers to 'line terminator'?
>
> Roger
>
>
> On 1/7/20 7:43 AM, Jim Laskey wrote:
>> Please review this CSR to clarify the javadoc details of String::indent
>> (proposed by Andrew Leonard).
>>
>> webrev: http://cr.openjdk.java.net/~jlaskey/8230800/webrev.00/index.html
>> <http://cr.openjdk.java.net/~jlaskey/8230800/webrev.00/index.html>
>> csr: https://bugs.openjdk.java.net/browse/JDK-8236688
>> jbs: https://bugs.openjdk.java.net/browse/JDK-8230800
>
