Skip to content
Success

Changes

Summary

  1. scripts/verify_log_statements: fix undefined name (details)
Commit c8ec5d58507718165b863c36b51e29e9afd1a4a1 by Oliver Smith
scripts/verify_log_statements: fix undefined name

Append it to "errors_found" instead of undefined variable "ignored".
This makes it work similarly to further above:

      # Skip 'LOG("bla" fmt )' strings that typically appear as #defines.
      if fmt_re.match(quoted):
        if debug:
          errors_found.append(error_found(f, log.start(), 'Skipping define', log.group(0)))
        continue

Fix the following bug:

  scripts/verify_log_statements.py:92:11: F821 Undefined name `ignored`
     |
  90 |         # hopefully eval broke because of some '## args' macro def
  91 |         if debug:
  92 |           ignored.append(error_found(f, log.start(), 'Ignoring', log.group(0)))
     |           ^^^^^^^ F821
  93 |         continue
     |

Change-Id: I4661d4677a1f0fc8d595985e73e2632b9a83b7d6
The file was modifiedscripts/verify_log_statements.py