On Thu, Apr 14, 2011 at 01:32:43PM +0200, Luca Barbato wrote:
> [...]

.. some nits ..

> --- a/libavformat/flvdec.c
> +++ b/libavformat/flvdec.c
> @@ -309,7 +308,14 @@ static int flv_read_metabody(AVFormatContext *s, int64_t 
> next_pos) {
> -    if(type != AMF_DATA_TYPE_STRING || amf_get_string(ioc, buffer, 
> sizeof(buffer)) < 0 || strcmp(buffer, "onMetaData"))
> +    if (type != AMF_DATA_TYPE_STRING ||
> +        amf_get_string(ioc, buffer, sizeof(buffer)) < 0)
> +        return -1;
> +
> +    if(!strcmp(buffer, "data-stream"))
> +        return 1;
> +
> +    if(strcmp(buffer, "onMetaData"))

if (

> @@ -382,6 +388,55 @@ static int flv_get_extradata(AVFormatContext *s, 
> AVStream *st, int size)
>  
> +    for(i=0;i<s->nb_streams;i++) {

Give those operators some room to breathe.

> +    pkt->size = ret;
> +    pkt->dts = dts;
> +    pkt->pts = dts;
> +    pkt->stream_index = st->index;
> +    pkt->flags |= AV_PKT_FLAG_KEY;

align

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to