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

2017-08-02 Thread Steinar H. Gunderson
On Wed, Aug 02, 2017 at 03:18:49AM +0200, Michael Niedermayer wrote: > This seems to break a full "make fate" OK, removing the offending line and resending. /* Steinar */ -- Homepage: https://www.sesse.net/ ___ ffmpeg-devel mailing list

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

2017-08-01 Thread James Almer
On 8/1/2017 7:48 PM, 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 >

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

2017-08-01 Thread Michael Niedermayer
On Wed, Aug 02, 2017 at 12:51:17AM +0200, Steinar H. Gunderson wrote: > On Wed, Aug 02, 2017 at 12:48:38AM +0200, Steinar H. Gunderson wrote: > > Also add simple FATE tests, based on output produced by the NDI SDK. > > Here are the required samples. I couldn't find much documentation on how to >

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

2017-08-01 Thread Michael Niedermayer
On Wed, Aug 02, 2017 at 12:48:38AM +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

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

2017-08-01 Thread Steinar H. Gunderson
On Wed, Aug 02, 2017 at 12:48:38AM +0200, Steinar H. Gunderson wrote: > Also add simple FATE tests, based on output produced by the NDI SDK. Here are the required samples. I couldn't find much documentation on how to add tests to FATE, so everything has been done by cargo culting. /* Steinar */

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

2017-08-01 Thread Steinar H. Gunderson
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

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

2017-07-29 Thread 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

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

2017-07-28 Thread Steinar H. Gunderson
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