[xiph-rtp] vorbis-rtp update (almost candidate)

[Luca Barbato]

David Barrett wrote:
> Looks really good.  My only two questions/concerns:
> 
> - While I don't use the comment header, it seems like a key feature to
> Vorbis overall, and I'm not sure why we're dropping it entirely.  The
> comment packet seems useful both in obvious ways (updating what's
> displayed to users mid-stream) and non-obvious ways
> (application-specific metadata delivered inline via RTP).  I entirely
> see why it shouldn't be included with the packed setup/codebook headers,
> but I don't see why it should be suppressed altogether.

Metadata should be part of the container infrastructure, not the codec
one. The comment feature in ogg/vorbis is quite abused already and the
decision was to avoid the same issue for vorbis-rtp.


> 
> - I don't follow this segment:
> 
[snip]
> 
> 1) By "the configuration and codebook headers" do you actually mean "the
> setup and codebook headers"?
I mean Configuration Packet, looks like that one slipped the first cross
read, thank you =)
> 
> 2) By "using the configuration attribute" are you referring to the
> "configuration-uri" attribute, or some other "configuration" attribute?
>  Do you mean a base16-encoded (for example) insertion of the packed
> setup/codebook headers, or a link to where these can be downloaded?

The "configuration" attribute (if someone could come up with a better
name I'll change it immediately) is the parameter in which put the
base16 encoded Configuration Packets. I guess I should change the SDP
inline rule to use the same format used in the out of band delivery for
consistency.

> 
> 3) Typo in the third paragraph: "pointto".  Here it says I can download
> using a URI.  So does this mean there are actually three delivery
> vectors: inline, embeddded in the SDP, and via a URI?
> 

2 defined vectors: inline and embedded SDP
Many user defined vectors: rtsp based, http based, xmpp based... The
only constraint is that the medium is reliable.

lu

-- 

Luca Barbato

Gentoo/linux Developer		Gentoo/PPC Operational Leader
http://dev.gentoo.org/~lu_zero