On Sunday, March 16, 2014, wm4 <[email protected]> wrote:

> From: Michael Niedermayer <[email protected] <javascript:;>>
>
> Signed-off-by: Michael Niedermayer <[email protected] <javascript:;>>
> ---
> Oops, that's my fault.
> ---
>  libavformat/isom.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)


Looks ok, but the title should be something like "isom: fix C99 style code"
and without the sign off.
Vittorio


>
> diff --git a/libavformat/isom.c b/libavformat/isom.c
> index e13f9cf..e15875e 100644
> --- a/libavformat/isom.c
> +++ b/libavformat/isom.c
> @@ -442,11 +442,12 @@ static int mov_rewrite_dvd_sub_extradata(AVStream
> *st)
>      char buf[256];
>      int pal_s_pos = 0;
>      uint8_t *src = st->codec->extradata;
> +    int i;
>
>      if (st->codec->extradata_size != 64)
>          return 0;
>
> -    for (int i = 0; i < 16; i++) {
> +    for (i = 0; i < 16; i++) {
>          uint32_t yuv = AV_RB32(src + i * 4);
>          uint32_t rgba = yuv_to_rgba(yuv);
>
> --
> 1.9.0
>
> _______________________________________________
> libav-devel mailing list
> [email protected] <javascript:;>
> https://lists.libav.org/mailman/listinfo/libav-devel
>
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to