RE: NCS: Bluetooth connection context library

db_lw — Wed, 23 Mar 2022 08:32:51 GMT

Hello Susheel,

thank you for the reply.

Ok, good to know.

Best regards,
David

RE: NCS: Bluetooth connection context library

Susheel Nuguru — Wed, 23 Mar 2022 08:26:09 GMT

It looks like it is intentional and is supposed to give you the array size to be able search the whole index to get the context by id , like below.

	const size_t contexts = bt_conn_ctx_count(hids_obj->conn_ctx);

	for (size_t i = 0; i < contexts; i++) {
		const struct bt_conn_ctx *ctx =
		    bt_conn_ctx_get_by_id(hids_obj->conn_ctx, i);

		if (ctx) {

Yes, it looks inefficient, memory wise to have fewer max clients and more array space but do not think this will cause any functional problems in the search for context by Id as the id match will happen before the search hits the tail of the array elements. But yes, this looks like an item for optimization.