[FFmpeg-devel] [PATCH][VAAPI][4/6] Add MPEG-4 / H.263 bitstream decoding (take 7)

Gwenole Beauchesne gbeauchesne
Fri Mar 27 09:50:32 CET 2009


On Fri, 27 Mar 2009, Michael Niedermayer wrote:

> On Thu, Mar 26, 2009 at 11:44:50PM +0100, Gwenole Beauchesne wrote:
>> Le 26 mars 09 ? 22:36, Michael Niedermayer a ?crit :
>>
>>> On Thu, Mar 26, 2009 at 07:32:30PM +0100, Gwenole Beauchesne wrote:
>>>> Le 26 mars 09 ? 19:17, Michael Niedermayer a ?crit :
>>>>
>>>>> [...]
>>>>>> +/** Reconstruct bitstream num_gobs_in_vop */
>>>>>> +static inline int mpeg4_get_num_gobs_in_vop(MpegEncContext *s)
>>>>>> +{
>>>>>> +    static const uint16_t num_gobs_in_vop_table[8] =
>>>>>> +        { 0, 6, 9, 18, 18, 18, 0, 0 };
>>>>>> +    return num_gobs_in_vop_table[mpeg4_get_source_format(s)];
>>>>>> +}
>>>>>
>>>>> please remove these functions
>>>>
>>>> Then please approve
>>>> <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/2009-March/066392.html>
>>>> ;-)
>>>> I don't see how I can do otherwise.
>>>
>>> i cant approve it because it contains the same unacceptable 1 line
>>> functions
>>
>> Why? And, what do you suggest then?
>
> just write func1() instead of
>
> func2(){
>    func1();
> }
>
> func2();
>
> why? because the code is totally unreadable as is

Ah? Here is a new patch then...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg.hwaccel.vaapi.mpeg4.7.patch
Type: text/x-diff
Size: 11466 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090327/81653072/attachment.patch>



More information about the ffmpeg-devel mailing list