文件名称:xvid 代码的几个小bug
文件大小:94KB
文件格式:HTM
更新时间:2013-05-19 05:12:30
xvid 代码\小bug
1. idct.c中void idct_int32(short *const block)函数 定义了几个中间变量 X0,X1,X2,X3,X4,X5...X8 变量类型使用了 static long 结果虽然不会出错,但是影响性能,因为static 变量在退出函数后还需要保留该变量的值,增加了运算量,而非static变量,则可以通过编译 器优化,能使该变量为寄存器变量,减少内存访问的时间。本人测试了2000帧QVGA(320x240)图像,其中static变量的idct耗时8.6s,而非 static的只要5.3s,可见对效率影响挺大。 不过,俺刚看了xvid 1.03版本已经将这个代码改过来了,xvid 1.01和0.91都是static的。 2.xvid_decraw.c中main函数中 if (already_in_buffer > 0) memcpy(mp4_buffer, mp4_ptr, already_in_buffer); /* Update mp4_ptr */ mp4_ptr = mp4_buffer; /* read new data */ if(feof(in_file)) break; 这段代码将导致,码流没有结束而提前退出程序的情况 因为在读文件的时候,文件结束并不代表,缓冲区中的数据也被读完了,缓冲区大小为 #define BUFFER_SIZE (2*1024*1024) //2兆啊 所以,即使遇到文件结束,也应该继续解码,直道already_in_buffer的值为0 3.bitstream.h中的static void __inline BitstreamInit()函数 ptr_t adjbitstream = (ptr_t)bitstream; /* * Start the stream on a uint32_t boundary, by rounding down to the * previous uint32_t and skipping the intervening bytes. */ bitpos = ((sizeof(uint32_t)-1) & (size_t)bitstream); adjbitstream = adjbitstream - bitpos; bs->start = bs->tail = (uint32_t *) adjbitstream; 该段代码,直接将位流指针设置成整型,在x86处理器下可以正常运行,而有些处理器是不支持在非32位对齐的地址做字(32bits)读取的,除非编译器做额外处理。 可以做改动的是,在传入参数时,就将其设置为字对齐的。如果是一个地址为addr,将其改为 addr&-4,低两位强制设0。这在xvid_decraw.c里可以修改