[Ironruby-core] Code Review: IronRuby Thread fixes

Shri Borde Shri.Borde at microsoft.com
Tue Dec 16 17:38:30 EST 2008

Affects: Microsoft.Scripting.Core.dll and IronRuby

  tfpt review "/shelveset:Thread;REDMOND\sborde"

  Comment  :

  Fixes to all the currently disabled Thread tests

  Thread#stop/Kernel#sleep use an Event which can be signalled from other threads, instead of using Thread.Sleep

  ThreadOps.CreateThread catches and ignores ThreadAbortException instead of publishing it as RubyThreadInfo.Exception

  Thread#main - its per ScriptRuntime, mainly because that was easier to implement, and also there is no good answer to how Thread#main should function in hosted scenarios

  Adds lock to RubyModule.EnsureInitialized to prevent use of uninitialized modules in multi-thread scenarios.

  Changes RunRSpec.cmd to match the flags used by "rake why_regression"


  Fixes ExpressionWriter to deal with GotoExpression.Value being null in VisitGoto

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://rubyforge.org/pipermail/ironruby-core/attachments/20081216/f101620a/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Thread.diff
Type: application/octet-stream
Size: 29669 bytes
Desc: Thread.diff
URL: <http://rubyforge.org/pipermail/ironruby-core/attachments/20081216/f101620a/attachment-0001.obj>

More information about the Ironruby-core mailing list