RFR JDK-8236688 Clarify String::indent javadoc (CSR)

[Jim Laskey]

Thanks Roger. Will apply your suggestions.

> On Jan 7, 2020, at 11:02 AM, Roger Riggs <Roger.Riggs at 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
>