# HG changeset patch # User diego # Date 1216896936 0 # Node ID 6d718739ee8e58e66b2405fcc3f0e585019cbe61 # Parent 266d4949aa15322215c3318287a69ce10dfde8cb misc spelling/grammar fixes diff -r 266d4949aa15 -r 6d718739ee8e h264.c --- a/h264.c Thu Jul 24 10:53:32 2008 +0000 +++ b/h264.c Thu Jul 24 10:55:36 2008 +0000 @@ -133,7 +133,7 @@ topleft_xy -= s->mb_stride; } else if(bottom && curr_mb_frame_flag && !left_mb_frame_flag) { topleft_xy += s->mb_stride; - // take topleft mv from the middle of the mb, as opposed to all other modes which use the bottom-right partition + // take top left mv from the middle of the mb, as opposed to all other modes which use the bottom right partition topleft_partition = 0; } if (bottom @@ -292,7 +292,7 @@ 4 L . .L . . . . 5 L . .. . . . . */ -//FIXME constraint_intra_pred & partitioning & nnz (lets hope this is just a typo in the spec) +//FIXME constraint_intra_pred & partitioning & nnz (let us hope this is just a typo in the spec) if(top_type){ h->non_zero_count_cache[4+8*0]= h->non_zero_count[top_xy][4]; h->non_zero_count_cache[5+8*0]= h->non_zero_count[top_xy][5]; @@ -673,7 +673,7 @@ } /** - * gets the predicted number of non zero coefficients. + * gets the predicted number of non-zero coefficients. * @param n block index */ static inline int pred_non_zero_count(H264Context *h, int n){ @@ -728,7 +728,7 @@ if(MB_FIELD && !IS_INTERLACED(mb_types[h->left_mb_xy[0]]) && i >= scan8[0]+8){ - // leftshift will turn LIST_NOT_USED into PART_NOT_AVAILABLE, but that's ok. + // left shift will turn LIST_NOT_USED into PART_NOT_AVAILABLE, but that's OK. SET_DIAG_MV(/2, <<1, s->mb_x*4-1, (s->mb_y&~1)*4 - 1 + ((i-scan8[0])>>3)*2); } } @@ -1454,7 +1454,7 @@ } /** - * idct tranforms the 16 dc values and dequantize them. + * IDCT transforms the 16 dc values and dequantizes them. * @param qp quantization parameter */ static void h264_luma_dc_dequant_idct_c(DCTELEM *block, int qp, int qmul){ @@ -1486,7 +1486,7 @@ const int z2= temp[4*1+i] - temp[4*3+i]; const int z3= temp[4*1+i] + temp[4*3+i]; - block[stride*0 +offset]= ((((z0 + z3)*qmul + 128 ) >> 8)); //FIXME think about merging this into decode_resdual + block[stride*0 +offset]= ((((z0 + z3)*qmul + 128 ) >> 8)); //FIXME think about merging this into decode_residual block[stride*2 +offset]= ((((z1 + z2)*qmul + 128 ) >> 8)); block[stride*8 +offset]= ((((z1 - z2)*qmul + 128 ) >> 8)); block[stride*10+offset]= ((((z0 - z3)*qmul + 128 ) >> 8)); @@ -1495,7 +1495,7 @@ #if 0 /** - * dct tranforms the 16 dc values. + * DCT transforms the 16 dc values. * @param qp quantization parameter ??? FIXME */ static void h264_luma_dc_dct_c(DCTELEM *block/*, int qp*/){ @@ -1684,7 +1684,7 @@ const int pic_width = 16*s->mb_width; const int pic_height = 16*s->mb_height >> MB_FIELD; - if(!pic->data[0]) //FIXME this is unacceptable, some senseable error concealment must be done for missing reference frames + if(!pic->data[0]) //FIXME this is unacceptable, some sensible error concealment must be done for missing reference frames return; if(mx&7) extra_width -= 3; @@ -2226,7 +2226,7 @@ /* * MPV_frame_start uses pict_type to derive key_frame. * This is incorrect for H.264; IDR markings must be used. - * Zero here; IDR markings per slice in frame or fields are OR'd in later. + * Zero here; IDR markings per slice in frame or fields are ORed in later. * See decode_nal_units(). */ s->current_picture_ptr->key_frame= 0; @@ -2256,7 +2256,7 @@ // s->decode= (s->flags&CODEC_FLAG_PSNR) || !s->encoding || s->current_picture.reference /*|| h->contains_intra*/ || 1; - // We mark the current picture as non reference after allocating it, so + // We mark the current picture as non-reference after allocating it, so // that if we break out due to an error it can be released automatically // in the next MPV_frame_start(). // SVQ3 as well as most other codecs have only last/next/current and thus @@ -2467,7 +2467,7 @@ linesize = h->mb_linesize = s->linesize * 2; uvlinesize = h->mb_uvlinesize = s->uvlinesize * 2; block_offset = &h->block_offset[24]; - if(mb_y&1){ //FIXME move out of this func? + if(mb_y&1){ //FIXME move out of this function? dest_y -= s->linesize*15; dest_cb-= s->uvlinesize*7; dest_cr-= s->uvlinesize*7; @@ -2876,7 +2876,7 @@ int out_i; int limit= INT_MIN; - /* sort frame according to poc in B slice */ + /* sort frame according to POC in B slice */ for(out_i=0; out_ishort_ref_count; out_i++){ int best_i=INT_MIN; int best_poc=INT_MAX; @@ -2903,7 +2903,7 @@ tprintf(h->s.avctx, "current poc: %d, smallest_poc_greater_than_current: %d\n", s->current_picture_ptr->poc, smallest_poc_greater_than_current); - // find the largest poc + // find the largest POC for(list=0; list<2; list++){ int index = 0; int j= -99; @@ -3033,7 +3033,7 @@ print_short_term(h); print_long_term(h); - if(h->slice_type_nos==FF_I_TYPE) return 0; //FIXME move before func + if(h->slice_type_nos==FF_I_TYPE) return 0; //FIXME move before function for(list=0; listlist_count; list++){ memcpy(h->ref_list[list], h->default_ref_list[list], sizeof(Picture)*h->ref_count[list]); @@ -3078,7 +3078,7 @@ if(ref->data[0] != NULL && ref->frame_num == frame_num && (ref->reference & pic_structure) && - ref->long_ref == 0) // ignore non existing pictures by testing data[0] pointer + ref->long_ref == 0) // ignore non-existing pictures by testing data[0] pointer break; } if(i>=0) @@ -3920,7 +3920,7 @@ h->slice_type= slice_type; h->slice_type_nos= slice_type & 3; - s->pict_type= h->slice_type; // to make a few old func happy, it's wrong though + s->pict_type= h->slice_type; // to make a few old functions happy, it's wrong though if (s->pict_type == FF_B_TYPE && s0->last_picture_ptr == NULL) { av_log(h->s.avctx, AV_LOG_ERROR, "B picture before any references, skipping\n"); @@ -3933,13 +3933,13 @@ return -1; } if(!h0->pps_buffers[pps_id]) { - av_log(h->s.avctx, AV_LOG_ERROR, "non existing PPS referenced\n"); + av_log(h->s.avctx, AV_LOG_ERROR, "non-existing PPS referenced\n"); return -1; } h->pps= *h0->pps_buffers[pps_id]; if(!h0->sps_buffers[h->pps.sps_id]) { - av_log(h->s.avctx, AV_LOG_ERROR, "non existing SPS referenced\n"); + av_log(h->s.avctx, AV_LOG_ERROR, "non-existing SPS referenced\n"); return -1; } h->sps = *h0->sps_buffers[h->pps.sps_id]; @@ -4130,7 +4130,7 @@ h->redundant_pic_count= get_ue_golomb(&s->gb); } - //set defaults, might be overriden a few line later + //set defaults, might be overridden a few lines later h->ref_count[0]= h->pps.ref_count[0]; h->ref_count[1]= h->pps.ref_count[1]; @@ -4524,7 +4524,7 @@ /** * decodes a macroblock - * @returns 0 if ok, AC_ERROR / DC_ERROR / MV_ERROR if an error is noticed + * @returns 0 if OK, AC_ERROR / DC_ERROR / MV_ERROR if an error is noticed */ static int decode_mb_cavlc(H264Context *h){ MpegEncContext * const s = &h->s; @@ -5642,7 +5642,7 @@ /** * decodes a macroblock - * @returns 0 if ok, AC_ERROR / DC_ERROR / MV_ERROR if an error is noticed + * @returns 0 if OK, AC_ERROR / DC_ERROR / MV_ERROR if an error is noticed */ static int decode_mb_cabac(H264Context *h) { MpegEncContext * const s = &h->s; @@ -7696,7 +7696,7 @@ else if(err == 1) { /* Slice could not be decoded in parallel mode, copy down * NAL unit stuff to context 0 and restart. Note that - * rbsp_buffer is not transfered, but since we no longer + * rbsp_buffer is not transferred, but since we no longer * run in parallel mode this should not be an issue. */ h->nal_unit_type = hx->nal_unit_type; h->nal_ref_idc = hx->nal_ref_idc; @@ -7731,7 +7731,7 @@ s->flags= avctx->flags; s->flags2= avctx->flags2; - /* end of stream, output whats still in the buffers */ + /* end of stream, output what is still in the buffers */ if (buf_size == 0) { Picture *out; int i, out_idx;