Skip to content

Conversation

@alabou
Copy link
Collaborator

@alabou alabou commented Nov 6, 2025

No description provided.

alabou and others added 2 commits November 6, 2025 10:55
Co-authored-by: cristian-recoseanu <[email protected]>
Co-authored-by: cristian-recoseanu <[email protected]>
alabou and others added 2 commits November 7, 2025 09:48
Co-authored-by: cristian-recoseanu <[email protected]>
Co-authored-by: cristian-recoseanu <[email protected]>
Co-authored-by: cristian-recoseanu <[email protected]>
@alabou
Copy link
Collaborator Author

alabou commented Nov 10, 2025

I want to make sure I understand the intent in using a sentence like "A singular capability value of true" ... This sentence does not imply that the capability supports only one value. The meaning must be "if the capability allows the value true then ..."

@pedro-alves-ferreira
Copy link

pedro-alves-ferreira commented Nov 10, 2025

@alabou I agree it might be a bit confusing.
What if we slightly rephrase it and remove the paragraph between the phrases that mention the singular and dual values, to clearly show that they all refer to the same:

A Receiver SHOULD provide a urn:x-nmos:cap:transport:hkep capability to indicate its support for Senders that use HDCP encryption and the HKEP protocol. If this capability has a singular value of true, that indicates support for HDCP encryption and the HKEP protocol. If this capability has a singular value of false, that indicates that they are not supported. Specifying both true and false values indicates the Receiver supports both HDCP encrypted and non-HDCP streams.

@alabou
Copy link
Collaborator Author

alabou commented Nov 11, 2025

I committed Pedro's version, which merges the two paragraphs. It seems to be our best attempt so far, but we can always revert to the previous one if necessary.

Copy link
Contributor

@cristian-recoseanu cristian-recoseanu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking great.

@alabou alabou merged commit 30e04b9 into v1.0-dev Nov 11, 2025
2 checks passed
@alabou alabou deleted the ARG-review branch November 11, 2025 14:59
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.

4 participants