Changes required to infrastructure file ble_glc.c attached

[dcollier8]

The issue is that I fixed some of your bugs in the attached file, as follows:

 

1. Changed to handle larger number of records greater than 256 records, we need 500 records capability, was 8-bit record counter, now is 16 bit records counter.
2. Fixed an endless loop issue in RACP code. The “first/last” RACP sending logic had an endless loop.
3. Changed m_next_seq_num to global variable to be global to access from other parts of the codebase. This is the record sequence number.
4. fix logic in glucose_meas_send call that never completes sending char notification, see lines 716-721

[dcollier8]

We found still more bugs in the RACP code, when sending all records, there are timing issues with TX recovery issues..

The problem was that no measurement context objects were being sent, the code was being forced to return prior to that, with an error code of zero, hiding the issue.

Once I fixed that, other errors started showing up, which we are fixing now.

We are finding more bugs by using a test suite, which tests more comprehensively.

So I will be updating the case with an updated fix file, hopefully this week.