From dfd9aa7808f140c95f8791874f3b56bd73cdd9d7 Mon Sep 17 00:00:00 2001
From: Senex Crenshaw <senexcrenshaw@gmail.com>
Date: Tue, 20 Apr 2021 14:05:09 +0000
Subject: [PATCH] Further q cleanup

---
 esphome/components/nextion/nextion.cpp | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/esphome/components/nextion/nextion.cpp b/esphome/components/nextion/nextion.cpp
index 4dedb0a0a4ef..9387c6f125b4 100644
--- a/esphome/components/nextion/nextion.cpp
+++ b/esphome/components/nextion/nextion.cpp
@@ -555,7 +555,6 @@ void Nextion::process_nextion_commands_() {
             component->get_queue_type() != NextionQueueType::SWITCH) {
           ESP_LOGE(TAG, "ERROR: Received numeric return but next in queue \"%s\" is not a valid sensor type %d",
                    component->get_variable_name().c_str(), component->get_queue_type());
-          break;
         } else {
           ESP_LOGN(TAG, "Received numeric return for variable %s, queue type %d:%s, value %d",
                    component->get_variable_name().c_str(), component->get_queue_type(),
@@ -646,15 +645,12 @@ void Nextion::process_nextion_commands_() {
 
         ESP_LOGN(TAG, "Got sensor variable_name=%s value=%d", variable_name.c_str(), value);
 
-        // NextionSensorResponseQueue *nq = new NextionSensorResponseQueue;
-        // nq->variable_name = variable_name;
-        // nq->state = value;
-        // this->sensorq_.push_back(nq);
         for (auto *sensor : this->sensortype_) {
           sensor->process_sensor(variable_name, value);
         }
         break;
       }
+
       // Data from nextion is
       // 0x92 - Start
       // variable length of 0x70 return formatted data (bytes) that contain the variable name: prints "temp1",0
@@ -802,7 +798,7 @@ void Nextion::process_nextion_commands_() {
 
         delete this->nextion_queue_[i];
 
-        this->nextion_queue_.pop_front();
+        this->nextion_queue_.erase(this->nextion_queue_.begin() + i);
 
       } else {
         break;
@@ -849,7 +845,9 @@ void Nextion::set_nextion_sensor_state(NextionQueueType queue_type, const std::s
       }
       break;
     }
-    default: { ESP_LOGW(TAG, "set_nextion_sensor_state does not support a queue type %d", queue_type); }
+    default: {
+      ESP_LOGW(TAG, "set_nextion_sensor_state does not support a queue type %d", queue_type);
+    }
   }
 }