1335025 - Async function expression's source notation have parens in wrong position

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Tooru Fujisawa [:arai]]

code:
  (async function f() {}).toSource()

actual result:
  async (function f() {})

expected result:
  (async function f() {})

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Fix source notation of async function expression.

swapped the position of "(" and "async ".
this doesn't affect toString, since toString doesn't have "(".

Comment 2

[Till Schneidereit [:till]]

Comment on attachment 8831689 [details] [diff] [review]
Fix source notation of async function expression.

Review of attachment 8831689 [details] [diff] [review]:
-----------------------------------------------------------------

Uh, yes, good catch.

Comment 3

[Tooru Fujisawa [:arai]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e717a96e766da2cf93c6d0457cac19c99b38e4b7
Bug 1335025 - Fix source notation of async function expression. r=till

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/e717a96e766d