This is important enough to trigger another release.

I wish I had known...

This PR fixes #1625 (50% CPU usage for a simple timer demo) for Windows.

Inspired by the ancient wisdom of Joe Duffy, we now yield a Pony scheduler thread by calling Sleep(1) instead of Sleep(0). This results in a sleep of anywhere from 1 to 16 milliseconds, depending on the global value of the system timer.

The example in #1625 now runs with minimal CPU usage on my system. Comparing libponyrt.benchmarks before and after the change doesn't seem to show a significant difference.

wasn't in the commit message, i would have added it on squashing.

That was an excellent message @kulibali, can you add things that in the body of your commit in the future so they live on in the repo?

Ok, will do.

Sleep(0) actually only gives up the current thread’s time-slice if a thread at equal priority is ready to run.

😱

I built ponyc from sources and can confirm that it now works in Windows 10, the timer example uses ~0% cpu.