[Webkit-unassigned] [Bug 148445] [ES6] Arrow function syntax. Error during invoking arrow function that created by 'eval' statement

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=148445

--- Comment #4 from Yusuke Suzuki <utatane.tea at gmail.com> ---
(In reply to comment #3)
> 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?

Yeah. I think describing the details in ChangeLog (and issue title) is preferable :)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150827/bdd9d498/attachment.html>