-
Notifications
You must be signed in to change notification settings - Fork 136
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Small cleanups and feedback from @mounirlamouri #109
Conversation
FYI @mounirlamouri |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks.
index.src.html
Outdated
@@ -73,23 +76,18 @@ | |||
<dfn>control messages</dfn>. | |||
|
|||
<dfn lt="Enqueues a control message|Queue a control message">Queuing a control | |||
message</dfn> | |||
means adding the message to the end of a codec’s <a>control | |||
message</dfn> means enqueing the message to the codec’s <a>control |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm ok with changing the prose here, but then you need to explain what enqueueing and dequeuing mean, or (and I think it would be better) to refer to https://infra.spec.whatwg.org/#queues.
My original prose in the Web Audio API text was written in a way that this was unnecessary, and predates this infra
text.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
Thanks Paul |
Hm, IPR is unhappy, but that seems like a bug (I'm still in the w3c incubator group). Filed an issue on them w3c/ash-nazg#188 I'll go ahead with this merge. |
This fixes a handful of formatting/style issues and attempts to rely more on concepts from the infra spec.
Still to come, I'm working on PR to refactor the relationship of methods/algorithms for the codec interfaces. What I have currently reduces copy paste, but its a bit hard to read.
Preview | Diff