RFR - JDK-8215493 String::indent inconsistency with blank lines
Roger Riggs
Roger.Riggs at oracle.com
Thu Dec 20 16:10:27 UTC 2018
Hi Jim,
Is there a missing test? I'd expect to see a corresponding test change
also.
otherwise, looks fine.
Regards, Roger
On 12/19/2018 03:13 PM, Jim Laskey wrote:
> Please review these changes for jdk 12.
>
> webrev: http://cr.openjdk.java.net/~jlaskey/8215493/webrev/index.html <http://cr.openjdk.java.net/~jlaskey/8215493/webrev/index.html>
>
> There is inconsistency with regards to blank lines and indentation. Adding indent space ignores blank lines, where subtraction indentation white space does not ignore blank lines. The solution is to remove the condition "Blank lines are unaffected" from the specification with appropriate changes to the implementation.
>
> Thank you.
>
> Cheers,
>
> — Jim
>
> CSR: https://bugs.openjdk.java.net/browse/JDK-8215499 <https://bugs.openjdk.java.net/browse/JDK-8215499>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8215493 <https://bugs.openjdk.java.net/browse/JDK-8215493>
>
>
More information about the core-libs-dev
mailing list