Skip to content

[BUG] warnings building Bluefruit52Lib's central_ti_sensortag_optical example #380

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
henrygab opened this issue Nov 8, 2019 · 0 comments · Fixed by #386
Closed

[BUG] warnings building Bluefruit52Lib's central_ti_sensortag_optical example #380

henrygab opened this issue Nov 8, 2019 · 0 comments · Fixed by #386
Labels

Comments

@henrygab
Copy link
Collaborator

henrygab commented Nov 8, 2019

Describe the bug
Sample code currently emits warnings.

.../Bluefruit52Lib/examples/Central/central_ti_sensortag_optical/central_ti_sensortag_optical.ino:
    In function 'void connect_callback(uint16_t)':
.../Bluefruit52Lib/examples/Central/central_ti_sensortag_optical/central_ti_sensortag_optical.ino:171:11:
    warning: unused variable 'buffer' [-Wunused-variable]

   uint8_t buffer[BLE_GAP_ADV_SET_DATA_SIZE_MAX] = { 0 };
           ^~~~~~

Set up (please complete the following information)

To Reproduce
Steps to reproduce the behavior:

  1. Start a build based on PR 376
  2. Allow the build to complete
  3. Expand one of the three build jobs that enable additional warnings
  4. Review the build logs
    for the sample being fixed.

Expected behavior
Sample code should build without warnings.

@henrygab henrygab added the Bug label Nov 8, 2019
henrygab added a commit to henrygab/Adafruit_nRF52_Arduino that referenced this issue Nov 8, 2019
hathach added a commit that referenced this issue Nov 8, 2019
NaotoFujihiro pushed a commit to cami/ArduinoCore-nRF52 that referenced this issue Nov 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant