[ogg-dev] [PATCH] oggzinfo: fix a couple leaks

ogg.k.ogg.k at googlemail.com ogg.k.ogg.k at googlemail.com
Mon Jun 2 03:38:13 PDT 2008


> thanks. I split it into two commits, the earlier of which could also
> be useful in other tools which use these skeleton.[ch] files:

Yes - I only did it for that skeleton.[ch] but it might be an idea to move these
into their own svn directory and pull it with an external, so they
stop diverging.

While there, I'd also suggest it could be useful to have a function like

const char *find_header(const fishead_packet *fp, const char *name)

which would be used as:

const char *length = find_header(fp, "Content-Length");
if (length) printf("%d bytes", atoi(length));

as it's a bit unwieldy to use at the moment.


More information about the ogg-dev mailing list