HamBook – Diff between revs 26 and 27

Subversion Repositories:

?pathlinks?

Rev 26 – Rev 29 – Go to most recent revision – Details – Blame – Last modification – View Log

     public partial class Form1 : Form
+    {
         private ScheduledContinuation _changedConfigurationContinuation;
         private ScheduledContinuation _squelchScheduledContinuation;
         private ScheduledContinuation _powerScheduledContinuation;
+        private ScheduledContinuation _vfoAScheduledContinuation;
+        private ReaderWriterLockSlim _vfoAReaderWriterLockSlim;
+        private ScheduledContinuation _vfoBScheduledContinuation;
+        private ReaderWriterLockSlim _vfoBReaderWriterLockSlim;
         private Configuration.Configuration Configuration { get; set; }
         private SerialPortStream _serialPort;
         private LogMemorySink _memorySink;
             _changedConfigurationContinuation = new ScheduledContinuation();
+            _squelchScheduledContinuation = new ScheduledContinuation();
+            _powerScheduledContinuation = new ScheduledContinuation();
+            _vfoAScheduledContinuation = new ScheduledContinuation();
+            _vfoBScheduledContinuation = new ScheduledContinuation();
-            _squelchScheduledContinuation = new ScheduledContinuation();
+            _vfoAReaderWriterLockSlim = new ReaderWriterLockSlim(LockRecursionPolicy.SupportsRecursion);
-            _powerScheduledContinuation = new ScheduledContinuation();
+            _vfoBReaderWriterLockSlim = new ReaderWriterLockSlim(LockRecursionPolicy.SupportsRecursion);
             MessageBox.Show(Resources.No_updates_available_at_this_time, Resources.HamBook, MessageBoxButtons.OK,
                 MessageBoxIcon.Asterisk,
                 MessageBoxDefaultButton.Button1, MessageBoxOptions.DefaultDesktopOnly, false);
+        }
-        private async void toolStripComboBox1_SelectedIndexChanged(object sender, EventArgs e)
-        {
-            var toolStripComboBox = (ToolStripComboBox)sender;
-            if(RadioMode.TryParse(toolStripComboBox.Text, out var radioMode))
-            {
-                try
-                {
-                    await _catAssemblies.CatSetAsync<RadioMode, bool>("MD", new object[] { radioMode }, _cancellationToken);
-                }
-                catch (Exception exception)
-                {
-                    Log.Error(exception, Resources.Failed_to_set_radio_mode, radioMode);
-                }
-            }
-        }
         private async void onToolStripMenuItem_Click(object sender, EventArgs e)
+        {
             try
+            {
+            {
                 Log.Error(exception, Resources.Failed_to_set_power_state);
+            }
+        }
-        private async void toolStripComboBox2_MouseWheel(object sender, MouseEventArgs e)
+        private void scrollableToolStripComboBox2_MouseWheel(object sender, MouseEventArgs e)
+        {
-        {
+            var toolStripComboBox = (ScrollableToolStripComboBox)sender;
-            using (var soundPlayer = new SoundPlayer(Assembly.GetExecutingAssembly().GetManifestResourceStream("HamBook.Effects.pot.wav")))
+            if (int.TryParse(toolStripComboBox.Text, out var frequency))
-            {
-                var toolStripComboBox = (ScrollableToolStripComboBox)sender;
+            {
-                if (int.TryParse(toolStripComboBox.Text, out var frequency))
+                switch (Math.Sign(e.Delta))
-                {
-                    switch (Math.Sign(e.Delta))
-                    {
+                {
-                        case -1:
+                    case -1:
-                            frequency = frequency - Configuration.Navigation.FrequencyStep;
+                        frequency = frequency - Configuration.Navigation.FrequencyStep;
-                            break;
+                        break;
-                        case 1:
+                    case 1:
-                            frequency = frequency + Configuration.Navigation.FrequencyStep;
+                        frequency = frequency + Configuration.Navigation.FrequencyStep;
-                            break;
-                    }
-                    if (frequency < 0)
-                    {
-                        frequency = 0;
-                    }
-                    soundPlayer.Play();
-                    try
-                    {
-                        await _catAssemblies.CatSetAsync<int, bool>("FA", new object[] { frequency }, _cancellationToken);
-                        toolStripComboBox.Text = $"{frequency}";
-                        Log.Information($"{Resources.Set_VFO_A_frequency} {frequency}Hz");
-                    }
-                    catch (Exception exception)
-                    {
-                        Log.Error(exception, Resources.Failed_to_set_VFO_A_frequency);
                         break;
-                }
-            }
-        }
-        private async void scrollableToolStripComboBox1_MouseWheel(object sender, MouseEventArgs e)
-        {
-            using (var soundPlayer = new SoundPlayer(Assembly.GetExecutingAssembly().GetManifestResourceStream("HamBook.Effects.pot.wav")))
-            {
+                }
                 var toolStripComboBox = (ScrollableToolStripComboBox)sender;
-                if (int.TryParse(toolStripComboBox.Text, out var frequency))
-                {
-                    switch (Math.Sign(e.Delta))
-                    {
-                        case -1:
-                            frequency = frequency - Configuration.Navigation.FrequencyStep;
-                            break;
-                        case 1:
-                            frequency = frequency + Configuration.Navigation.FrequencyStep;
-                            break;
-                    }
-                    if (frequency < 0)
+                if (frequency < 0)
-                    {
+                {
-                        frequency = 0;
-                    }
+                    frequency = 0;
+                }
-                    soundPlayer.Play();
-                    try
+                if (!_vfoAReaderWriterLockSlim.TryEnterReadLock(250))
+                {
+                    return;
-                    {
+                }
-                        await _catAssemblies.CatSetAsync<int, bool>("FB", new object[] { frequency }, _cancellationToken);
+                try
+                {
-                        toolStripComboBox.Text = $"{frequency}";
+                    toolStripComboBox.Value = $"{frequency}";
+                }
-                        Log.Information($"{Resources.Set_VFO_B_frequency} {frequency}Hz");
+                finally
+                {
-                    catch (Exception exception)
+                    if (_vfoAReaderWriterLockSlim.IsReadLockHeld)
+                    {
-                        Log.Error(exception, Resources.Failed_to_set_VFO_B_frequency);
+                        _vfoAReaderWriterLockSlim.ExitReadLock();
                     }
+                }
+            }
+        }
-        private async void scrollableToolStripComboBox1_TextChanged(object sender, EventArgs e)
+        private async void scrollableToolStripComboBox2_SelectedIndexChanged(object sender, EventArgs e)
+        {
+            var toolStripComboBox = (ToolStripComboBox)sender;
+            if (int.TryParse(toolStripComboBox.Text, out var frequency))
-            var toolStripComboBox = (ToolStripComboBox)sender;
+            {
+                try
+                {
+                    using (var soundPlayer = new SoundPlayer(Assembly.GetExecutingAssembly().GetManifestResourceStream("HamBook.Effects.pot.wav")))
             if (int.TryParse(toolStripComboBox.Text, out var frequency))
                         await _catAssemblies.CatSetAsync<int, bool>("FB", new object[] { frequency }, _cancellationToken);
                 try
-                {
+                        soundPlayer.Play();
                     await _catAssemblies.CatSetAsync<int, bool>("FA", new object[] { frequency }, _cancellationToken);
+                }
                 catch (Exception exception)
                 {
-                    Log.Error(exception, Resources.Failed_to_set_VFO_A_frequency);
+                    Log.Error(exception, Resources.Failed_to_set_VFO_B_frequency);
-                }
-            }
+                }
+            }
+        }
+        private void scrollableToolStripComboBox2_TextChanged(object sender, EventArgs e)
+        {
+            if (!_vfoAReaderWriterLockSlim.TryEnterReadLock(250))
+            {
+                return;
+            }
-        private async void scrollableToolStripComboBox2_TextChanged(object sender, EventArgs e)
+            try
-        {
+            {
-            var toolStripComboBox = (ToolStripComboBox)sender;
+                var toolStripComboBox = (ScrollableToolStripComboBox)sender;
+                var frequency = toolStripComboBox.Text;
+                _vfoAScheduledContinuation.Schedule(TimeSpan.FromSeconds(1), async () =>
-            if (int.TryParse(toolStripComboBox.Text, out var frequency))
+                {
+                    contextMenuStrip1.InvokeIfRequired(contextMenuStrip =>
+                    {
+                        toolStripComboBox.Value = $"{frequency}";
+                    });
-                try
+                }, _cancellationToken);
+            }
-                    await _catAssemblies.CatSetAsync<int, bool>("FB", new object[] { frequency }, _cancellationToken);
+            finally
+            {
-                catch (Exception exception)
+                if (_vfoAReaderWriterLockSlim.IsReadLockHeld)
+                {
                     var fa = await _catAssemblies.CatReadAsync<int>("FA", new object[] { }, _cancellationToken);
                     contextMenuStrip1.InvokeIfRequired(contextMenuStrip =>
                     {
-                        scrollableToolStripComboBox1.Text = $"{fa}";
+                        scrollableToolStripComboBox1.Value = $"{fa}";
                     });
+                {
                     var fb = await _catAssemblies.CatReadAsync<int>("FB", new object[] { }, _cancellationToken);
                     contextMenuStrip1.InvokeIfRequired(contextMenuStrip =>
                     {
-                        scrollableToolStripComboBox2.Text = $"{fb}";
+                        scrollableToolStripComboBox2.Value = $"{fb}";
                     });
+                }
+            {
                 Log.Error(exception, Resources.Error_while_cycling_text);
+            }
+        }
-        private void scrollableToolStripComboBox5_MouseWheel(object sender, MouseEventArgs e)
+        private async void scrollableToolStripComboBox5_SelectedIndexChanged(object sender, EventArgs e)
-        {
-            var toolStripComboBox = (ScrollableToolStripComboBox)sender;
+        {
-            if (int.TryParse(toolStripComboBox.Text, out var memoryChannel))
+            using (var soundPlayer = new SoundPlayer(Assembly.GetExecutingAssembly().GetManifestResourceStream("HamBook.Effects.pot.wav")))
+            {
-            {
+                var toolStripComboBox = (ToolStripComboBox)sender;
-                switch (Math.Sign(e.Delta))
+                if (int.TryParse(toolStripComboBox.Text, out var channel))
-                {
-                    case -1:
+                {
-                        memoryChannel = memoryChannel - 1;
-                        break;
+                    if (_memoryChannelStore.TryGetValue(channel, out var memoryChannel))
-                    case 1:
-                        memoryChannel = memoryChannel + 1;
+                    {
-                        break;
-                }
-                if (memoryChannel < 0)
+                        try
-                {
-                    memoryChannel = 0;
-                }
+                        {
-                toolStripComboBox.Text = $"{memoryChannel}";
+                            await _catAssemblies.CatWriteAsync<int>("MC", new object[] { channel }, _cancellationToken);
-                Log.Information($"{Resources.Set_VFO_A_frequency} {memoryChannel}");
-            }
-        }
-        private async void scrollableToolStripComboBox5_TextChanged(object sender, EventArgs e)
-        {
-            var toolStripComboBox = (ToolStripComboBox)sender;
-            if (int.TryParse(toolStripComboBox.Text, out var channel))
-            {
-                if (_memoryChannelStore.TryGetValue(channel, out var memoryChannel))
-                {
-                    try
+                            scrollableToolStripComboBox1.Text = $"{memoryChannel.Frequency}";
-                    {
-                        await _catAssemblies.CatWriteAsync<int>("MC", new object[] { channel }, _cancellationToken);
+                            toolStripMenuItem27.Text = memoryChannel.Text;
-                        toolStripMenuItem27.Text = memoryChannel.Text;
+                            soundPlayer.Play();
+                        }
                         catch (Exception exception)
                     catch (Exception exception)
                             Log.Error(exception, Resources.Failed_to_set_memory_channel);
                         Log.Error(exception, Resources.Failed_to_set_memory_channel);
+            {
                 Log.Error(exception, Resources.Failed_tuning_current_frequency);
+            }
+        }
-        private void scrollableToolStripComboBox7_MouseWheel(object sender, MouseEventArgs e)
-        {
-            var toolStripComboBox = (ScrollableToolStripComboBox)sender;
-            contextMenuStrip1.InvokeIfRequired(async contextMenuStrip1 =>
-            {
-                if (int.TryParse(toolStripComboBox.Text, out var amplification))
-                {
-                    switch (Math.Sign(e.Delta))
-                    {
-                        case -1:
-                            amplification = amplification - 1;
-                            break;
-                        case 1:
-                            amplification = amplification + 1;
-                            break;
-                    }
-                    if (amplification < 0)
-                    {
-                        amplification = 0;
-                    }
-                    toolStripComboBox.Text = $"{amplification}";
-                    Log.Information($"{Resources.Amplification_set} {amplification}W");
-                }
-            });
-        }
-        private void scrollableToolStripComboBox7_TextChanged(object sender, EventArgs e)
-        {
-            var toolStripComboBox = (ScrollableToolStripComboBox)sender;
-            _powerScheduledContinuation.Schedule(TimeSpan.FromSeconds(1), () =>
-            {
-                contextMenuStrip1.InvokeIfRequired(async contextMenuStrip1 =>
-                {
-                    if (int.TryParse(toolStripComboBox.Text, out var amplification))
-                    {
-                        try
-                        {
-                            await _catAssemblies.CatWriteAsync<int>("PC", new object[] { amplification }, _cancellationToken);
-                            toolStripComboBox.Text = $"{amplification}";
-                            Log.Information($"{Resources.Amplification_set} {amplification}W");
-                        }
-                        catch (Exception exception)
-                        {
-                            Log.Error(exception, Resources.Failed_to_set_amplification);
-                        }
-                    }
-                });
-            }, _cancellationToken);
-        }
         private async void toolStripMenuItem22_CheckStateChanged(object sender, EventArgs e)
+        {
             var toolStripMenuItem = (ToolStripMenuItem)sender;
             try
+            {
                 Log.Error(exception, Resources.Failed_to_read_squelch);
+            }
+        }
-        private void scrollableToolStripComboBox8_MouseWheel(object sender, MouseEventArgs e)
+        private void scrollableToolStripComboBox1_MouseWheel(object sender, MouseEventArgs e)
+        {
+            var toolStripComboBox = (ScrollableToolStripComboBox)sender;
+            if (int.TryParse(toolStripComboBox.Text, out var frequency))
+            {
+                switch (Math.Sign(e.Delta))
+                {
+                    case -1:
+                        frequency = frequency - Configuration.Navigation.FrequencyStep;
+                        break;
+                    case 1:
+                        frequency = frequency + Configuration.Navigation.FrequencyStep;
+                        break;
-            var toolStripComboBox = (ScrollableToolStripComboBox)sender;
+                }
+                if (frequency < 0)
+                {
+                    frequency = 0;
-            var value = _catAssemblies.CatGetDefault<int>("SQ", new object[] { });
+                }
+                if (!_vfoAReaderWriterLockSlim.TryEnterReadLock(250))
+                {
+                    return;
             contextMenuStrip1.InvokeIfRequired(contextMenuStrip1 =>
-            {
+                try
+                {
+                    toolStripComboBox.Value = $"{frequency}";
+                }
-                if (int.TryParse(toolStripComboBox.Text, out var squelch))
+                finally
-                {
-                    switch (Math.Sign(e.Delta))
-                    {
-                        case -1:
-                            squelch = squelch - value;
+                {
-                            break;
-                        case 1:
+                    if (_vfoAReaderWriterLockSlim.IsReadLockHeld)
+                    {
+                        _vfoAReaderWriterLockSlim.ExitReadLock();
+                    }
+                }
+            }
+        }
+        private async void scrollableToolStripComboBox1_SelectedIndexChanged(object sender, EventArgs e)
-                            squelch = squelch + value;
+        {
+            var toolStripComboBox = (ToolStripComboBox)sender;
+            if (int.TryParse(toolStripComboBox.Text, out var frequency))
                             break;
+                try
-                    }
+                {
                     using (var soundPlayer = new SoundPlayer(Assembly.GetExecutingAssembly().GetManifestResourceStream("HamBook.Effects.pot.wav")))
+                    {
+                        await _catAssemblies.CatSetAsync<int, bool>("FA", new object[] { frequency }, _cancellationToken);
+                        soundPlayer.Play();
+                    }
+                }
+                catch (Exception exception)
+                {
+                    Log.Error(exception, Resources.Failed_to_set_VFO_A_frequency);
+                }
-                    if (squelch < 0)
+            }
+        }
+        private void scrollableToolStripComboBox1_TextChanged(object sender, EventArgs e)
+        {
+            if(!_vfoAReaderWriterLockSlim.TryEnterReadLock(250))
+            {
+                return;
+            }
+            try
+            {
+                var toolStripComboBox = (ScrollableToolStripComboBox)sender;
-                    {
+                var frequency = toolStripComboBox.Text;
+                _vfoAScheduledContinuation.Schedule(TimeSpan.FromSeconds(1), async () =>
+                {
+                    contextMenuStrip1.InvokeIfRequired(contextMenuStrip =>
+                    {
+                        toolStripComboBox.Value = $"{frequency}";
+                    });
+                }, _cancellationToken);
                         squelch = 0;
-                    }
+            finally
+            {
-                    toolStripComboBox.Text = $"{squelch}";
+                if (_vfoAReaderWriterLockSlim.IsReadLockHeld)
+                {
-                    Log.Information($"{Resources.Squelch_set} {squelch}");
+                    _vfoAReaderWriterLockSlim.ExitReadLock();
+                }
             });
+                        {
                             await _catAssemblies.CatWriteAsync<int>("SQ", new object[] { squelch }, _cancellationToken);
                             toolStripComboBox.Text = $"{squelch}";
-                            Log.Information($"{Resources.Squelch_set} {squelch}W");
+                            Log.Information($"{Resources.Squelch_set} {squelch}");
+                        }
                         catch (Exception exception)
+                        {
                             Log.Error(exception, Resources.Failed_to_set_squelch);
+                        }
+                    }
                 });
+            }, _cancellationToken);
+        }
+        private async void toolStripComboBox1_SelectedIndexChanged(object sender, EventArgs e)
+        {
+            var toolStripComboBox = (ToolStripComboBox)sender;
+            if (RadioMode.TryParse(toolStripComboBox.Text, out var radioMode))
+            {
+                try
+                {
+                    await _catAssemblies.CatSetAsync<RadioMode, bool>("MD", new object[] { radioMode }, _cancellationToken);
+                }
+                catch (Exception exception)
+                {
+                    Log.Error(exception, Resources.Failed_to_set_radio_mode, radioMode);
+                }
+            }
+        }
+        private void scrollableToolStripComboBox7_SelectedIndexChanged(object sender, EventArgs e)
+        {
+            var toolStripComboBox = (ScrollableToolStripComboBox)sender;
+            _powerScheduledContinuation.Schedule(TimeSpan.FromSeconds(1), () =>
+            {
+                contextMenuStrip1.InvokeIfRequired(async contextMenuStrip1 =>
+                {
+                    if (int.TryParse(toolStripComboBox.Text, out var amplification))
+                    {
+                        try
+                        {
+                            await _catAssemblies.CatWriteAsync<int>("PC", new object[] { amplification }, _cancellationToken);
+                            toolStripComboBox.Text = $"{amplification}";
+                            Log.Information($"{Resources.Amplification_set} {amplification}W");
+                        }
+                        catch (Exception exception)
+                        {
+                            Log.Error(exception, Resources.Failed_to_set_amplification);
+                        }
+                    }
+                });
+            }, _cancellationToken);
             }, _cancellationToken);