Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Line 199, BLEScanner::checkReportForUuid -- Remove const specifier fr… #809

Closed
wants to merge 1 commit into from
Closed

Line 199, BLEScanner::checkReportForUuid -- Remove const specifier fr… #809

wants to merge 1 commit into from

Commits on Jan 30, 2024

  1. Line 199, BLEScanner::checkReportForUuid -- Remove const specifier fr… 

    …om uuid variable.

The local variable uuid in function bool BLEScanner::checkReportForUuid was defined as const, but it was not initialized with a value and is later assigned one (on lines 210 or 216). A const variable must be declared with an initial value, which cannot be changed. What is needed here is not a const value, but a cast where uuid was assigned a value. This was done correctly in one spot, but not on the other.
    @ScottCase-Stemco
    ScottCase-Stemco authored Jan 30, 2024
    Configuration menu
    Copy the full SHA
    89624eb View commit details
    Browse the repository at this point in the history