[Webkit-unassigned] [Bug 174212] New: [JSC] Add suppor for class fields to the parser

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 6 11:19:19 PDT 2017


            Bug ID: 174212
           Summary: [JSC] Add suppor for class fields to the parser
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: JavaScriptCore
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: xan.lopez at gmail.com

Created attachment 314736

  --> https://bugs.webkit.org/attachment.cgi?id=314736&action=review

WIP data field parsing

Ok, a really simple/rough WIP, but I think enough to ask a couple questions. I had a larger patch with a new parseDataField() method (lots of copy/paste) that I'd run in a loop at the begining of parseClass() until it stopped parsing fields, but then realized data fields can be mixed with methods in any order, so had to scrap that. Now I just branch after parsing an identifier to check if it's a method or a field. Issues:

1) ASI is not done. Actually not quite sure how to do that, probably I'm missing something trivial.
2) I'm just using as-is the code to parse assignments. Probably needs tweaking.
3) I'm not generating any code.

My idea for the first patch would be to figure out 1) and 2) and to pass the stuff in https://github.com/tc39/test262/pull/1064
I tried a bunch of examples and they all work, including new ones like class { field; method; field; etc }

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

More information about the webkit-unassigned mailing list