Copilot commented on code in PR #3062:
URL: https://github.com/apache/tika/pull/3062#discussion_r3846569195
##########
tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-audiovideo-module/src/test/java/org/apache/tika/parser/mp4/MP4ParserTest.java:
##########
@@ -352,10 +365,12 @@ private Set<String> getVals(Metadata m, String k) {
@Test
public void testDrmProtectedM4a() throws Exception {
- //the sample description declares a protected 'drms' sample entry
+ //the sample description declares a protected 'drms' sample entry: the
+ //format is the protection scheme and has-drm is set
Review Comment:
Comment says “format is the protection scheme”, but the value being asserted
("drms") is the protected sample-entry type fourcc (the protection scheme type
is elsewhere, e.g., in `schm`). This wording is misleading and contradicts the
clarified terminology used elsewhere in this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
