I added a test for it, so we shouldn't regress in this way ever again.

Peter K Chan wrote:
I can verify that the bug has been fixed in revision 6796. I didn't make any 
further progress in producing a test case, but I assume that it is no longer 
necessary, since the bug is fixed.

Thanks to Joshua for filing the bug.

Peter

-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Charles Oliver 
Nutter
Sent: Monday, May 26, 2008 3:24 PM
To: [email protected]
Subject: Re: [jruby-dev] Regression in Truck: ReturnJump

Vladimir Sizikov wrote:
Hi Peter,

Joshua Hull reported most probably the very same issue over the #jruby
IRC channel, and
filed the followiing Jira issue:
http://jira.codehaus.org/browse/JRUBY-2573
Revision 6754 randomly dispatches the wrong method under multithreaded loads.

Ok, I get a big fat stupid award for the day. I wanted to make there be a single ReturnJump for each thread, since there should be no way for a return to cascade out across threads. So I added a RETURN_JUMP field to ThreadContext...and then muscle memory took over and I made it static final.

I'm repairing it to be simply final, which should resolve this. Stupid stupid stupid...

- Charlie

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email




---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email




---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email


Reply via email to