On Thu, Feb 19, 2015 at 9:50 PM, Ben Coman <[email protected]> wrote:
> > > 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 > > btw, after the cleanup done by Issue 14942, Issue 14966 can delete ten class variables from Delay. When 14942 is integrated, can 14966 be integrated very soon after that. I am basing the change of Delay to microsecond delay off 14966. cheers -ben > > 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 >
