# HG changeset patch # User michael # Date 1115464067 0 # Node ID d321fbf0b2ce627ba5fe8478b2e63a8d2f6b1f81 # Parent b2e6c5e9240b185cc821f07efc3563c996a73fa5 reverse 1 hunk from 1.96->1.97, fixes decoding of (MR1_BT_A.h264 and MR1_MW_A.264) diff -r b2e6c5e9240b -r d321fbf0b2ce h264.c --- a/h264.c Fri May 06 17:04:03 2005 +0000 +++ b/h264.c Sat May 07 11:07:47 2005 +0000 @@ -3301,21 +3301,31 @@ else pred+= abs_diff_pic_num; pred &= h->max_pic_num - 1; - for(i= h->short_ref_count-1; i>=0; i--){ - ref = h->short_ref[i]; - if(ref->data[0] != NULL && ref->frame_num == pred && ref->long_ref == 0) // ignore non existing pictures by testing data[0] pointer + for(i= h->ref_count[list]-1; i>=0; i--){ + if(h->ref_list[list][i].data[0] != NULL && h->ref_list[list][i].pic_id == pred && h->ref_list[list][i].long_ref==0) // ignore non existing pictures by testing data[0] pointer break; } }else{ pic_id= get_ue_golomb(&s->gb); //long_term_pic_idx - ref = h->long_ref[pic_id]; + + for(i= h->ref_count[list]-1; i>=0; i--){ + if(h->ref_list[list][i].pic_id == pic_id && h->ref_list[list][i].long_ref==1) // no need to ignore non existing pictures as non existing pictures have long_ref==0 + break; + } } if (i < 0) { av_log(h->s.avctx, AV_LOG_ERROR, "reference picture missing during reorder\n"); memset(&h->ref_list[list][index], 0, sizeof(Picture)); //FIXME - } else { - h->ref_list[list][index]= *ref; + } else if (i != index) /* this test is not necessary, it is only an optimisation to skip double copy of Picture structure in this case */ { + Picture tmp= h->ref_list[list][i]; + if (i < index) { + i = h->ref_count[list]; + } + for(; i > index; i--){ + h->ref_list[list][i]= h->ref_list[list][i-1]; + } + h->ref_list[list][index]= tmp; } }else{ av_log(h->s.avctx, AV_LOG_ERROR, "illegal reordering_of_pic_nums_idc\n");