Changeset 2339 for trunk/eraser


Ignore:
Timestamp:
10/30/2011 2:07:22 AM (3 years ago)
Author:
lowjoel
Message:

Fixed initialisation order. The settings panel should be initialised first because it loads settings and may raise a dialog for invalid settings. The scheduler may then be affected and be in a deadlock when it receives events from the Executor instance. Potential fix for http://bbs.heidi.ie/viewtopic.php?f=2&t=8207&p=24805#p24535.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/eraser/Eraser/MainForm.cs

    r2279 r2339  
    4141    { 
    4242        private BasePanel CurrPage; 
    43         private SchedulerPanel SchedulerPage = new SchedulerPanel(); 
    44         private SettingsPanel SettingsPage = new SettingsPanel(); 
     43        private SchedulerPanel SchedulerPage; 
     44        private SettingsPanel SettingsPage; 
    4545 
    4646        public MainForm() 
    4747        { 
    4848            InitializeComponent(); 
     49            SettingsPage = new SettingsPanel(); 
     50            SchedulerPage = new SchedulerPanel(); 
    4951            contentPanel.Controls.Add(SchedulerPage); 
    5052            contentPanel.Controls.Add(SettingsPage); 
Note: See TracChangeset for help on using the changeset viewer.