[Ffmpeg-devel] [PATCH] from DivX, Part 1: cosmectic changes

Steve Lhomme steve.lhomme
Fri Dec 16 21:31:11 CET 2005


Benjamin Larsson wrote:
> Hi,
> 
> Steve Lhomme wrote:
> 
> 
>>Hi everyone,
>>
>>This is the first patch from DivX in order to make our code as similar
>>as the official FFMPEG as possible.
>>
>>This patch includes only the cosmetic changes so that after that we
>>can proceed with the real juice (otherwise the noise from these
>>changes is too big to find the actual fixes).
>>
>>It has :
>>- tab removal (yes the official code has tabs)
>>- use av_log instead of fprintf
>>- missing includes (help MSVC at least and won't hurt anyone)
>>- a few #ifdef to hide code when it's not enabled in the main config
>>(to avoid having to fix MSVC portability issues)
>>- more PRId64, PRIx64, etc
>>
> 
> - asserts added

OK, it's not a cosmetic change then. But it doesn't change any 
bitstream. It just makes the code nicer & consistent.





More information about the ffmpeg-devel mailing list