Changeset 1660 for trunk/eraser6/Eraser/LogForm.cs

Timestamp:

1/25/2010 2:42:53 AM (2 years ago)

Author:

lowjoel

Message:

Forward Port from Eraser 6.0: 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.

Location:

trunk/eraser6

Files:

• . (modified) (1 prop)
• Eraser/LogForm.cs (modified) (2 diffs)

trunk/eraser6/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)
             {