[FFmpeg-devel] [PATCH][H264] Fix weight flags init in implicit_weight_table()

Gwenole Beauchesne gbeauchesne
Thu Feb 5 18:13:53 CET 2009


Hi,

On Tue, 3 Feb 2009, Carl Eugen Hoyos wrote:

>>> OK, patch attached. Also applied vertical alignment suggestions though
>>> coding style becomes less consistent IMHO: original and surrounding code is
>>> mis-aligned too so this new hunk looks "alone".
>>
>> patch looks ok
>
> Patch applied.

Thanks. I also noticed it would be better to move up initilization of the 
weight flags in implicit_weight_table() because we could return early. I 
don't have any testcase, it's just that I believe the attached patch gets 
it better.

Regards,
Gwenole.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 005_h264_move_up_weight_flags_init.patch
Type: text/x-diff
Size: 1216 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090205/1151f60a/attachment.patch>



More information about the ffmpeg-devel mailing list