From a492935951040a2fd54a4cffe1ce317102ee3511 Mon Sep 17 00:00:00 2001
From: Euripedes Rocha <rocha.euripedes@espressif.com>
Date: Fri, 9 Jun 2023 10:11:18 +0200
Subject: [PATCH] Removes leftover calls to event_handler

The possibility to add a callback as custom handler was removed from
the client in favor of esp_event. These removes the older alternative
that wasn't possible to use.
---
 include/mqtt_client.h          |  1 -
 lib/include/mqtt_client_priv.h |  1 -
 mqtt_client.c                  | 12 ------------
 3 files changed, 14 deletions(-)

diff --git a/include/mqtt_client.h b/include/mqtt_client.h
index 389abf2..b458b09 100644
--- a/include/mqtt_client.h
+++ b/include/mqtt_client.h
@@ -213,7 +213,6 @@ typedef struct esp_mqtt_event_t {
 
 typedef esp_mqtt_event_t *esp_mqtt_event_handle_t;
 
-typedef esp_err_t (*mqtt_event_callback_t)(esp_mqtt_event_handle_t event);
 
 /**
  * *MQTT* client configuration structure
diff --git a/lib/include/mqtt_client_priv.h b/lib/include/mqtt_client_priv.h
index 2e6f923..90e86b1 100644
--- a/lib/include/mqtt_client_priv.h
+++ b/lib/include/mqtt_client_priv.h
@@ -61,7 +61,6 @@ typedef struct mqtt_state {
 } mqtt_state_t;
 
 typedef struct {
-    mqtt_event_callback_t event_handle;
     esp_event_loop_handle_t event_loop_handle;
     int task_stack;
     int task_prio;
diff --git a/mqtt_client.c b/mqtt_client.c
index 9a9e4d8..8967006 100644
--- a/mqtt_client.c
+++ b/mqtt_client.c
@@ -980,16 +980,12 @@ static esp_err_t esp_mqtt_dispatch_event(esp_mqtt_client_handle_t client)
     client->event.protocol_ver = client->connect_info.protocol_ver;
     esp_err_t ret = ESP_FAIL;
 
-    if (client->config->event_handle) {
-        ret = client->config->event_handle(&client->event);
-    } else {
 #ifdef MQTT_SUPPORTED_FEATURE_EVENT_LOOP
         esp_event_post_to(client->config->event_loop_handle, MQTT_EVENTS, client->event.event_id, &client->event, sizeof(client->event), portMAX_DELAY);
         ret = esp_event_loop_run(client->config->event_loop_handle, 0);
 #else
         return ESP_FAIL;
 #endif
-    }
     if (client->connect_info.protocol_ver == MQTT_PROTOCOL_V_5) {
 #ifdef MQTT_PROTOCOL_5
         esp_mqtt5_client_delete_user_property(client->event.property->user_property);
@@ -2132,10 +2128,6 @@ esp_err_t esp_mqtt_client_register_event(esp_mqtt_client_handle_t client, esp_mq
         return ESP_ERR_INVALID_ARG;
     }
 #ifdef MQTT_SUPPORTED_FEATURE_EVENT_LOOP
-    if (client->config->event_handle) {
-        ESP_LOGW(TAG, "Registering event loop while event callback is not null, clearing callback");
-        client->config->event_handle = NULL;
-    }
 
     return esp_event_handler_register_with(client->config->event_loop_handle, MQTT_EVENTS, event, event_handler, event_handler_arg);
 #else
@@ -2150,10 +2142,6 @@ esp_err_t esp_mqtt_client_unregister_event(esp_mqtt_client_handle_t client, esp_
         return ESP_ERR_INVALID_ARG;
     }
 #ifdef MQTT_SUPPORTED_FEATURE_EVENT_LOOP
-    if (client->config->event_handle) {
-        ESP_LOGW(TAG, "Unregistering event loop while event callback is not null, clearing callback");
-        client->config->event_handle = NULL;
-    }
 
     return esp_event_handler_unregister_with(client->config->event_loop_handle, MQTT_EVENTS, event, event_handler);
 #else