-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Open
flutter/engine
#21927Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: contributor-productivityTeam-specific productivity, code health, technical debt.Team-specific productivity, code health, technical debt.engineflutter/engine related. See also e: labels.flutter/engine related. See also e: labels.from: sanitizerIssues caught by engine sanitizers. May be false positives.Issues caught by engine sanitizers. May be false positives.team-engineOwned by Engine teamOwned by Engine teamtriaged-engineTriaged by Engine teamTriaged by Engine team
Description
In flutter/vulkan/vulkan_window.cc, the call to GrDirectContext::MakeVulkan is undefined when SK_VULKAN is not defined.
If clang-tidy supports runs with a -D or similar mechanism for symbols, we could try running with that defined. Otherwise, we should disable linting immediately at the call site, for that line only.
Metadata
Metadata
Assignees
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: contributor-productivityTeam-specific productivity, code health, technical debt.Team-specific productivity, code health, technical debt.engineflutter/engine related. See also e: labels.flutter/engine related. See also e: labels.from: sanitizerIssues caught by engine sanitizers. May be false positives.Issues caught by engine sanitizers. May be false positives.team-engineOwned by Engine teamOwned by Engine teamtriaged-engineTriaged by Engine teamTriaged by Engine team