[FFmpeg-devel] [PATCH] speedhq: fix behavior of single-field decoding

[Michael Niedermayer]

On Fri, Jul 28, 2017 at 11:57:24AM +0200, Steinar H. Gunderson wrote:
> The height convention for decoding frames with only a single field made sense
> for compatibility with legacy decoders, but doesn't really match the convention
> used by NDI, which is the primary (only?) user. Thus, change it to simply
> assuming that if the two fields overlap, the frame is meant to be a single
> field and the frame height matches the field height.
> ---
>  libavcodec/speedhq.c | 15 +++++++++------
>  1 file changed, 9 insertions(+), 6 deletions(-)

can you add a fate test for this ?

you also may want to add yourself to MAINTAINERs for speedhq.c
it seems you are the one primarly maintaining this code

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The bravest are surely those who have the clearest vision
of what is before them, glory and danger alike, and yet
notwithstanding go out to meet it. -- Thucydides
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170730/c51c0c92/attachment.sig>