[xiph-rtp] Merge all metadata in one single packet.

[David Barrett]

How about packing the setup/codebook together into a single packet, but 
allowing comments to be sent at will?  In effect, we treat entirely 
separate the comment and setup/codebook packets.

Personally I don't use comments, but I think it's a good feature to keep 
(or, said another way, I don't see a compelling reason to remove it 
entirely).

-david

Luca Barbato wrote:
> Michael Smith wrote:
> 
>>
>> I think this is a bad idea. We should either disallow comment headers
>> altogether, or allow them at arbitrary points.
> 
> 
> I see.
> 
>> Given that we now have the opportunity to avoid this problem the
>> second time around, we should take it. Allowing comment headers at
>> arbitrary points solves the problem neatly, but your argument that
>> this should really be handled in a seperate stream is somewhat
>> compelling - hence my suggestion to just disallow them entirely.
> 
> 
> I'd like to remove them for sake of simplicity and because I don't see 
> any usage that couldn't be covered in other ways (if somebody know 
> speaks now, please).
> 
>>
>> Either way, packing the two really-mandatory setup packets (primary
>> and 'codebook') together seems reasonably sensible.
> 
> 
> I'll try something. Packing everything together is consistent with the 
> other non ogg usage of vorbis (matroska and ffmpeg families). Resending 
> everything just to change the comment informations is pretty overkill, 
> but updating the comment with the codebook and setup makes sense.
> 
> lu
>