RFR: 8027302: Identifiers containing unicode escapes are not recognized as reserved words

Sundararajan Athijegannathan sundararajan.athijegannathan at oracle.com
Fri Oct 13 12:35:46 UTC 2017


   36 } catch (e if e instanceof SyntaxError) {
   37     Assert.assertTrue(e instanceof SyntaxError);
   38 }
   39

In test. Do you want to just catch catch(e) ?

-Sundar

On 13/10/17, 2:06 PM, Hannes Wallnöfer wrote:
> Please review:
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8027302
> Webrev: http://cr.openjdk.java.net/~hannesw/8027302/webrev.01
>
> ES6 and ES5 require different handling regarding unencoding Unicode escapes in keywords and identifiers. This patch makes handling more compliant and more efficient in ES6 mode (previously we didn’t cover all cases, and checked every identifier instead of just those containing escapes).
>
> About the removal of Lexer.isEscapeCharacter: this method always returns true in Lexer, and it’s only purpose was to be overridden in JSON parser, but JSON parser doesn’t extend Lexer/Parser anymore.
>
> Thanks,
> Hannes


More information about the nashorn-dev mailing list