On Mon, Feb 16, 2015 at 4:01 PM, Holger Hans Peter Freyther < [email protected]> 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
