From c11bb1721af4d1418c42716b54c43ed032a397e5 Mon Sep 17 00:00:00 2001
From: Julien STAUB <atsju2@yahoo.fr>
Date: Wed, 12 Aug 2026 11:32:37 +0200
Subject: [PATCH] #358 yes button was not working at all

---
 mainwindow.h       |  1 -
 surfacemanager.cpp | 16 +++++++++-------
 surfacemanager.h   |  1 -
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/mainwindow.h b/mainwindow.h
index 1f03c955..7aa53a7d 100644
--- a/mainwindow.h
+++ b/mainwindow.h
@@ -105,7 +105,6 @@ public slots:
 signals:
     void load(SurfaceManager *);
     //void load(QStringList, SurfaceManager *);
-    void messageResult(int);
     void gammaChanged(bool, double);
 private slots:
     void updateChannels(const cv::Mat&);
diff --git a/surfacemanager.cpp b/surfacemanager.cpp
index 06deb989..dffe1d35 100644
--- a/surfacemanager.cpp
+++ b/surfacemanager.cpp
@@ -1349,8 +1349,8 @@ wavefront * SurfaceManager::readWaveFront(const QString &fileName){
     }
     wf->m_inside = CircleOutline(QPointF(xo,yo), rado);
 
-
     if (lambda != md->lambda){
+        int resp = QMessageBox::No;
         if (lambdResp == ASK){
             QString message("The interferogram wavelength (");
             message += QString("%1").arg( lambda, 6, 'f', 3) +
@@ -1358,7 +1358,7 @@ wavefront * SurfaceManager::readWaveFront(const QString &fileName){
                     "Do you want to make the config match?";
 
 
-            int resp = QMessageBox(QMessageBox::Information,"config",message,QMessageBox::Yes|QMessageBox::No |
+            resp = QMessageBox(QMessageBox::Information,"config",message,QMessageBox::Yes|QMessageBox::No |
                                  QMessageBox::YesToAll | QMessageBox::NoToAll).exec();
 
             switch (resp){
@@ -1371,7 +1371,7 @@ wavefront * SurfaceManager::readWaveFront(const QString &fileName){
             }
         }
 
-        if ( lambdResp == YES || messageResult == QMessageBox::Yes){
+        if ( lambdResp == YES || resp == QMessageBox::Yes){
             md->newLambda(QString::number(lambda));
         }
     }
@@ -1382,8 +1382,9 @@ wavefront * SurfaceManager::readWaveFront(const QString &fileName){
         message += QString("%1").arg(diam, 6, 'f', 3) +
                 ") Of the wavefront does not match the config value of " + QString("%1\n").arg(md->diameter, 6, 'f', 3) +
                 "Do you want to make the config match?";
+        int resp = QMessageBox::No;
         if (diamResp == ASK){
-            int resp = QMessageBox(QMessageBox::Information,"config", message,QMessageBox::Yes|QMessageBox::No |
+            resp = QMessageBox(QMessageBox::Information,"config", message,QMessageBox::Yes|QMessageBox::No |
                                  QMessageBox::YesToAll | QMessageBox::NoToAll).exec();
 
             switch (resp){
@@ -1395,7 +1396,7 @@ wavefront * SurfaceManager::readWaveFront(const QString &fileName){
                 break;
             }
         }
-        if (diamResp == YES || messageResult == QMessageBox::Yes){
+        if (diamResp == YES || resp == QMessageBox::Yes){
             emit diameterChanged(diam);
         }
         else {
@@ -1411,8 +1412,9 @@ wavefront * SurfaceManager::readWaveFront(const QString &fileName){
                 ") Of the wavefront does not match the config value of " + QString("%1\n").arg(md->roc, 6, 'f', 3) +
                 "Do you want to make the config match?";
         //qDebug() << message;
+        int resp = QMessageBox::No;
         if (rocResp == ASK){
-            int resp = QMessageBox(QMessageBox::Information,"config",message,QMessageBox::Yes|QMessageBox::No |
+            resp = QMessageBox(QMessageBox::Information,"config",message,QMessageBox::Yes|QMessageBox::No |
                                  QMessageBox::YesToAll | QMessageBox::NoToAll).exec();
 
             switch (resp){
@@ -1424,7 +1426,7 @@ wavefront * SurfaceManager::readWaveFront(const QString &fileName){
                 break;
             }
         }
-        if (rocResp == YES || messageResult == QMessageBox::Yes){
+        if (rocResp == YES || resp == QMessageBox::Yes){
             emit rocChanged(roc);
         }
         else {
diff --git a/surfacemanager.h b/surfacemanager.h
index 16ce5d87..66c50993 100644
--- a/surfacemanager.h
+++ b/surfacemanager.h
@@ -104,7 +104,6 @@ class SurfaceManager : public QObject
     wavefront* m_standRemoved;
     int insideOffset;
     int outsideOffset;
-    int messageResult;
     void inspectWavefront();
 
     cv::Mat_<double> subtractPlane(cv::Mat_<double> phase, cv::Mat_<uint8_t> mask);