On Wed, Aug 02, 2017 at 06:08:28PM +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. >
> Also add simple FATE tests, based on output produced by the NDI SDK. this line belongs to the ther patch > --- > libavcodec/speedhq.c | 15 +++++++++------ > 1 file changed, 9 insertions(+), 6 deletions(-) > > diff --git a/libavcodec/speedhq.c b/libavcodec/speedhq.c > index 60efb0222b..eca45beb67 100644 > --- a/libavcodec/speedhq.c > +++ b/libavcodec/speedhq.c > @@ -448,12 +448,15 @@ static int speedhq_decode_frame(AVCodecContext *avctx, > frame->key_frame = 1; > > if (second_field_offset == 4) { > - /* > - * Overlapping first and second fields is used to signal > - * encoding only a single field (the second field then comes > - * as a separate, later frame). > - */ > - frame->height >>= 1; > + /* tabs are not allowed in ffmpeg git (except makefiles) [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Take away the freedom of one citizen and you will be jailed, take away the freedom of all citizens and you will be congratulated by your peers in Parliament.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel