Changeset 1659 for branches/eraser6/6.0/Eraser/LogForm.cs

Timestamp:

1/25/2010 2:35:37 AM (2 years ago)

Author:

lowjoel

Message:

When dealing with the LogForm? and the ProgressForm? classes, for events in which we wait for triggering by the task object, we should check that the form we belong to exists (i.e. not disposed) before we do anything on the form since forms which have Disposed() called will still receive events. There is a fix removed in this revision (in ProgressForm?.cs/WrapItemName) which may introduce new bugs but since this is really hard to reproduce I'll commit a blind fix and wait for more reports to come in.

File:

• branches/eraser6/6.0/Eraser/LogForm.cs (modified) (2 diffs)

branches/eraser6/6.0/Eraser/LogForm.cs

@@ -77 +77 @@
         private void task_NewSession(object sender, EventArgs e)
         {
+            if (!IsHandleCreated)
+                return;
             if (InvokeRequired)
             {
@@ -88 +90 @@
         private void task_Logged(object sender, LogEventArgs e)
         {
+            if (!IsHandleCreated)
+                return;
             if (InvokeRequired)
             {