Message ID | fa3001984e5ecf600283091c92d27fe1b9419edd.1584458872.git.porschegt23@foxmail.com |
---|---|
State | New |
Headers | show |
Series | [FFmpeg-devel,v3] examples/extract_mvs.c: don't disply motionless vectors | expand |
Context | Check | Description |
---|---|---|
andriy/ffmpeg-patchwork | success | Make fate finished |
LGTM without comments. On 3/17/20, numberwolf <porschegt23@foxmail.com> wrote: > Signed-off-by: numberwolf <porschegt23@foxmail.com> > --- > > > > Here, don't need show motionless vectors,because they're useless. > doc/examples/extract_mvs.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c > index de31ccd..f6d19b7 100644 > --- a/doc/examples/extract_mvs.c > +++ b/doc/examples/extract_mvs.c > @@ -60,6 +60,13 @@ static int decode_packet(const AVPacket *pkt) > const AVMotionVector *mvs = (const AVMotionVector > *)sd->data; > for (i = 0; i < sd->size / sizeof(*mvs); i++) { > const AVMotionVector *mv = &mvs[i]; > + > + // dont need show motionless vectors > + // because they're useless > + if (mv->src_x == mv->dst_x && mv->src_y == mv->dst_y) { > + continue; > + } > + > printf("%d,%2d,%2d,%2d,%4d,%4d,%4d,%4d,0x%"PRIx64"\n", > video_frame_count, mv->source, > mv->w, mv->h, mv->src_x, mv->src_y, > -- > 2.17.2 (Apple Git-113) > > Thanks > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
> LGTM without comments. > > On 3/17/20, numberwolf <porschegt23@foxmail.com> wrote: > > Signed-off-by: numberwolf <porschegt23@foxmail.com> > > --- > > > > > > > > Here, don't need show motionless vectors,because they're useless. > > doc/examples/extract_mvs.c | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c > > index de31ccd..f6d19b7 100644 > > --- a/doc/examples/extract_mvs.c > > +++ b/doc/examples/extract_mvs.c > > @@ -60,6 +60,13 @@ static int decode_packet(const AVPacket *pkt) > > const AVMotionVector *mvs = (const AVMotionVector > > *)sd->data; > > for (i = 0; i < sd->size / sizeof(*mvs); i++) { > > const AVMotionVector *mv = &mvs[i]; > > + > > + // dont need show motionless vectors > > + // because they're useless > > + if (mv->src_x == mv->dst_x && mv->src_y == mv->dst_y) { > > + continue; > > + } > > + > > printf("%d,%2d,%2d,%2d,%4d,%4d,%4d,%4d,0x%"PRIx64"\n", > > video_frame_count, mv->source, > > mv->w, mv->h, mv->src_x, mv->src_y, > > -- > > 2.17.2 (Apple Git-113) > > > > Thanks > > _______________________________________________ > > ffmpeg-devel mailing list > > ffmpeg-devel@ffmpeg.org > > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > To unsubscribe, visit link above, or email > > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ping. Thanks
Am Fr., 20. März 2020 um 17:59 Uhr schrieb porschegt23@foxmail.com <porschegt23@foxmail.com>: > > LGTM without comments. > > > ping. Please resend your patch without the comments (and the mail without the html codes). Carl Eugen
diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c index de31ccd..f6d19b7 100644 --- a/doc/examples/extract_mvs.c +++ b/doc/examples/extract_mvs.c @@ -60,6 +60,13 @@ static int decode_packet(const AVPacket *pkt) const AVMotionVector *mvs = (const AVMotionVector *)sd->data; for (i = 0; i < sd->size / sizeof(*mvs); i++) { const AVMotionVector *mv = &mvs[i]; + + // dont need show motionless vectors + // because they're useless + if (mv->src_x == mv->dst_x && mv->src_y == mv->dst_y) { + continue; + } + printf("%d,%2d,%2d,%2d,%4d,%4d,%4d,%4d,0x%"PRIx64"\n", video_frame_count, mv->source, mv->w, mv->h, mv->src_x, mv->src_y,
Signed-off-by: numberwolf <porschegt23@foxmail.com> --- Here, don't need show motionless vectors,because they're useless. doc/examples/extract_mvs.c | 7 +++++++ 1 file changed, 7 insertions(+)