SDK improvement suggestion - BLE_GATT_DB_MAX_CHARS, BLE_DB_DISCOVERY_MAX_SRV

[Richard D.]

ble_gatt_db.h and ble_db_discovery.h each contain #defines that are problematic, BLE_GATT_DB_MAX_CHARS and BLE_DB_DISCOVERY_MAX_SRV respectively.

Of course, one can change those values to suit the application but there are 2 drawbacks to this approach:
1. We have multiple applications with different requirements sharing the same SDK.
2. Changing SDK code invites problems when updating to a new SDK version.

A simple solution would be to move the #defines to sdk_config.h

Alternatively, allow one to define it without modifying the sdk
#ifndef BLE_GATT_DB_MAX_CHARS
#define BLE_GATT_DB_MAX_CHARS 5 /**< The maximum number of characteristics present in a service record. */
#endif

[Mttrinh]

Hi,

I have reported this back to our SDK team. They are aware of the problem and they plan to fix it but it won't be fixed in the next SDK. 

We appreciate the feedback. 

[DuaneS]

Not being able to override BLE_DB_DISCOVERY_MAX_SRV in sdk_config.h is a royal pain. This is a 5 minute fix why 3 years and still there?