[FFmpeg-devel] [PATCH] avcodec: export motion vectors in frame side data on demand

[Stefano Sabatini]

On date Sunday 2014-08-17 20:08:35 +0200, Clément Bœsch encoded:
[...]
> From 76f24f87bdfe1ca8778a6d39751fd70246c3b093 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Cl=C3=A9ment=20B=C5=93sch?= <clement at stupeflix.com>
> Date: Wed, 16 Jul 2014 16:42:42 +0200
> Subject: [PATCH] avcodec: export motion vectors in frame side data on demand
> 
> The reasoning behind this addition is that various third party
> applications are interested in getting some motion information out of a
> video "for free" when it is available.
> 
> It was considered to export other information as well (such as the intra
> information about the block, or the quantization) but the structure
> might have ended up into a half full-generic, half full of codec
> specific cruft. If more information is necessary, it should either be
> added in the "flags" field of the AVMotionVector structure, or in
> another side-data.
> 
> This commit also includes an example exporting them in a CSV stream.
> ---
>  .gitignore                 |   1 +
>  configure                  |   2 +
>  doc/APIchanges             |   3 +
>  doc/Makefile               |   1 +
>  doc/codecs.texi            |   3 +
>  doc/examples/Makefile      |   1 +
>  doc/examples/extract_mvs.c | 185 +++++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/avcodec.h       |   1 +
>  libavcodec/mpegvideo.c     | 103 ++++++++++++++++++++++++-
>  libavcodec/options_table.h |   1 +
>  libavutil/Makefile         |   1 +
>  libavutil/frame.c          |   1 +
>  libavutil/frame.h          |   7 ++
>  libavutil/motion_vector.h  |  50 ++++++++++++
>  libavutil/version.h        |   2 +-
>  15 files changed, 360 insertions(+), 2 deletions(-)
>  create mode 100644 doc/examples/extract_mvs.c
>  create mode 100644 libavutil/motion_vector.h

LGTM (but I'm no lavc maintainer), thanks.
-- 
FFmpeg = Freak & Frightening Muttering Pitiless Explosive Gorilla