Unexpected parsing behavior after upgrading Acorn from v8.9.0 to v8.11.3

[aladdin-add]

I recently upgraded the Acorn version in my project from v8.9.0 to v8.11.3 and noticed unexpected changes in the parsing behavior. Specifically, the parsing of certain constructs is different than the previous version.

1. The following export statements now parse "default" as "Identifier" instead of "Keyword":

export { default } from "foo";
export { foo as default } from "foo";
import { default as foo } from "foo";

2. The parsing of "var" and "null" in import statements has also changed. Previously, "var" was parsed as "Keyword" and "null" as "Null", but now both are parsed as "Identifier":

import { var as baz } from "foo";
import { null as nil } from "bar";

I am unsure if this change in parsing behavior is intentional or a bug. In the previous version, "default" was recognized as a "Keyword," and "var" and "null" were parsed differently. Please let me know if this is the expected behavior or if it is a bug introduced in the new version.

[marijnh]

These changes are intentional. In this context the identifiers are binding names, not keywords.