Potential crash in ParallelTask (Bug #283)

RubenWillems · RubenWillems · commit 9c46286e169d · 2013-08-04T09:00:05.000+02:00
diff --git a/project/core/tasks/ParallelTask.cs b/project/core/tasks/ParallelTask.cs
@@ -221,10 +221,12 @@ protected override bool Execute(IIntegrationResult result)
                         }
                     }
 
-                    // Tell everyone the task is done
-                    events[taskNumber].Set();
                     tasksDetails[taskNumber].Finished = true;
                     tasksDetails[taskNumber].ParentResult.BuildProgressInformation.UpdateStartupInformation(GetStatusInformation(tasksDetails[taskNumber]));
+
+                    // Tell everyone the task is done
+                    events[taskNumber].Set();
+
                 }, loop);
 
             }

Original file line number	Diff line number	Diff line change
`@@ -221,10 +221,12 @@ protected override bool Execute(IIntegrationResult result)`
`221`	`221`	`}`
`222`	`222`	`}`
`223`	`223`
`224`		`- // Tell everyone the task is done`
`225`		`- events[taskNumber].Set();`
`226`	`224`	`tasksDetails[taskNumber].Finished = true;`
`227`	`225`	`tasksDetails[taskNumber].ParentResult.BuildProgressInformation.UpdateStartupInformation(GetStatusInformation(tasksDetails[taskNumber]));`
	`226`	`+`
	`227`	`+ // Tell everyone the task is done`
	`228`	`+ events[taskNumber].Set();`
	`229`	`+`
`228`	`230`	`}, loop);`
`229`	`231`
`230`	`232`	`}`