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

Sleep for Windows #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

andrewstarks
Copy link
Contributor

Added check for windows and sleep method for it, if found.

Also, I have a tiny C library that just does sleep. It also works for unix/mac/windows and support usleep, as well as sleep. I added some commented-out code that you can add in, if you wish to support it. I thought it was presumptuous of me to do so. :)

Andrew Starks added 4 commits January 31, 2014 14:43
…about pushing this up as a pull request. Sorry. First: require paths were done in a non-standard way. They should still work as they had, but they follow the more traditional way of prefixing everything with the library name. There are other ways to tighten this up, but this method didn't touch the structure. Second, unpack was moved to table.unpack, so I added that. Lastly, I made sleep more modular, adding support for windows and also for my lsleep library. It's transparent to the user, so it shouldn't be a problem.
@xopxe
Copy link
Owner

xopxe commented Feb 2, 2014

Cant merge this from the web interface (claims some conflict), will look into it later

@andrewstarks
Copy link
Contributor Author

Yeah. This is because I included the idle code in some other commit. Sorry about that. I'll sync my master with yours, remove idle.lua and then merge the branch into it.

@andrewstarks
Copy link
Contributor Author

Yeah. I tried to do this in some sort of "right way." In short, keep the changes in idle.lua that are in this commit. As I said, I added the old file to every commit, by accident. :(

@srdgame
Copy link

srdgame commented May 8, 2014

ok, I found the bug creator here 🎱

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.

3 participants