search

w3c / media-source

Media Source Extensions
https://w3c.github.io/media-source/
Other
267 stars 59 forks source link

Modernize var syntax, including type info where known #285

Closed wolenetz closed 2 years ago

wolenetz commented 2 years ago

Updates <var>...</var> to instead use modern respec |...|, and in most cases, on first definition of a variable, includes type of variable as a suffix to ... in the definition.

Links IDL method arguments in the definition to the argument variable's detail by also wrapping those argument names in |...|.

Includes a couple other minor changes: uses minus instead of - in algorithms in a couple places.

Replaces a couple instances of "source buffer" with "{{SourceBuffer}}" for improved linkage.

Note, the MSE-in-Workers initial draft PR (#282) contains some similar changes to lines only relevant to that draft, and will likely need rebasing and manual merge resolution on top of this once this lands.

Preview | Diff

wolenetz commented 2 years ago

Note, https://github.com/w3c/media-source/commit/3775bc8b7e6304fe9f09878c01dfa2602b44fdff was a force push of the result of manual rebase over the top of the now-landed PR#282.

wolenetz commented 2 years ago

@mwatson2: since @tidoust has approved this and it's been out for review for a month, I'll merge it shortly. If you prefer different, we can revert and fix later.