From 36714877d8c747b1aa3c1f2a4c040733e189aa76 Mon Sep 17 00:00:00 2001
From: Joe Taylor <k1jt@arrl.org>
Date: Thu, 11 Jun 2015 15:32:55 +0000
Subject: [PATCH] Correct the bounds error test in avecho.  Ensure proper GUI
 controls visible in Echo mode. Reset nsum=0 when Tx Enable is toggled ON in
 Echo mode. Don't restart Monitor after finishing a sequence of Echo
 transmissions.

git-svn-id: svn+ssh://svn.code.sf.net/p/wsjt/wsjt/branches/wsjtx@5584 ab8295b8-cf94-4d9e-aec4-7959e3be5d79
---
 lib/avecho.f90 |  6 ++++--
 mainwindow.cpp | 11 ++++++++++-
 mainwindow.h   |  1 +
 3 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/lib/avecho.f90 b/lib/avecho.f90
index e36c32807..9ee799828 100644
--- a/lib/avecho.f90
+++ b/lib/avecho.f90
@@ -94,11 +94,13 @@ subroutine avecho(id2,ndop,nfrit,nqual,f1,xlevel,sigdb,snr,dfreq,width)
 
   sum=0.
   do i=ipk,ipk+300
-     if(i.gt.NZ .or. red(i).lt.1.0) exit
+     if(i.gt.NZ) exit
+     if(red(i).lt.1.0) exit
      sum=sum+(red(i)-1.0)
   enddo
   do i=ipk-1,ipk-300,-1
-     if(i.lt.1 .or. red(i).lt.1.0) exit
+     if(i.lt.1) exit
+     if(red(i).lt.1.0) exit
      sum=sum+(red(i)-1.0)
   enddo
   bins=sum/(red(ipk)-1.0)
diff --git a/mainwindow.cpp b/mainwindow.cpp
index f0a98da66..6bbbba3f3 100644
--- a/mainwindow.cpp
+++ b/mainwindow.cpp
@@ -438,6 +438,7 @@ MainWindow::MainWindow(bool multiple, QSettings * settings, QSharedMemory *shdme
   m_bEchoTxOK=false;
   m_bTransmittedEcho=false;
   m_nclearave=1;
+  m_bEchoTxed=false;
 
   signalMeter = new SignalMeter(ui->meterFrame);
   signalMeter->resize(50, 160);
@@ -840,6 +841,8 @@ void MainWindow::dataSink(qint64 frames)
       ui->decodedTextBrowser->appendText(t);
       if(m_echoGraph->isVisible()) m_echoGraph->plotSpec();
       m_nclearave=0;
+//Don't restart Monitor after an Echo transmission
+      if(m_bEchoTxed and !m_auto) monitor(false);
       return;
     }
     if( m_dialFreqRxWSPR==0) m_dialFreqRxWSPR=m_dialFreq;
@@ -965,7 +968,7 @@ void MainWindow::on_monitorButton_clicked (bool checked)
   if (!m_transmitting)
     {
       auto prior = m_monitoring;
-      monitor (checked);
+      monitor (checked and (m_mode!="Echo"));
 
       if (checked && !prior)
         {
@@ -1018,6 +1021,10 @@ void MainWindow::on_autoButton_clicked (bool checked)
                                   m_modeTx, ui->autoButton->isChecked (),
                                   m_transmitting);
   m_bEchoTxOK=false;
+  if(m_auto and (m_mode=="Echo")) {
+    m_nclearave=1;
+    echocom_.nsum=0;
+  }
   if(m_mode.mid(0,4)=="WSPR")  {
     QPalette* palette = new QPalette();
     if(m_auto or m_pctx==0) {
@@ -1844,6 +1851,7 @@ void MainWindow::guiUpdate()
     tx1=0.0;
     tx2=txDuration;
     if(m_auto and m_s6>4.0) m_bEchoTxOK=true;
+    if(m_transmitting) m_bEchoTxed=true;
   }
 
   if(m_mode.mid(0,4)=="WSPR") {
@@ -3156,6 +3164,7 @@ void MainWindow::on_actionWSPR_15_triggered()
 
 void MainWindow::on_actionEcho_triggered()
 {
+  on_actionJT4_triggered();
   m_mode="Echo";
   ui->actionEcho->setChecked(true);
   m_TRperiod=3;
diff --git a/mainwindow.h b/mainwindow.h
index 70533cc9d..92514ac20 100644
--- a/mainwindow.h
+++ b/mainwindow.h
@@ -386,6 +386,7 @@ private:
   int     m_nonWSPRTab;
   bool    m_bEchoTxOK;
   bool    m_bTransmittedEcho;
+  bool    m_bEchoTxed;
 
   float   m_pctZap;