Skip to content

fix: identify_display 中跟踪 _SetDeviceGammaRamp 返回值#2

Merged
SMWHff merged 1 commit into
mainfrom
worktree-agent-a153e1db
Jul 9, 2026
Merged

fix: identify_display 中跟踪 _SetDeviceGammaRamp 返回值#2
SMWHff merged 1 commit into
mainfrom
worktree-agent-a153e1db

Conversation

@SMWHff

@SMWHff SMWHff commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

修复 identify_display() 在闪烁循环中忽略 _SetDeviceGammaRamp 返回值的问题。

  • 在闪烁循环中检查每次 _SetDeviceGammaRamp 的返回值
  • 如果闪白屏或闪黑屏的调用失败,记录警告日志并跳出循环
  • 原有 finally 块中的恢复逻辑保持不变
  • 失败后最终返回 False 而非 True

测试

  • 新增 test_identify_flash_fails:模拟第一次闪白屏调用失败
  • 新增 test_identify_blank_fails:模拟第一次闪黑屏调用失败
  • 全部 402 项测试通过,ruff lint 检查通过

闪烁循环中之前忽略了 SetDeviceGammaRamp 的返回值,如果伽马写入被
静默拒绝(如远程桌面、权限不足),屏幕不会闪烁但函数仍返回 True。

修复后,在循环中检查每次调用的返回值,失败时记录警告并 break,
最终恢复后返回 False。
@SMWHff SMWHff merged commit ff94034 into main Jul 9, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant