00:02 | <jridgewell> | Side topic, devsnek: any chance you could make engine262 print its AST tree? |
00:02 | <devsnek> | its just acorn |
00:02 | <jridgewell> | I was curious how you implemented Optional Chaining |
00:02 | <devsnek> | oh |
00:02 | <devsnek> | i guess that part isn't acorn |
00:02 | <jridgewell> | Because ESTree is taking soooo long. |
00:02 | <jridgewell> | I was attempting to rewrite Babel parsing so we could get some progress. |
00:02 | <devsnek> | it's just BinaryExpression with operator: '??=' and whatnot |
00:03 | <jridgewell> | Huh? |
00:03 | <jridgewell> | `foo?.bar.baz?.qux`? |
00:03 | <devsnek> | oh optional chaining |
00:03 | <devsnek> | dunno how i goofed that |
00:03 | <devsnek> | https://github.com/engine262/engine262/blob/master/src/parse.mjs#L176-L211 |
00:04 | <devsnek> | those are some ASTs |
00:04 | <devsnek> | they're missing the property names i guess |
00:04 | <jridgewell> | Yah, so I was curious if we could implement an option to `JSON.stringify` your AST tree |
00:05 | <devsnek> | in the web ui you mean? |
00:05 | <jridgewell> | Yah |
00:05 | <jridgewell> | Or even to console, just with the option in the UI. |
00:06 | <devsnek> | i'm not sure when i might get around to it but the repo is here https://github.com/engine262/engine262.github.io |
00:07 | <rkirsling> | this is how JSC does it, if it helps: https://usercontent.irccloud-cdn.com/file/zk2ohYYP/jsc-optional-chaining.png |
00:08 | <devsnek> | that's how v8 does it |
00:08 | <devsnek> | its nothing like how engine262 or the spec does it |
00:08 | <devsnek> | because they can't jump |
00:08 | <rkirsling> | (a slide from a presentation I hope to give sometime this year 😅) |
00:08 | <devsnek> | they have to know before entering the node whether or not to skip it |
00:09 | <jmdyck> | ljharb: https://www.w3.org/Help/Webmaster#sitecomments |
00:12 | <ljharb> | ah thanks |
21:48 | <rkirsling> | caiolima: I think you opened this on the wrong repo? https://github.com/tc39/proposal-decimal/issues/55 |
21:49 | <jridgewell> | Lol |
21:49 | <ljharb> | caiolima: rkirsling: someone who has admin on both repos can transfer it |
22:00 | <caiolima> | 🙀 |
22:00 | <caiolima> | I think I got it right now, |
22:07 | <caiolima> | ljharb: I just closed this issue and created a new one into test262 https://github.com/tc39/test262/issues/2563 |
22:07 | <ljharb> | ok |