[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