[FFmpeg-devel] [PATCH] Simplify getSubSampleFactors()

Stefano Sabatini stefano.sabatini-lala
Sat Jan 2 12:29:35 CET 2010


Hi,

this is indeed a full rewrite using the information provided in
pixdescs.

This is the result of the test:
Different value for gray: [1, 1] != [0, 0]
Different value for yuvj420p: [0, 0] != [1, 1]
Different value for yuvj422p: [0, 0] != [1, 0]
Different value for uyyvyy411: [0, 0] != [2, 0]
Different value for gray16be: [1, 1] != [0, 0]
Different value for gray16le: [1, 1] != [0, 0]
Different value for yuvj440p: [0, 0] != [0, 1]
Different value for vdpau_h264: [0, 0] != [1, 1]
Different value for vdpau_mpeg1: [0, 0] != [1, 1]
Different value for vdpau_mpeg2: [0, 0] != [1, 1]
Different value for vdpau_wmv3: [0, 0] != [1, 1]
Different value for vdpau_vc1: [0, 0] != [1, 1]
Different value for vaapi_moco: [0, 0] != [1, 1]
Different value for vaapi_idct: [0, 0] != [1, 1]
Different value for vaapi_vld: [0, 0] != [1, 1]
Different value for vdpau_mpeg4: [0, 0] != [1, 1]

Apart from the different values for the HW accel formats which are
simply missing in the current version, the differences:
Different value for yuvj420p: [0, 0] != [1, 1]
Different value for yuvj422p: [0, 0] != [1, 0]
Different value for uyyvyy411: [0, 0] != [2, 0]
Different value for yuvj440p: [0, 0] != [0, 1]

seem simply due the fact that the function has not been updated,
while I cannot make sense of the wrong values which are provided for
the gray8 and gray16 formats.

Regression test passed.

Regards.
-- 
FFmpeg = Faithful and Freak Magnificient Power EniGma
-------------- next part --------------
A non-text attachment was scrubbed...
Name: simplify-get-sub-sample-factors.patch
Type: text/x-diff
Size: 1357 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100102/5cc7bbe4/attachment.patch>



More information about the ffmpeg-devel mailing list