On Fri, 19 Jan 2024 14:14:26 GMT, Jim Laskey <[email protected]> wrote:
>> Currently String::translateEscapes does not support unicode escapes,
>> reported as a IllegalArgumentException("Invalid escape sequence: ...").
>> String::translateEscapes should translate unicode escape sequences to
>> provide full coverage,
>
> Jim Laskey has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Update copyright year of test
test/jdk/java/lang/String/TranslateEscapes.java line 2:
> 1: /*
> 2: * Copyright (c) 2019, 2024 Oracle and/or its affiliates. All rights
> reserved.
Sorry for this late note.
Seems like each copyright year must be followed by a comma `,`, otherwise
validate-source fails. See, for example, my own oversight
[here](https://github.com/openjdk/jdk/pull/17490/files).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17491#discussion_r1459422630
