Re: [libav-devel] [PATCH 14/25] intrax8: Pass the output frame to the decoding function

2016-03-22 Thread Vittorio Giovara 
On Tue, Mar 22, 2016 at 2:03 PM, Diego Biurrun  wrote:
> On Sat, Mar 19, 2016 at 05:42:45PM -0400, Vittorio Giovara wrote:
>> --- a/libavcodec/intrax8.c
>> +++ b/libavcodec/intrax8.c
>> @@ -827,7 +828,7 @@ int ff_intrax8_decode_picture(IntraX8Context *const w, 
>> int dquant,
>>  /* emulate MB info in the relevant tables */
>>  s->mbskip_table[mb_xy] = 0;
>>  s->mbintra_table[mb_xy]= 1;
>> -s->current_picture.qscale_table[mb_xy] = w->quant;
>> +pict->qscale_table[mb_xy] = w->quant;
>
> I'd maintain alignment here.

the above lines are removed in a later commit
-- 
Vittorio
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 14/25] intrax8: Pass the output frame to the decoding function

2016-03-22 Thread Diego Biurrun 
On Sat, Mar 19, 2016 at 05:42:45PM -0400, Vittorio Giovara wrote:
> --- a/libavcodec/intrax8.c
> +++ b/libavcodec/intrax8.c
> @@ -827,7 +828,7 @@ int ff_intrax8_decode_picture(IntraX8Context *const w, 
> int dquant,
>  /* emulate MB info in the relevant tables */
>  s->mbskip_table[mb_xy] = 0;
>  s->mbintra_table[mb_xy]= 1;
> -s->current_picture.qscale_table[mb_xy] = w->quant;
> +pict->qscale_table[mb_xy] = w->quant;

I'd maintain alignment here.

OK

Diego
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel