Changeset 7975 for trunk/sources

Timestamp:

06/10/12 18:26:30 (12 years ago)

Author:

gkronber

Message:

#1851: fixed cross-thread exception related to the backgroundworker for creating experiments.

File:

• trunk/sources/HeuristicLab.Optimizer/3.3/CreateExperimentDialog.cs (modified) (12 diffs)

trunk/sources/HeuristicLab.Optimizer/3.3/CreateExperimentDialog.cs

@@ -284 +284 @@
           selectAllCheckBox.Checked = treeViewNodes.Count() == instances.SelectMany(x => x.Value).Count();
           selectNoneCheckBox.Checked = !treeViewNodes.Any(x => x.Checked);
-        } finally { suppressCheckAllNoneEventHandling = false; }
+        }
+        finally { suppressCheckAllNoneEventHandling = false; }
         UpdateVariationsLabel();
       }
@@ -298 +299 @@
           }
         }
-      } finally { suppressTreeViewEventHandling = false; }
+      }
+      finally { suppressTreeViewEventHandling = false; }
     }
 
@@ -320 +322 @@
         try {
           node.Parent.Checked = allChecked;
-        } finally { suppressTreeViewEventHandling = false; }
+        }
+        finally { suppressTreeViewEventHandling = false; }
       }
     }
@@ -328 +331 @@
         if (selectAllCheckBox.Checked) {
           suppressCheckAllNoneEventHandling = true;
-          try { selectNoneCheckBox.Checked = false; } finally { suppressCheckAllNoneEventHandling = false; }
+          try { selectNoneCheckBox.Checked = false; }
+          finally { suppressCheckAllNoneEventHandling = false; }
           try {
             suppressTreeViewEventHandling = true;
@@ -337 +341 @@
               }
             }
-          } finally { suppressTreeViewEventHandling = false; }
+          }
+          finally { suppressTreeViewEventHandling = false; }
         }
       }
@@ -346 +351 @@
         if (selectNoneCheckBox.Checked) {
           suppressCheckAllNoneEventHandling = true;
-          try { selectAllCheckBox.Checked = false; } finally { suppressCheckAllNoneEventHandling = false; }
+          try { selectAllCheckBox.Checked = false; }
+          finally { suppressCheckAllNoneEventHandling = false; }
           try {
             suppressTreeViewEventHandling = true;
@@ -355 +361 @@
               }
             }
-          } finally { suppressTreeViewEventHandling = false; }
+          }
+          finally { suppressTreeViewEventHandling = false; }
         }
       }
@@ -677 +684 @@
           node.Collapse();
         selectNoneCheckBox.Checked = true;
-      } catch { }
+      }
+      catch { }
       try {
         SetMode(DialogMode.Normal);
         if (e.Error != null) MessageBox.Show(e.Error.Message, "Error occurred", MessageBoxButtons.OK, MessageBoxIcon.Error);
-      } catch { }
+      }
+      catch { }
     }
     #endregion
@@ -710 +719 @@
             try {
               ProblemInstanceManager.LoadData(provider, descriptor, (IProblemInstanceConsumer)algorithm.Problem);
-            } catch (Exception ex) {
+            }
+            catch (Exception ex) {
               failedInstances.AppendLine(descriptor.Name + ": " + ex.Message);
               failed = true;
@@ -721 +731 @@
       }
       if (localExperiment != null) {
+        // don't do GUI stuff in do_work
         // this step can take some time
-        SetMode(DialogMode.PreparingExperiment);
         experimentCreationBackgroundWorker.ReportProgress(-1);
         localExperiment.Prepare(true);
@@ -833 +843 @@
         experimentCreationProgressBar.Value = e.ProgressPercentage;
       } else {
+        SetMode(DialogMode.PreparingExperiment);
+
         experimentCreationProgressBar.Style = ProgressBarStyle.Marquee;
       }
@@ -847 +859 @@
           Close();
         }
-      } catch { }
+      }
+      catch { }
     }
     #endregion