-
-
Notifications
You must be signed in to change notification settings - Fork 647
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[std] Move target.vm.Thread (and friends) to sys.thread.Thread #7999
Conversation
I'm keeping the typedefs in std for now because it conveniently also deals with the hxcpp build tool problem. Last thing to do is make this fail on non-threaded targets. We have to be a bit careful with ImportAll/dox in that regard. |
std/sys/thread/Deque.hx
Outdated
package sys.thread; | ||
|
||
#if (!target.threaded) | ||
#error "This target is not available on this target" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"This class" ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh god I need vacation...
…oundation#7999) * [std] move all thread classes to sys.thread * [neko] add @:coreApi * [cpp] add @:coreApi * [java] add @:coreApi * [std] purge redundant documentation * [std] document Deque and Lock * [tests] fix java.vm leftovers * [std] add deprecation typedefs * [std] fix Tls * [std] fix Deque * [typer] error if we access sys.thread on non-threaded target * [std] nobody saw that
This PR moves types neko.vm.Thread to sys.thread.Thread. It also adds
@:coreApi
to the neko, cpp and java classes. It does not add@:coreApi
for HL and eval because their interfaces are not 100% compatible. I consider this a separate (non-)issue though.This currently breaks the hxcpp build tool because it relies on neko.vm.Thread. This raises the question of deprecation strategy. We can add typedefs to hx3compat, or to std itself. I think we should fix the hxcpp build tool independently.
TODO