2008/8/15 Michael Niedermayer <[EMAIL PROTECTED]>:
> On Fri, Aug 15, 2008 at 01:47:53PM +0200, superdump wrote:
>> Author: superdump
>> Date: Fri Aug 15 13:47:52 2008
>> New Revision: 3325
>>
>> Log:
>> Swap logic of tns_max_order condition for clarity
>>
>>
>> Modified:
>> aac/aac.c
>>
>> Modified: aac/aac.c
>> ==============================================================================
>> --- aac/aac.c (original)
>> +++ aac/aac.c Fri Aug 15 13:47:52 2008
>> @@ -738,7 +738,12 @@ static int decode_tns(AACContext * ac, T
>> for (filt = 0; filt < tns->n_filt[w]; filt++) {
>> tns->length[w][filt] = get_bits(gb, 6 - 2*is8);
>>
>> - if ((tns->order[w][filt] = get_bits(gb, 5 - 2*is8)) <=
>> tns_max_order) {
>> + if ((tns->order[w][filt] = get_bits(gb, 5 - 2*is8)) >
>> tns_max_order) {
>> + av_log(ac->avccontext, AV_LOG_ERROR, "TNS filter order %d
>> is greater than maximum %d.",
>> + tns->order[w][filt], tns_max_order);
>> + tns->order[w][filt] = 0;
>> + return -1;
>> + } else {
>
> you dont need the else (because of the return) and also consistency wise
> the code tends not to have else after error checks
Of course. I'm a bit tired this morning.
Rob
_______________________________________________
FFmpeg-soc mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
