9 Dec 18:48
Re: [PATCH] Added Musepack support with libmpcdec
Ed Schouten <ed <at> 80386.nl>
2008-12-09 17:48:58 GMT
2008-12-09 17:48:58 GMT
Hello Maxime,
* Maxime COSTE <frrrwww <at> gmail.com> wrote:
> Here is a patch that add support for libmpcdec in herrie, allowing it to play
> musepack files.
>
> Tell me if I need to change anything.
Sorry for letting you wait this long - I've been very busy lately at
work/uni lately.
> + if (!fd->stream) {
> + fseek(fd->fp, 0, SEEK_END);
> + data->size = ftell(fd->fp);
> + fseek(fd->fp, 0, SEEK_SET);
> + } else
> + data->size = 0;
Herrie's streaming implementation is very primitive and has no way to
`taste' a stream to match with an audio module. We should actually have
a list of mime types, but I haven't found any time to do this. Maybe the
best thing would be to just return an error when you hit a stream? I
know, it's not very good, but right now it prevents MP3 streams from
being played.
What's your opinion on this matter?
--
--
Ed Schouten <ed <at> 80386.nl>
WWW: http://80386.nl/
(Continue reading)
RSS Feed