HID devices with large report maps don't allocate enough space?

[benjie-git]

My HID descriptor is large at 283 bytes.

It looks to me like the m_reportMapChr characteristic in NimBLEHIDDevice.cpp gets set up with a default max Characteristic size of 20. Does this need to be re-created with a bigger max size in NimBLEHIDDevice::setReportMap?
I get very strange behavior with inconsistent bond failures, which resolved when I change NimBLEHIDDevice.cpp, line 48 to:

    m_reportMapCharacteristic = m_hidService->createCharacteristic((uint16_t)0x2a4b, NIMBLE_PROPERTY::READ, 300);

to allocate 300 bytes for this characteristic's value.

Obviously this is not the right fix, but maybe we sould create the characteristic in setReportMap() with the necessary size, and then add a lazy call to createCharacteristic() on startServices() if setReportMap() was never called?

I'm still using NimBLE-Arduino 1.4.3 and esp32 3.1.0.

[h2zero]

All characteristics are created by default with a max value size of 512 bytes, your code would actually reduce the size. The value container will self adjust up to the max size provided.