Skip to content
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

Fixed System.read function #38

Merged
merged 4 commits into from May 9, 2014
Merged

Fixed System.read function #38

merged 4 commits into from May 9, 2014

Conversation

SquidDev
Copy link
Contributor

@SquidDev SquidDev commented May 8, 2014

This is a re-implementation of the read function from bios.lua. Supports replacement characters, history, paste etc...

@SquidDev
Copy link
Contributor Author

SquidDev commented May 9, 2014

I'm going to have to rollback the last commit. Now I realise why Dan200 writes mods and I don't!

@Yevano
Copy link
Owner

Yevano commented May 9, 2014

Why port over the entire function? You should just cast the parameters and pass them to the native function.

@SquidDev
Copy link
Contributor Author

SquidDev commented May 9, 2014

The read function relies on os.pullEvent. This doesn't appear to work in SEE (this is the same reason Rednet doesn't work) and so needs to be ported .

@Yevano
Copy link
Owner

Yevano commented May 9, 2014

Ah, I see.

Yevano pushed a commit that referenced this pull request May 9, 2014
Fixed System.read function
@Yevano Yevano merged commit 0f61c23 into Yevano:master May 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants