[Ffmpeg-devel] [PATCH] support S302M streams in MPEG TS

Reimar Döffinger Reimar.Doeffinger
Tue Dec 12 20:27:43 CET 2006


I just noticed something that annoys me quite a bit:

On Fri, Dec 01, 2006 at 11:58:21AM +0100, Baptiste Coudurier wrote:
> +    if (s->frame_size > 0) {
> +        memcpy(s->inbuf_ptr, buf, buf_size);
> +        s->inbuf_ptr += buf_size;
> +        if (s->inbuf_ptr - s->inbuf == s->frame_size) {

Why this buffering? It costs speed, adds complexity and is completely
unneeded for the daud demuxer, isn't it?
That kind of thing (i.e. framing) is what AVParsers are usually there

Reimar D?ffinger

More information about the ffmpeg-devel mailing list