Changeset 1765 for branches/eraser6/CodeReview/Eraser.Manager/Executor.cs

Timestamp:

2/8/2010 6:44:59 AM (2 years ago)

Author:

lowjoel

Message:

Removed the TaskProcessing? and TaskProcessed? events in Executor which duplicate the TaskStarted? and TaskFinished? events in Task. The TaskEventArgs? class can't disappear just yet as the TaskAdded? and TaskDeleted? events still need them (the sender is not the task, it's the executor) Addresses #275: Code Review

File:

• branches/eraser6/CodeReview/Eraser.Manager/Executor.cs (modified) (1 diff)

branches/eraser6/CodeReview/Eraser.Manager/Executor.cs

@@ -126 +126 @@
                 TaskDeleted(this, e);
         }
-
-        /// <summary>
-        /// The task processing event object.
-        /// </summary>
-        public EventHandler<TaskEventArgs> TaskProcessing { get; set; }
-
-        /// <summary>
-        /// Helper function for the Task processing event.
-        /// </summary>
-        protected void OnTaskProcessing(TaskEventArgs e)
-        {
-            if (TaskProcessing != null)
-                TaskProcessing(this, e);
-        }
-
-        /// <summary>
-        /// The task processed event object.
-        /// </summary>
-        public EventHandler<TaskEventArgs> TaskProcessed { get; set; }
-
-        /// <summary>
-        /// Helper function for the Task processed event.
-        /// </summary>
-        protected void OnTaskProcessed(TaskEventArgs e)
-        {
-            if (TaskProcessed != null)
-                TaskProcessed(this, e);
-        }
     }