<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#c3">Comment # 3</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:gskachkov@gmail.com" title="GSkachkov <gskachkov@gmail.com>"> <span class="fn">GSkachkov</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=148445#c2">comment #2</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=259876&action=diff" name="attach_259876" title="Patch">attachment 259876</a> <a href="attachment.cgi?id=259876&action=edit" title="Patch">[details]</a></span>
> Patch
>
> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=259876&action=review">https://bugs.webkit.org/attachment.cgi?id=259876&action=review</a>
>
> > LayoutTests/ChangeLog:5
> > +
>
> Could you tell me which part fixed this issue in the original patch?</span >
All changes, that were made in 'parser' module, fix current issue.
<span class="quote">> <a href="http://trac.webkit.org/changeset/188928">http://trac.webkit.org/changeset/188928</a>
> Describing it here is nice.
>
> And could you tell me why `functionInfo.endOffset - 1` is needed in the
> original patch?</span >
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?</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>