From bbc5914e63275138526b00cdc5fa38ac09de9726 Mon Sep 17 00:00:00 2001 From: Mahavir Jain Date: Wed, 21 Dec 2022 11:33:04 +0530 Subject: [PATCH] esp_http_client: remove "Wno-format" cflag and fix formatting errors --- components/esp_http_client/CMakeLists.txt | 2 -- components/esp_http_client/esp_http_client.c | 3 ++- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/components/esp_http_client/CMakeLists.txt b/components/esp_http_client/CMakeLists.txt index 3a2dcef3e6..cc1f844cff 100644 --- a/components/esp_http_client/CMakeLists.txt +++ b/components/esp_http_client/CMakeLists.txt @@ -7,5 +7,3 @@ idf_component_register(SRCS "esp_http_client.c" # lwip is a public requirement because esp_http_client.h includes sys/socket.h REQUIRES lwip PRIV_REQUIRES tcp_transport http_parser) - -target_compile_options(${COMPONENT_LIB} PRIVATE "-Wno-format") diff --git a/components/esp_http_client/esp_http_client.c b/components/esp_http_client/esp_http_client.c index ca8fe9eccb..681bfef2ed 100644 --- a/components/esp_http_client/esp_http_client.c +++ b/components/esp_http_client/esp_http_client.c @@ -6,6 +6,7 @@ #include +#include #include "esp_system.h" #include "esp_log.h" @@ -253,7 +254,7 @@ static int http_on_headers_complete(http_parser *parser) client->response->data_offset = parser->nread; client->response->content_length = parser->content_length; client->response->data_process = 0; - ESP_LOGD(TAG, "http_on_headers_complete, status=%d, offset=%d, nread=%d", parser->status_code, client->response->data_offset, parser->nread); + ESP_LOGD(TAG, "http_on_headers_complete, status=%d, offset=%d, nread=%" PRId32, parser->status_code, client->response->data_offset, parser->nread); client->state = HTTP_STATE_RES_COMPLETE_HEADER; if (client->connection_info.method == HTTP_METHOD_HEAD) { /* In a HTTP_RESPONSE parser returning '1' from on_headers_complete will tell the