You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This obviously doesn't return "undefined" when setImmediate is in fact undefined, meaning that the test is failed and we end up in a code path that isn't expected. This was discovered with Gecko and setImmediate (which is only available on NodeJS 0.9+ and Edge 10+), but it should be reproducible with almost anything.
The text was updated successfully, but these errors were encountered:
…ase.
This requires two things:
- make sure that `SelectStatic` does not expand references to
global scopes, and
- when compiling `js.typeOf(globalRef)`, emit a
`JSTypeOfGlobalRef(_)` instead of a
`JSUnaryOp(typeof, JSGlobalRef(_))`.
sjrd
added a commit
to dotty-staging/dotty
that referenced
this issue
Sep 14, 2020
…ase.
This requires two things:
- make sure that `SelectStatic` does not expand references to
global scopes, and
- when compiling `js.typeOf(globalRef)`, emit a
`JSTypeOfGlobalRef(_)` instead of a
`JSUnaryOp(typeof, JSGlobalRef(_))`.
sjrd
added a commit
to dotty-staging/dotty
that referenced
this issue
Sep 14, 2020
…ase.
This requires two things:
- make sure that `SelectStatic` does not expand references to
global scopes, and
- when compiling `js.typeOf(globalRef)`, emit a
`JSTypeOfGlobalRef(_)` instead of a
`JSUnaryOp(typeof, JSGlobalRef(_))`.
sjrd
added a commit
to dotty-staging/dotty
that referenced
this issue
Sep 14, 2020
Discovered this as part of trying to port Cats Effect 3's build to DottyJS on 0.27.0-RC1 (SJS 1.2.0). Rough minimization:
(this is part of
PolyfillExecutionContext
btw)On Scala 2.13.3 (SJS 1.2.0), this line results in the following:
Which is exactly what I would expect. However, on 0.27.0-RC1 we get the following:
This obviously doesn't return
"undefined"
whensetImmediate
is in factundefined
, meaning that the test is failed and we end up in a code path that isn't expected. This was discovered with Gecko andsetImmediate
(which is only available on NodeJS 0.9+ and Edge 10+), but it should be reproducible with almost anything.The text was updated successfully, but these errors were encountered: