ruby-changes:24331
From: nobu <ko1@a...>
Date: Sat, 14 Jul 2012 16:16:53 +0900 (JST)
Subject: [ruby-changes:24331] nobu:r36382 (trunk): test/unit.rb: _run_parallel
nobu 2012-07-14 16:16:14 +0900 (Sat, 14 Jul 2012) New Revision: 36382 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=36382 Log: test/unit.rb: _run_parallel * lib/test/unit.rb (Test::Unit::Runner#_run_parallel): move initializations with nothing to release outside begin/ensure. Modified files: trunk/ChangeLog trunk/lib/test/unit.rb Index: ChangeLog =================================================================== --- ChangeLog (revision 36381) +++ ChangeLog (revision 36382) @@ -1,3 +1,29 @@ +Sat Jul 14 16:16:12 2012 Nobuyoshi Nakada <nobu@r...> + + * lib/test/unit.rb (Test::Unit::Runner#_run_parallel): move + initializations with nothing to release outside begin/ensure. + + * lib/test/unit.rb (Test::Unit::Runner#_run_parallel): use + Array#uniq!. + + * lib/test/unit.rb (Test::Unit::Runner#deal): deal tasks to workers. + + * lib/test/unit.rb (Test::Unit::Runner#quit_workers): close and kill + all workers. + + * lib/test/unit.rb (Test::Unit::Runner#delete_worker): delete dead + worker from working set. + + * lib/test/unit.rb (Test::Unit::Runner#launch_worker): add new worker + to working set. + + * lib/test/unit.rb (Test::Unit::Runner#launch_worker): extract. + + * lib/test/unit.rb (Test::Unit::Runner#start_watchdog): extract. + + * lib/test/unit.rb (Test::Unit::Runner#_run_parallel): move + initializations with nothing to release outside begin/ensure. + Sat Jul 14 16:04:24 2012 Nobuyoshi Nakada <nobu@r...> * array.c (rb_ary_join): should not infected by separator if it is not Index: lib/test/unit.rb =================================================================== --- lib/test/unit.rb (revision 36381) +++ lib/test/unit.rb (revision 36382) @@ -458,18 +458,18 @@ return end + # Require needed things for parallel running + require 'thread' + require 'timeout' + @tasks = @files.dup # Array of filenames. + @need_quit = false + @dead_workers = [] # Array of dead workers. + @warnings = [] + @total_tests = @tasks.size.to_s(10) + shutting_down = false + rep = [] # FIXME: more good naming + begin - # Require needed things for parallel running - require 'thread' - require 'timeout' - @tasks = @files.dup # Array of filenames. - @need_quit = false - @dead_workers = [] # Array of dead workers. - @warnings = [] - @total_tests = @tasks.size.to_s(10) - shutting_down = false - rep = [] # FIXME: more good naming - # Array of workers. launch_worker = Proc.new { begin -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/