-
Notifications
You must be signed in to change notification settings - Fork 58
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
stdin #13
base: master
Are you sure you want to change the base?
stdin #13
Conversation
Not sure why this was breaking compatibility with Edge, spread syntax is supposed to be supported by Edge.
I believe the issue is crypto is a member of Window, so it is not accessible in a worker.
Polyfill for Edge.
Hello, I'm trying to get stdin to work without any success.. Unfortunately I can't get stdin to work, am I missing something? Thank you! |
Checking the commits, I think there is a typo: In the Clang.html file Line 438: "userInput:" In front of "document.get...." Changing these still can't get it to work. The emModule.stdin doesn't seem to get called by clang behind. |
The property for the input text in the object that is passed to the worker should be userInput not inputText
Thanks for catching that. That was definitely an issue. I just copied these contents of clang.html and process-runtime.js into the files in the gh-pages branch and everything works fine. Not sure why it isn't working for you. Try downloading the gh-pages branch, downloading these two files from here, paste them in there, and open up clang.html. |
Yes, thank you, I got it working too! I don't know what the issue was. Yeah, unfortunately chrome gives "RuntimeError: memory access out of bounds", I am using Safari. |
User's program can now take stdin. There is an Input pane with a textarea for the input. The input needs to be in there before the program runs.
You can see in the commits that I reverted my changes from the Edge compatibility pull request, because I made the mistake of doing those changes in my forked master branch and I wasn't sure if you wanted to merge it. So, this pull request is just for stdin.