[ogg-dev] [PATCH] skeleton.c
conrad at metadecks.org
Thu Jan 31 16:19:22 PST 2008
On 31/01/2008, ogg.k.ogg.k at googlemail.com <ogg.k.ogg.k at googlemail.com> wrote:
> not sure who to send this to, since it seems to be duplicated in both
> speex and liboggz,
> so I'm sending to the whole list.
thanks, this is an appropriate place for that patch. That skeleton.c
comes from a Summer of Code project which put skeleton support into
many projects; we'll need to apply this patch in those too.
> This fixes an off by one bug in the user of snprintf, and returns
> negative if writing the
> header returns negative (otherwise we'd just get a short write, losing
> the error).
> This patch isn't tested though, but I compiled it :)
ok, I'll test it out :-)
> Also, I've not fixed this (yet ?) but I believe the filling of the ogg
> packets is bogus as it
> will fill it with values in whatever endianness the current host has.
> It should be using
> oggpack_write and friends instead, I think. I may be missing something though.
eww, you're right, it's broken on big-endian hosts. I'll follow up
with a patch unless someone else does first ...
> I found this while having a look at liboggz, which Ivo pointed me to.
> This seems to do
> what ogg-tools do, only more and better, from the admittedly quick
> read I had of it..
> Are there plans to use these rather than ogg-tools as the "official"
> way of manipulating
> ogg streams ?
there's been a lot of contributions to oggz lately, more welcome too
:-) I guess the only reason we have separate projects for ogg-tools
and oggz-tools is library dependencies.
More information about the ogg-dev