feat(android): 支援未知 Canon 屬性診斷 - #124
Merged
Merged
Conversation
js051
marked this pull request as ready for review
August 9, 2026 17:26
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
變更內容
0xC189與0xC18A事件中的未知 vendor property 結構原因
新版韌體或其他 EOS 機型可能公告目前尚未映射的 Canon property。先前解析器會直接忽略這些事件,導致診斷無法區分「相機沒有公告」與「相機公告了未知結構」。本變更保留最小且不含內容的協定證據,讓後續真機回報可用於安全映射,同時避免洩漏相機 metadata 或把未證實指令暴露到一般 UI。
驗證
testDebugUnitTest: 386 tests,0 failureslintDebugassembleDebugCanonEosPtpTest與UsbPtpCameraBackendTest再次單獨通過