Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Sleep for Windows#10

Open
andrewstarks wants to merge 4 commits intoxopxe:masterfrom
andrewstarks:fix_sleep
Open

Sleep for Windows#10
andrewstarks wants to merge 4 commits intoxopxe:masterfrom
andrewstarks:fix_sleep

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

Comments