# HG changeset patch # User diego # Date 1181640565 0 # Node ID 4dbe6578f811ab24d8c5d20ada61a68c3271a3f3 # Parent 7982b376b58a8efa2876bbeeb40c803fa6110db0 misc spelling fixes diff -r 7982b376b58a -r 4dbe6578f811 apiexample.c --- a/apiexample.c Tue Jun 12 08:06:54 2007 +0000 +++ b/apiexample.c Tue Jun 12 09:29:25 2007 +0000 @@ -336,11 +336,11 @@ picture= avcodec_alloc_frame(); if(codec->capabilities&CODEC_CAP_TRUNCATED) - c->flags|= CODEC_FLAG_TRUNCATED; /* we dont send complete frames */ + c->flags|= CODEC_FLAG_TRUNCATED; /* we do not send complete frames */ - /* for some codecs, such as msmpeg4 and mpeg4, width and height - MUST be initialized there because these info are not available - in the bitstream */ + /* For some codecs, such as msmpeg4 and mpeg4, width and height + MUST be initialized there because this information is not + available in the bitstream. */ /* open it */ if (avcodec_open(c, codec) < 0) { diff -r 7982b376b58a -r 4dbe6578f811 audioconvert.c --- a/audioconvert.c Tue Jun 12 08:06:54 2007 +0000 +++ b/audioconvert.c Tue Jun 12 09:29:25 2007 +0000 @@ -54,8 +54,8 @@ }while(po < end);\ } -//FIXME put things below under ifdefs so we dont waste space for cases no codec will need -//FIXME rounding and cliping ? +//FIXME put things below under ifdefs so we do not waste space for cases no codec will need +//FIXME rounding and clipping ? CONV(SAMPLE_FMT_U8 , uint8_t, SAMPLE_FMT_U8 , *(uint8_t*)pi) else CONV(SAMPLE_FMT_S16, int16_t, SAMPLE_FMT_U8 , (*(uint8_t*)pi - 0x80)<<8) diff -r 7982b376b58a -r 4dbe6578f811 avcodec.h --- a/avcodec.h Tue Jun 12 08:06:54 2007 +0000 +++ b/avcodec.h Tue Jun 12 09:29:25 2007 +0000 @@ -2885,7 +2885,7 @@ /** * Parses \p str and put in \p width_ptr and \p height_ptr the detected values. * - * @return 0 in case of a successfull parsing, a negative value otherwise + * @return 0 in case of a successful parsing, a negative value otherwise * @param[in] str the string to parse: it has to be a string in the format * x or a valid video frame size abbreviation. * @param[in,out] width_ptr pointer to the variable which will contain the detected @@ -2898,7 +2898,7 @@ /** * Parses \p str and put in \p frame_rate the detected values. * - * @return 0 in case of a successfull parsing, a negative value otherwise + * @return 0 in case of a successful parsing, a negative value otherwise * @param[in] str the string to parse: it has to be a string in the format * /, a float number or a valid video rate abbreviation * @param[in,out] frame_rate pointer to the AVRational which will contain the detected diff -r 7982b376b58a -r 4dbe6578f811 beosthread.c --- a/beosthread.c Tue Jun 12 08:06:54 2007 +0000 +++ b/beosthread.c Tue Jun 12 09:29:25 2007 +0000 @@ -73,8 +73,8 @@ } /** - * free what has been allocated by avcodec_thread_init(). - * must be called after decoding has finished, especially dont call while avcodec_thread_execute() is running + * Free what has been allocated by avcodec_thread_init(). + * Must be called after decoding has finished, especially do not call while avcodec_thread_execute() is running. */ void avcodec_thread_free(AVCodecContext *s){ ThreadContext *c= s->thread_opaque; diff -r 7982b376b58a -r 4dbe6578f811 bitstream.h --- a/bitstream.h Tue Jun 12 08:06:54 2007 +0000 +++ b/bitstream.h Tue Jun 12 09:29:25 2007 +0000 @@ -335,8 +335,8 @@ } /** - * skips the given number of bits. - * must only be used if the actual values in the bitstream dont matter + * Skips the given number of bits. + * Must only be used if the actual values in the bitstream do not matter. */ static inline void skip_put_bits(PutBitContext *s, int n){ #ifdef ALT_BITSTREAM_WRITER diff -r 7982b376b58a -r 4dbe6578f811 cabac.h --- a/cabac.h Tue Jun 12 08:06:54 2007 +0000 +++ b/cabac.h Tue Jun 12 09:29:25 2007 +0000 @@ -683,7 +683,7 @@ } //FIXME the x86 code from this file should be moved into i386/h264 or cabac something.c/h (note ill kill you if you move my code away from under my fingers before iam finished with it!) -//FIXME use some macros to avoid duplicatin get_cabac (cant be done yet as that would make optimization work hard) +//FIXME use some macros to avoid duplicatin get_cabac (cannot be done yet as that would make optimization work hard) #if defined(ARCH_X86) && defined(CONFIG_7REGS) && defined(HAVE_EBX_AVAILABLE) && !defined(BROKEN_RELOCATIONS) static int decode_significance_x86(CABACContext *c, int max_coeff, uint8_t *significant_coeff_ctx_base, int *index){ void *end= significant_coeff_ctx_base + max_coeff - 1; diff -r 7982b376b58a -r 4dbe6578f811 dct-test.c --- a/dct-test.c Tue Jun 12 08:06:54 2007 +0000 +++ b/dct-test.c Tue Jun 12 09:29:25 2007 +0000 @@ -337,7 +337,7 @@ for(i=0; i<64; i++) block[i]= block1[i]; // memcpy(block, block1, sizeof(DCTELEM) * 64); -// dont memcpy especially not fastmemcpy because it does movntq !!! +// do not memcpy especially not fastmemcpy because it does movntq !!! fdct_func(block); } it1 += NB_ITS_SPEED; @@ -497,7 +497,7 @@ for(i=0; i<64; i++) block[i]= block1[i]; // memcpy(block, block1, sizeof(DCTELEM) * 64); -// dont memcpy especially not fastmemcpy because it does movntq !!! +// do not memcpy especially not fastmemcpy because it does movntq !!! idct248_put(img_dest, 8, block); } it1 += NB_ITS_SPEED; diff -r 7982b376b58a -r 4dbe6578f811 dsputil.h --- a/dsputil.h Tue Jun 12 08:06:54 2007 +0000 +++ b/dsputil.h Tue Jun 12 09:29:25 2007 +0000 @@ -144,7 +144,7 @@ /* motion estimation */ // h is limited to {width/2, width, 2*width} but never larger than 16 and never smaller then 2 -// allthough currently h<4 is not used as functions with width <8 are not used and neither implemented +// although currently h<4 is not used as functions with width <8 are neither used nor implemented typedef int (*me_cmp_func)(void /*MpegEncContext*/ *s, uint8_t *blk1/*align width (8 or 16)*/, uint8_t *blk2/*align 1*/, int line_size, int h)/* __attribute__ ((const))*/; diff -r 7982b376b58a -r 4dbe6578f811 error_resilience.c --- a/error_resilience.c Tue Jun 12 08:06:54 2007 +0000 +++ b/error_resilience.c Tue Jun 12 09:29:25 2007 +0000 @@ -108,7 +108,7 @@ } /** - * guess the dc of blocks which dont have a undamaged dc + * guess the dc of blocks which do not have an undamaged dc * @param w width in 8 pixel blocks * @param h height in 8 pixel blocks */ diff -r 7982b376b58a -r 4dbe6578f811 flicvideo.c --- a/flicvideo.c Tue Jun 12 08:06:54 2007 +0000 +++ b/flicvideo.c Tue Jun 12 09:29:25 2007 +0000 @@ -579,8 +579,8 @@ } /* Now FLX is strange, in that it is "byte" as opposed to "pixel" run length compressed. - * This doesnt give us any good oportunity to perform word endian conversion - * during decompression. So if its requried (ie, this isnt a LE target, we do + * This does not give us any good oportunity to perform word endian conversion + * during decompression. So if it is required (i.e., this is not a LE target, we do * a second pass over the line here, swapping the bytes. */ pixel = 0xFF00; @@ -716,11 +716,11 @@ buf, buf_size); } - /* Shouldnt get here, ever as the pix_fmt is processed */ + /* Should not get here, ever as the pix_fmt is processed */ /* in flic_decode_init and the above if should deal with */ /* the finite set of possibilites allowable by here. */ - /* but in case we do, just error out. */ - av_log(avctx, AV_LOG_ERROR, "Unknown Format of FLC. My Science cant explain how this happened\n"); + /* But in case we do, just error out. */ + av_log(avctx, AV_LOG_ERROR, "Unknown FLC format, my science cannot explain how this happened.\n"); return -1; } diff -r 7982b376b58a -r 4dbe6578f811 h261dec.c --- a/h261dec.c Tue Jun 12 08:06:54 2007 +0000 +++ b/h261dec.c Tue Jun 12 09:29:25 2007 +0000 @@ -532,7 +532,7 @@ */ static int get_consumed_bytes(MpegEncContext *s, int buf_size){ int pos= get_bits_count(&s->gb)>>3; - if(pos==0) pos=1; //avoid infinite loops (i doubt thats needed but ...) + if(pos==0) pos=1; //avoid infinite loops (i doubt that is needed but ...) if(pos+10>buf_size) pos=buf_size; // oops ;) return pos; @@ -565,7 +565,7 @@ return -1; } - //we need to set current_picture_ptr before reading the header, otherwise we cant store anyting im there + //we need to set current_picture_ptr before reading the header, otherwise we cannot store anyting im there if(s->current_picture_ptr==NULL || s->current_picture_ptr->data[0]){ int i= ff_find_unused_picture(s, 0); s->current_picture_ptr= &s->picture[i]; @@ -580,7 +580,7 @@ } if (s->width != avctx->coded_width || s->height != avctx->coded_height){ - ParseContext pc= s->parse_context; //FIXME move these demuxng hack to avformat + ParseContext pc= s->parse_context; //FIXME move this demuxing hack to libavformat s->parse_context.buffer=0; MPV_common_end(s); s->parse_context= pc; diff -r 7982b376b58a -r 4dbe6578f811 h263dec.c --- a/h263dec.c Tue Jun 12 08:06:54 2007 +0000 +++ b/h263dec.c Tue Jun 12 09:29:25 2007 +0000 @@ -141,7 +141,7 @@ if(pos<0) pos=0; // padding is not really read so this might be -1 return pos; }else{ - if(pos==0) pos=1; //avoid infinite loops (i doubt thats needed but ...) + if(pos==0) pos=1; //avoid infinite loops (i doubt that is needed but ...) if(pos+10>buf_size) pos=buf_size; // oops ;) return pos; @@ -382,7 +382,8 @@ return -1; } - //we need to set current_picture_ptr before reading the header, otherwise we cant store anyting im there + /* We need to set current_picture_ptr before reading the header, + * otherwise we cannot store anyting in there */ if(s->current_picture_ptr==NULL || s->current_picture_ptr->data[0]){ int i= ff_find_unused_picture(s, 0); s->current_picture_ptr= &s->picture[i]; @@ -621,7 +622,7 @@ ff_er_frame_start(s); //the second part of the wmv2 header contains the MB skip bits which are stored in current_picture->mb_type - //which isnt available before MPV_frame_start() + //which is not available before MPV_frame_start() if (s->msmpeg4_version==5){ if(!ENABLE_WMV2_DECODER || ff_wmv2_decode_secondary_picture_header(s) < 0) return -1; diff -r 7982b376b58a -r 4dbe6578f811 h264.c --- a/h264.c Tue Jun 12 08:06:54 2007 +0000 +++ b/h264.c Tue Jun 12 09:29:25 2007 +0000 @@ -1602,7 +1602,7 @@ return chroma_qp[av_clip(qscale + chroma_qp_index_offset, 0, 51)]; } -//FIXME need to check that this doesnt overflow signed 32 bit for low qp, i am not sure, it's very close +//FIXME need to check that this does not overflow signed 32 bit for low qp, i am not sure, it's very close //FIXME check that gcc inlines this (and optimizes intra & separate_dc stuff away) static inline int quantize_c(DCTELEM *block, uint8_t *scantable, int qscale, int intra, int separate_dc){ int i; @@ -4816,7 +4816,7 @@ if(IS_INTRA_PCM(mb_type)){ unsigned int x, y; - // we assume these blocks are very rare so we dont optimize it + // We assume these blocks are very rare so we do not optimize it. align_get_bits(&s->gb); // The pixels are stored in the same order as levels in h->mb array. @@ -5957,7 +5957,7 @@ const uint8_t *ptr; unsigned int x, y; - // We assume these blocks are very rare so we dont optimize it. + // We assume these blocks are very rare so we do not optimize it. // FIXME The two following lines get the bitstream position in the cabac // decode, I think it should be done by a function in cabac.h (or cabac.c). ptr= h->cabac.bytestream; @@ -7721,7 +7721,7 @@ buf_index += consumed; - if( (s->hurry_up == 1 && h->nal_ref_idc == 0) //FIXME dont discard SEI id + if( (s->hurry_up == 1 && h->nal_ref_idc == 0) //FIXME do not discard SEI id ||(avctx->skip_frame >= AVDISCARD_NONREF && h->nal_ref_idc == 0)) continue; @@ -7818,7 +7818,7 @@ return pos; }else{ - if(pos==0) pos=1; //avoid infinite loops (i doubt thats needed but ...) + if(pos==0) pos=1; //avoid infinite loops (i doubt that is needed but ...) if(pos+10>buf_size) pos=buf_size; // oops ;) return pos; @@ -8084,7 +8084,7 @@ START_TIMER j= get_ue_golomb(&gb); if(j != i){ - printf("missmatch! at %d (%d should be %d) bits:%6X\n", i, j, i, s); + printf("mismatch! at %d (%d should be %d) bits:%6X\n", i, j, i, s); // return -1; } STOP_TIMER("get_ue_golomb"); @@ -8109,7 +8109,7 @@ START_TIMER j= get_se_golomb(&gb); if(j != i - COUNT/2){ - printf("missmatch! at %d (%d should be %d) bits:%6X\n", i, j, i, s); + printf("mismatch! at %d (%d should be %d) bits:%6X\n", i, j, i, s); // return -1; } STOP_TIMER("get_se_golomb"); @@ -8213,7 +8213,7 @@ } if(memcmp(bitstream, out, COUNT)){ - printf("missmatch\n"); + printf("mismatch\n"); return -1; } } diff -r 7982b376b58a -r 4dbe6578f811 h264.h --- a/h264.h Tue Jun 12 08:06:54 2007 +0000 +++ b/h264.h Tue Jun 12 09:29:25 2007 +0000 @@ -34,7 +34,7 @@ #include "mpegvideo.h" #define interlaced_dct interlaced_dct_is_a_bad_name -#define mb_intra mb_intra_isnt_initalized_see_mb_type +#define mb_intra mb_intra_is_not_initalized_see_mb_type #define LUMA_DC_BLOCK_INDEX 25 #define CHROMA_DC_BLOCK_INDEX 26 diff -r 7982b376b58a -r 4dbe6578f811 i386/dsputil_mmx_avg.h --- a/i386/dsputil_mmx_avg.h Tue Jun 12 08:06:54 2007 +0000 +++ b/i386/dsputil_mmx_avg.h Tue Jun 12 09:29:25 2007 +0000 @@ -96,7 +96,7 @@ "add $16, %2 \n\t" "subl $4, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) @@ -143,7 +143,7 @@ "add $32, %2 \n\t" "subl $4, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) @@ -213,7 +213,7 @@ "add $32, %2 \n\t" "subl $4, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) @@ -268,7 +268,7 @@ "add $16, %2 \n\t" "subl $4, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) @@ -320,7 +320,7 @@ "add $32, %2 \n\t" "subl $4, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) @@ -408,7 +408,7 @@ "add $32, %2 \n\t" "subl $2, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) @@ -462,7 +462,7 @@ "add $32, %2 \n\t" "subl $2, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) @@ -535,7 +535,7 @@ "add $32, %2 \n\t" "subl $2, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) @@ -748,7 +748,8 @@ :"%"REG_a, "memory"); } -// Note this is not correctly rounded, but this function is only used for b frames so it doesnt matter +/* Note this is not correctly rounded, but this function is only + * used for B-frames so it does not matter. */ static void DEF(avg_pixels8_xy2)(uint8_t *block, const uint8_t *pixels, int line_size, int h) { MOVQ_BONE(mm6); diff -r 7982b376b58a -r 4dbe6578f811 i386/dsputil_mmx_rnd.h --- a/i386/dsputil_mmx_rnd.h Tue Jun 12 08:06:54 2007 +0000 +++ b/i386/dsputil_mmx_rnd.h Tue Jun 12 09:29:25 2007 +0000 @@ -98,7 +98,7 @@ "add %5, %3 \n\t" "subl $4, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) @@ -193,7 +193,7 @@ "add $32, %2 \n\t" "subl $2, %0 \n\t" "jnz 1b \n\t" -#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cant be used +#ifdef PIC //Note "+bm" and "+mb" are buggy too (with gcc 3.2.2 at least) and cannot be used :"+m"(h), "+a"(src1), "+c"(src2), "+d"(dst) #else :"+b"(h), "+a"(src1), "+c"(src2), "+d"(dst) diff -r 7982b376b58a -r 4dbe6578f811 i386/fdct_mmx.c --- a/i386/fdct_mmx.c Tue Jun 12 08:06:54 2007 +0000 +++ b/i386/fdct_mmx.c Tue Jun 12 09:29:25 2007 +0000 @@ -475,7 +475,7 @@ static av_always_inline void fdct_row_mmx(const int16_t *in, int16_t *out, const int16_t *table) { -//FIXME reorder (i dont have a old mmx only cpu here to benchmark ...) +//FIXME reorder (I do not have an old MMX-only CPU here to benchmark ...) movd_m2r(*(in + 6), mm1); punpcklwd_m2r(*(in + 4), mm1); movq_r2r(mm1, mm2); diff -r 7982b376b58a -r 4dbe6578f811 i386/mpegvideo_mmx.c --- a/i386/mpegvideo_mmx.c Tue Jun 12 08:06:54 2007 +0000 +++ b/i386/mpegvideo_mmx.c Tue Jun 12 09:29:25 2007 +0000 @@ -397,7 +397,7 @@ : "%"REG_a, "memory" ); block[0]= block0; - //Note, we dont do mismatch control for intra as errors cannot accumulate + //Note, we do not do mismatch control for intra as errors cannot accumulate } static void dct_unquantize_mpeg2_inter_mmx(MpegEncContext *s, diff -r 7982b376b58a -r 4dbe6578f811 i386/mpegvideo_mmx_template.c --- a/i386/mpegvideo_mmx_template.c Tue Jun 12 08:06:54 2007 +0000 +++ b/i386/mpegvideo_mmx_template.c Tue Jun 12 09:29:25 2007 +0000 @@ -103,7 +103,7 @@ assert((7&(int)(&temp_block[0])) == 0); //did gcc align it correctly? //s->fdct (block); - RENAMEl(ff_fdct) (block); //cant be anything else ... + RENAMEl(ff_fdct) (block); //cannot be anything else ... if(s->dct_error_sum) s->denoise_dct(s, block); diff -r 7982b376b58a -r 4dbe6578f811 jpeglsenc.c --- a/jpeglsenc.c Tue Jun 12 08:06:54 2007 +0000 +++ b/jpeglsenc.c Tue Jun 12 09:29:25 2007 +0000 @@ -381,7 +381,7 @@ return 0; } -AVCodec jpegls_encoder = { //FIXME avoid MPV_* lossless jpeg shouldnt need them +AVCodec jpegls_encoder = { //FIXME avoid MPV_* lossless JPEG should not need them "jpegls", CODEC_TYPE_VIDEO, CODEC_ID_JPEGLS, diff -r 7982b376b58a -r 4dbe6578f811 libmp3lame.c --- a/libmp3lame.c Tue Jun 12 08:06:54 2007 +0000 +++ b/libmp3lame.c Tue Jun 12 09:29:25 2007 +0000 @@ -188,7 +188,7 @@ s->buffer_index -= len; memmove(s->buffer, s->buffer+len, s->buffer_index); - //FIXME fix the audio codec API, so we dont need the memcpy() + //FIXME fix the audio codec API, so we do not need the memcpy() /*for(i=0; iuvstride= 8*s->mb_width + 16; } - // 8x8 fullpel search would need a 4x4 chroma compare, which we dont have yet, and even if we had the motion estimation code doesnt expect it + /* 8x8 fullpel search would need a 4x4 chroma compare, which we do + * not have yet, and even if we had, the motion estimation code + * does not expect it. */ if(s->codec_id != CODEC_ID_SNOW){ if((c->avctx->me_cmp&FF_CMP_CHROMA)/* && !s->dsp.me_cmp[2]*/){ s->dsp.me_cmp[2]= zero_cmp; @@ -1980,7 +1982,7 @@ type |= CANDIDATE_MB_TYPE_BIDIR_I; } //FIXME something smarter - if(dmin>256*256*16) type&= ~CANDIDATE_MB_TYPE_DIRECT; //dont try direct mode if its invalid for this MB + if(dmin>256*256*16) type&= ~CANDIDATE_MB_TYPE_DIRECT; //do not try direct mode if it is invalid for this MB if(s->codec_id == CODEC_ID_MPEG4 && type&CANDIDATE_MB_TYPE_DIRECT && s->flags&CODEC_FLAG_MV0 && *(uint32_t*)s->b_direct_mv_table[xy]) type |= CANDIDATE_MB_TYPE_DIRECT0; #if 0 diff -r 7982b376b58a -r 4dbe6578f811 mp3_header_decompress_bsf.c --- a/mp3_header_decompress_bsf.c Tue Jun 12 08:06:54 2007 +0000 +++ b/mp3_header_decompress_bsf.c Tue Jun 12 09:29:25 2007 +0000 @@ -60,7 +60,7 @@ break; } if(bitrate_index == 30){ - av_log(avctx, AV_LOG_ERROR, "couldnt find bitrate_index\n"); + av_log(avctx, AV_LOG_ERROR, "Could not find bitrate_index.\n"); return -1; } diff -r 7982b376b58a -r 4dbe6578f811 mpeg12.c --- a/mpeg12.c Tue Jun 12 08:06:54 2007 +0000 +++ b/mpeg12.c Tue Jun 12 09:29:25 2007 +0000 @@ -1000,7 +1000,7 @@ /* store the vlc & sign at once */ put_bits(&s->pb, table_vlc[code][1]+1, (table_vlc[code][0]<<1) + sign); } else { - /* escape seems to be pretty rare <5% so i dont optimize it */ + /* escape seems to be pretty rare <5% so I do not optimize it */ put_bits(&s->pb, table_vlc[111][1], table_vlc[111][0]); /* escape: only clip in this case */ put_bits(&s->pb, 6, run); @@ -3178,18 +3178,18 @@ int mb_y= start_code - SLICE_MIN_START_CODE; if(s2->last_picture_ptr==NULL){ - /* skip b frames if we dont have reference frames */ + /* Skip B-frames if we do not have reference frames. */ if(s2->pict_type==B_TYPE) break; - /* skip P frames if we dont have reference frame no valid header */ + /* Skip P-frames if we do not have reference frame no valid header. */ // if(s2->pict_type==P_TYPE && s2->first_field && !s2->first_slice) break; } - /* skip b frames if we are in a hurry */ + /* Skip B-frames if we are in a hurry. */ if(avctx->hurry_up && s2->pict_type==B_TYPE) break; if( (avctx->skip_frame >= AVDISCARD_NONREF && s2->pict_type==B_TYPE) ||(avctx->skip_frame >= AVDISCARD_NONKEY && s2->pict_type!=I_TYPE) || avctx->skip_frame >= AVDISCARD_ALL) break; - /* skip everything if we are in a hurry>=5 */ + /* Skip everything if we are in a hurry>=5. */ if(avctx->hurry_up>=5) break; if (!s->mpeg_enc_ctx_allocated) break; diff -r 7982b376b58a -r 4dbe6578f811 mpegaudioenc.c --- a/mpegaudioenc.c Tue Jun 12 08:06:54 2007 +0000 +++ b/mpegaudioenc.c Tue Jun 12 09:29:25 2007 +0000 @@ -470,7 +470,7 @@ sf[1] = sf[2] = sf[0]; break; default: - assert(0); //cant happen + assert(0); //cannot happen code = 0; /* kill warning */ } diff -r 7982b376b58a -r 4dbe6578f811 mpegvideo.c --- a/mpegvideo.c Tue Jun 12 08:06:54 2007 +0000 +++ b/mpegvideo.c Tue Jun 12 09:29:25 2007 +0000 @@ -372,7 +372,7 @@ * The pixels are allocated/set by calling get_buffer() if shared=0 */ static int alloc_picture(MpegEncContext *s, Picture *pic, int shared){ - const int big_mb_num= s->mb_stride*(s->mb_height+1) + 1; //the +1 is needed so memset(,,stride*height) doesnt sig11 + const int big_mb_num= s->mb_stride*(s->mb_height+1) + 1; //the +1 is needed so memset(,,stride*height) does not sig11 const int mb_array_size= s->mb_stride*s->mb_height; const int b8_array_size= s->b8_stride*s->mb_height*2; const int b4_array_size= s->b4_stride*s->mb_height*4; @@ -492,7 +492,7 @@ CHECKED_ALLOCZ(s->allocated_edge_emu_buffer, (s->width+64)*2*21*2); //(width + edge + align)*interlaced*MBsize*tolerance s->edge_emu_buffer= s->allocated_edge_emu_buffer + (s->width+64)*2*21; - //FIXME should be linesize instead of s->width*2 but that isnt known before get_buffer() + //FIXME should be linesize instead of s->width*2 but that is not known before get_buffer() CHECKED_ALLOCZ(s->me.scratchpad, (s->width+64)*4*16*2*sizeof(uint8_t)) s->rd_scratchpad= s->me.scratchpad; s->b_scratchpad= s->me.scratchpad; @@ -1633,7 +1633,7 @@ s->error_resilience= avctx->error_resilience; /* set dequantizer, we can't do it during init as it might change for mpeg4 - and we can't do it in the header decode as init isnt called for mpeg4 there yet */ + and we can't do it in the header decode as init is not called for mpeg4 there yet */ if(s->mpeg_quant || s->codec_id == CODEC_ID_MPEG2VIDEO){ s->dct_unquantize_intra = s->dct_unquantize_mpeg2_intra; s->dct_unquantize_inter = s->dct_unquantize_mpeg2_inter; diff -r 7982b376b58a -r 4dbe6578f811 mpegvideo.h --- a/mpegvideo.h Tue Jun 12 08:06:54 2007 +0000 +++ b/mpegvideo.h Tue Jun 12 09:29:25 2007 +0000 @@ -133,7 +133,7 @@ #define IS_ACPRED(a) ((a)&MB_TYPE_ACPRED) #define IS_QUANT(a) ((a)&MB_TYPE_QUANT) #define IS_DIR(a, part, list) ((a) & (MB_TYPE_P0L0<<((part)+2*(list)))) -#define USES_LIST(a, list) ((a) & ((MB_TYPE_P0L0|MB_TYPE_P1L0)<<(2*(list)))) ///< does this mb use listX, note doesnt work if subMBs +#define USES_LIST(a, list) ((a) & ((MB_TYPE_P0L0|MB_TYPE_P1L0)<<(2*(list)))) ///< does this mb use listX, note does not work if subMBs #define HAS_CBP(a) ((a)&MB_TYPE_CBP) int field_poc[2]; ///< h264 top/bottom POC @@ -161,9 +161,9 @@ typedef struct MotionEstContext{ AVCodecContext *avctx; int skip; ///< set if ME is skipped for the current MB - int co_located_mv[4][2]; ///< mv from last p frame for direct mode ME + int co_located_mv[4][2]; ///< mv from last P-frame for direct mode ME int direct_basis_mv[4][2]; - uint8_t *scratchpad; ///< data area for the me algo, so that the ME doesnt need to malloc/free + uint8_t *scratchpad; ///< data area for the ME algo, so that the ME does not need to malloc/free uint8_t *best_mb; uint8_t *temp_mb[2]; uint8_t *temp; @@ -248,8 +248,8 @@ /* sequence parameters */ int context_initialized; - int input_picture_number; ///< used to set pic->display_picture_number, shouldnt be used for/by anything else - int coded_picture_number; ///< used to set pic->coded_picture_number, shouldnt be used for/by anything else + int input_picture_number; ///< used to set pic->display_picture_number, should not be used for/by anything else + int coded_picture_number; ///< used to set pic->coded_picture_number, should not be used for/by anything else int picture_number; //FIXME remove, unclear definition int picture_in_gop_number; ///< 0-> first pic in gop, ... int b_frames_since_non_b; ///< used for encoding, relative to not yet reordered input diff -r 7982b376b58a -r 4dbe6578f811 msmpeg4.c --- a/msmpeg4.c Tue Jun 12 08:06:54 2007 +0000 +++ b/msmpeg4.c Tue Jun 12 09:29:25 2007 +0000 @@ -1006,7 +1006,7 @@ /* luminance h263 */ uni_code= DCtab_lum[size][0]; uni_len = DCtab_lum[size][1]; - uni_code ^= (1< 0) { uni_code<<=size; uni_code|=l; @@ -1022,7 +1022,7 @@ /* chrominance h263 */ uni_code= DCtab_chrom[size][0]; uni_len = DCtab_chrom[size][1]; - uni_code ^= (1< 0) { uni_code<<=size; uni_code|=l; @@ -1132,7 +1132,7 @@ break; } - s->slice_height= s->mb_height; //to avoid 1/0 if the first frame isnt a keyframe + s->slice_height= s->mb_height; //to avoid 1/0 if the first frame is not a keyframe return 0; } diff -r 7982b376b58a -r 4dbe6578f811 os2thread.c --- a/os2thread.c Tue Jun 12 08:06:54 2007 +0000 +++ b/os2thread.c Tue Jun 12 09:29:25 2007 +0000 @@ -62,7 +62,7 @@ /** * free what has been allocated by avcodec_thread_init(). - * must be called after decoding has finished, especially dont call while avcodec_thread_execute() is running + * must be called after decoding has finished, especially do not call while avcodec_thread_execute() is running */ void avcodec_thread_free(AVCodecContext *s){ ThreadContext *c= s->thread_opaque; diff -r 7982b376b58a -r 4dbe6578f811 ratecontrol.c --- a/ratecontrol.c Tue Jun 12 08:06:54 2007 +0000 +++ b/ratecontrol.c Tue Jun 12 09:29:25 2007 +0000 @@ -160,7 +160,7 @@ next= strchr(p, ';'); if(next){ - (*next)=0; //sscanf in unbelieavle slow on looong strings //FIXME copy / dont write + (*next)=0; //sscanf in unbelievably slow on looong strings //FIXME copy / do not write next++; } e= sscanf(p, " in:%d ", &picture_number); @@ -396,7 +396,7 @@ else if(q < last_q - maxdiff) q= last_q - maxdiff; } - rcc->last_qscale_for[pict_type]= q; //Note we cant do that after blurring + rcc->last_qscale_for[pict_type]= q; //Note we cannot do that after blurring if(pict_type!=B_TYPE) rcc->last_non_b_pict_type= pict_type; @@ -605,7 +605,7 @@ bits_tab[i]= bits; } - /* handle qmin/qmax cliping */ + /* handle qmin/qmax clipping */ if(s->flags&CODEC_FLAG_NORMALIZE_AQP){ float factor= bits_sum/cplx_sum; for(i=0; imb_num; i++){ diff -r 7982b376b58a -r 4dbe6578f811 snow.c --- a/snow.c Tue Jun 12 08:06:54 2007 +0000 +++ b/snow.c Tue Jun 12 09:29:25 2007 +0000 @@ -2112,7 +2112,7 @@ my= ref_my; } } - //FIXME if mb_cmp != SSE then intra cant be compared currently and mb_penalty vs. lambda2 + //FIXME if mb_cmp != SSE then intra cannot be compared currently and mb_penalty vs. lambda2 // subpel search base_bits= get_rac_count(&s->c) - 8*(s->c.bytestream - s->c.bytestream_start); @@ -2892,7 +2892,7 @@ } *b= backup; - return av_clip(((ab< 0 && (block->type&BLOCK_INTRA)){ int color0[3]= {block->color[0], block->color[1], block->color[2]}; check_block(s, mb_x, mb_y, color0, 1, *obmc_edged, &best_rd); diff -r 7982b376b58a -r 4dbe6578f811 svq1.c --- a/svq1.c Tue Jun 12 08:06:54 2007 +0000 +++ b/svq1.c Tue Jun 12 09:29:25 2007 +0000 @@ -499,7 +499,7 @@ int mvx= pmv[i]->x + (i&1)*16; int mvy= pmv[i]->y + (i>>1)*16; - ///XXX /FIXME cliping or padding? + ///XXX /FIXME clipping or padding? if(y + (mvy >> 1)<0) mvy= 0; if(x + (mvx >> 1)<0) diff -r 7982b376b58a -r 4dbe6578f811 svq3.c --- a/svq3.c Tue Jun 12 08:06:54 2007 +0000 +++ b/svq3.c Tue Jun 12 09:29:25 2007 +0000 @@ -891,11 +891,11 @@ s->current_picture.pict_type = s->pict_type; s->current_picture.key_frame = (s->pict_type == I_TYPE); - /* skip b frames if we dont have reference frames */ + /* Skip B-frames if we do not have reference frames. */ if (s->last_picture_ptr == NULL && s->pict_type == B_TYPE) return 0; - /* skip b frames if we are in a hurry */ + /* Skip B-frames if we are in a hurry. */ if (avctx->hurry_up && s->pict_type == B_TYPE) return 0; - /* skip everything if we are in a hurry >= 5 */ + /* Skip everything if we are in a hurry >= 5. */ if (avctx->hurry_up >= 5) return 0; if( (avctx->skip_frame >= AVDISCARD_NONREF && s->pict_type==B_TYPE) ||(avctx->skip_frame >= AVDISCARD_NONKEY && s->pict_type!=I_TYPE) @@ -993,7 +993,7 @@ avctx->frame_number = s->picture_number - 1; - /* dont output the last pic after seeking */ + /* Do not output the last pic after seeking. */ if (s->last_picture_ptr || s->low_delay) { *data_size = sizeof(AVFrame); } diff -r 7982b376b58a -r 4dbe6578f811 utils.c --- a/utils.c Tue Jun 12 08:06:54 2007 +0000 +++ b/utils.c Tue Jun 12 09:29:25 2007 +0000 @@ -273,7 +273,7 @@ pixel_size= picture.linesize[0]*8 / w; //av_log(NULL, AV_LOG_ERROR, "%d %d %d %d\n", (int)picture.data[1], w, h, s->pix_fmt); assert(pixel_size>=1); - //FIXME next ensures that linesize= 2^x uvlinesize, thats needed because some MC code assumes it + //FIXME next ensures that linesize= 2^x uvlinesize, that is needed because some MC code assumes it if(pixel_size == 3*8) w= ALIGN(w, STRIDE_ALIGN<current_picture_ptr==NULL || s->current_picture_ptr->data[0]){ int i= ff_find_unused_picture(s, 0); s->current_picture_ptr= &s->picture[i]; diff -r 7982b376b58a -r 4dbe6578f811 vorbis_enc.c --- a/vorbis_enc.c Tue Jun 12 08:06:54 2007 +0000 +++ b/vorbis_enc.c Tue Jun 12 09:29:25 2007 +0000 @@ -756,7 +756,7 @@ int maxval = 1; if (c->books[l] != -1) maxval = venc->codebooks[c->books[l]].nentries; - // coded could be -1, but this still works, cause thats 0 + // coded could be -1, but this still works, cause that is 0 if (coded[counter + k] < maxval) break; } assert(l != csub); diff -r 7982b376b58a -r 4dbe6578f811 w32thread.c --- a/w32thread.c Tue Jun 12 08:06:54 2007 +0000 +++ b/w32thread.c Tue Jun 12 09:29:25 2007 +0000 @@ -56,8 +56,8 @@ } /** - * free what has been allocated by avcodec_thread_init(). - * must be called after decoding has finished, especially dont call while avcodec_thread_execute() is running + * Free what has been allocated by avcodec_thread_init(). + * Must be called after decoding has finished, especially do not call while avcodec_thread_execute() is running. */ void avcodec_thread_free(AVCodecContext *s){ ThreadContext *c= s->thread_opaque; diff -r 7982b376b58a -r 4dbe6578f811 wmv2.c --- a/wmv2.c Tue Jun 12 08:06:54 2007 +0000 +++ b/wmv2.c Tue Jun 12 09:29:25 2007 +0000 @@ -194,8 +194,9 @@ return 0; } -// nearly idential to wmv1 but thats just because we dont use the useless M$ crap features -// its duplicated here in case someone wants to add support for these carp features +/* Nearly identical to wmv1 but that is just because we do not use the + * useless M$ crap features. It is duplicated here in case someone wants + * to add support for these crap features. */ void ff_wmv2_encode_mb(MpegEncContext * s, DCTELEM block[6][64], int motion_x, int motion_y)