On Mon, Feb 16, 2015 at 4:01 PM, Holger Hans Peter Freyther <holger@freyther.de> wrote:

On Thu, Oct 30, 2014 at 06:32:48PM -0400, stepharo wrote:

Good Morning Ben, Stef,

> We really need people to have a look at the code of Ben.

I downloaded a Pharo-4 image today and executed:

To clarify.�� The fix is not yet in Pharo-4. ��

��

time pharo --nodisplay --headless ./Pharo-40490.image�� eval "(Delay forSeconds: 5) wait"
a Delay(5000 msecs)

real�� �� 0m1.254s
user�� �� 0m1.192s
sys�� �� ��0m0.060s


So the first delay doesn't appear to wait the five seconds. I
would like to work on kqueue/epoll and remove polling and it
looks this involves code with Delay as well. I wonder if you
could provide an overview of what needs to be done and what is
done?

The pre-requisite issue 14942 is waiting for review. It really should take only 10 minutes to review...��

1. In latest Pharo 4, inspect "Delay newCodeEnabled" and observe that it is "true". ��

2. Open the Pharo4Inbox and filter on 14942. ����

3. Select newest slice and <Merge>.

4. Observe the diff shows all "false" parts removed (it doesn't really get much simpler than this to review).��

5. Click <merge> to finish.

6. Observe the image continues to run smoothly.

7. Report your observations on https://pharo.fogbugz.com/default.asp?14942��

I need to refresh myself on the fix I did back in 2-Dec.�� Its languished while some support infrastructure was added to help the system ride-through the change. ��

cheers -ben