| 06:54 | <rkirsling> | oh interesting |
| 06:55 | <rkirsling> | browsers are unified in not supporting https://github.com/tc39/test262/blob/main/test/language/expressions/assignment/S11.13.1_A7_T3.js |
| 06:55 | <rkirsling> | do I smell another web reality PR opportunity |
| 07:00 | <bakkot> | rkirsling: https://github.com/tc39/ecma262/issues/2659 |
| 07:01 | <bakkot> | actually that's a slightly different issue |
| 07:01 | <bakkot> | anyway yes there's some web reality bugs here |
| 07:01 | <bakkot> | if you want to figure out the actual semantics in engines and write those down, please do |
| 07:07 | <rkirsling> | will put that on my list right after I fix our (JSC's) double-eval of p in o[p] += foo |
| 07:08 | <rkirsling> | (which V8 also does, but that seems "obviously wrong" enough to go ahead and fix) |
| 07:15 | <bakkot> | see also https://github.com/tc39/ecma262/issues/1224 |
| 07:15 | <bakkot> | and https://github.com/tc39/ecma262/issues/467 |
| 07:25 | <rkirsling> | oh, related to the latter: this is now landed :) https://github.com/WebKit/WebKit/commit/9b945bdd8d46d2755cb115acb4596b3ed14d97eb |
| 08:02 | <rkirsling> | ...yeah so extending that to o[p] += 1 was just a copy-paste job lol |
| 08:02 | <rkirsling> | but |
| 08:03 | <rkirsling> | what's tricky is that if o[p] += f() evaluates p before f (as it does everywhere, regardless of the double-eval) |
| 08:04 | <rkirsling> | then arguing that o[p] = f() should evaluate f first seems unlikely |
| 08:04 | <rkirsling> | probably should just update impls |
| 08:05 | <rkirsling> | (short of it breaking websites, which certainly could happen, being web reality and all) |
| 08:44 | <rkirsling> | ...er well, I guess the argument for reality is "get LHS, eval RHS, do operation, set LHS" vs. "eval RHS, set LHS" |
| 08:44 | <rkirsling> | and the argument for spec is "but they look so similar" |
| 15:38 | <Rob Palmer> | The Interest Survey for the Helsinki June Plenary Meeting has been posted 🎉 Finland is famous for Aurora Borealis, the happiest people in the world, very clean tap water, Fazer chocolate, and saunas being mandatory as per buildings regulations. Please add yourself - it only takes 30s to complete. Responses are requested by Tuesday 12th March. |