<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [ES6] Arrow function syntax. Error during invoking arrow function that created by 'eval' statement"
href="https://bugs.webkit.org/show_bug.cgi?id=148445#c4">Comment # 4</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [ES6] Arrow function syntax. Error during invoking arrow function that created by 'eval' statement"
href="https://bugs.webkit.org/show_bug.cgi?id=148445">bug 148445</a>
from <span class="vcard"><a class="email" href="mailto:utatane.tea@gmail.com" title="Yusuke Suzuki <utatane.tea@gmail.com>"> <span class="fn">Yusuke Suzuki</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=148445#c3">comment #3</a>)
<span class="quote">> The issue was that during first parsing arrow function, with body as
> expression (z=>z*2), has redundant symbol at the end, i.e. var f = x => x+1;
> and after parsing it has own source as 'x => x+1;' with redundant ';' at the
> end. It is OK for all cases except case for eval, when it leads to error
> during invoking function. I found this issue when was fixing issue with
> toString method, because toString returns extra ';' but should not. So I
> decided to add test that will prevent this issue with eval in future.
>
> Possible it would be better to split original patch into two patches, 'fix
> parsing' and add 'correct toString' to avoid misunderstanding, also possible
> I need to rename current issue to just 'adding additional tests for arrow
> function'.
>
> What do you think I need to do to make this issue more clear?</span >
Yeah. I think describing the details in ChangeLog (and issue title) is preferable :)</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>