examples/bluedroid/gattc_demo.c - Use correct struct type (IDFGH-9899)

[ckcr4lyf]

If the event is ESP_GATTC_DIS_SRVC_CMPL_EVT , then the struct should be gattc_dis_srvc_cmpl_evt_param as per

esp-idf/components/bt/host/bluedroid/api/include/api/esp_gattc_api.h

Lines 243 to 249 in f404fe9

	/**
	* @brief ESP_GATTC_DIS_SRVC_CMPL_EVT
	*/
	struct gattc_dis_srvc_cmpl_evt_param {
	esp_gatt_status_t status; /*!< Operation status */
	uint16_t conn_id; /*!< Connection id */
	} dis_srvc_cmpl; /*!< Gatt client callback param of ESP_GATTC_DIS_SRVC_CMPL_EVT */

Currently it is using gattc_cfg_mtu_evt_param , which is defined as

esp-idf/components/bt/host/bluedroid/api/include/api/esp_gattc_api.h

Lines 98 to 105 in f404fe9

	/**
	* @brief ESP_GATTC_CFG_MTU_EVT
	*/
	struct gattc_cfg_mtu_evt_param {
	esp_gatt_status_t status; /*!< Operation status */
	uint16_t conn_id; /*!< Connection id */
	uint16_t mtu; /*!< MTU size */
	} cfg_mtu; /*!< Gatt client callback param of ESP_GATTC_CFG_MTU_EVT */

Through coincidence, it does work since in memory the field .conn_id is at the same offset. However I think it is better to use the correct struct type , so it is less confusing to someone reading the example.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[ckcr4lyf]

I keep signing the CLA, it keeps removing it. Is this a bug?

[esp-cjh]

@ckcr4lyf We will incorporate your updates in the subsequent versions. Thanks.

[weiyuhannnn]

@ckcr4lyf Thanks for your contribution. The change you suggested has been merged with the following commits:

• master / v5.3 branch: be29217
• v5.2 : daa6a0d
• v5.1: 8519d3c
• v5.0: d388c85