[FFmpeg-devel] theora depayloader

Ronald S. Bultje rsbultje
Mon Mar 22 15:51:18 CET 2010


Hi,

On Mon, Mar 22, 2010 at 9:59 AM, Josh Allmann <joshua.allmann at gmail.com> wrote:
> Awesome, almost there...

Small nitty stuff then:
> +
> +static PayloadContext *theora_new_context(void)

one empty line too many.

> +{
> +    PayloadContext* data    = av_mallocz(sizeof(PayloadContext));
> +    data->fragment          = NULL;
> +    data->timestamp         = 0;
> +    return data;
> +}

mallocz() already zeroes, so NULL/0 lines aren't needed, simply return
av_mallocz(sizeof(PayloadContext));

Rest OK, if others are OK also, feel free to apply.

Ronald



More information about the ffmpeg-devel mailing list