On Sat, Mar 01, 2014 at 07:44:01PM +0100, Keiji Costantini wrote:
> --- a/libavcodec/rv10.c
> +++ b/libavcodec/rv10.c
> @@ -304,10 +304,14 @@ static int rv20_decode_picture_header(RVDecContext *rv)
> switch (i) {
> - case 0: s->pict_type = AV_PICTURE_TYPE_I; break;
> - case 1: s->pict_type = AV_PICTURE_TYPE_I; break; //hmm ...
> - case 2: s->pict_type = AV_PICTURE_TYPE_P; break;
> - case 3: s->pict_type = AV_PICTURE_TYPE_B; break;
> + case 0: s->pict_type = AV_PICTURE_TYPE_I;
> + break;
> + case 1: s->pict_type = AV_PICTURE_TYPE_I;
> + break; //hmm ...
> + case 2: s->pict_type = AV_PICTURE_TYPE_P;
> + break;
> + case 3: s->pict_type = AV_PICTURE_TYPE_B;
> + break;
You should move all lines below the case label here.
Pushed with that and a few other nits fixed.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
