[Vorbis-dev] padding in comment header

[Ralph Giles]

On Sat, Feb 04, 2006 at 12:03:33PM +0000, Ian Malone wrote:

> I was thinking of looking at patching vorbiscomment to handle non-
> Vorbis I.

Yay! If it handled theora and speex comments too you could rename it 
oggcomment. :)

>  Since padding seems to be something people want (from
> looking at Hydrogen Audio, not the most representative sample) it
> seems worth considering it at the same time.  To be honest, to make
> it even noticeably useful, oggenc needs an add padding option as
> well (otherwise only people who re-tag large numbers of files twice
> will see any benefit).  It's not something that worries me really.
> If it is done it would probably be best as a non-default option for
> people who care enough to look into it.

That sounds like a wise approach.

> Actually, that's what lead to the next question, I was trying to get
> away without having to use the vorbiscomment repaging (repaginating?)
> in my toy padder (when the number of pages in the header changes).

I guess if it were me I'd write custom code for this so you have more
control, but it seems like libogg ought to support repagination through
the call sequence you're trying. It just may have to be fixed first!

You could check your implementation by adding a packetout -> packetin
loop between the pagein and pageout calls (this time to different
stream states). That will definitely work.

Cool that you're working on all this,
 -r