On Thu, Aug 25, 2016 at 2:07 AM, Nablet Developer <s...@nablet.com> wrote:
> From: ChaoX A Liu <chaox.a....@intel.com>
>
> Signed-off-by: ChaoX A Liu <chaox.a....@intel.com>
> ---
>  ffmpeg.c                  |   2 +-
>  ffmpeg.h                  |   2 +
>  ffmpeg_opt.c              |   2 +-
>  ffmpeg_qsv.c              | 636 
> +++++++++++++++++++++++++++++++++++++++++++++-
>  libavcodec/qsv.h          |   3 +
>  libavcodec/qsv_internal.h |   2 +
>  libavcodec/qsvdec.c       |   5 +-
>  libavcodec/qsvenc.c       |   2 +
>  8 files changed, 649 insertions(+), 5 deletions(-)
>

This is a giant patch that doesnt even begin to describe what it does.
So, whats it good for? We can already do transcoding of video from QSV
decoder to QSV encoder all in GPU memory without 600+ lines of new
code. Admittedly it currently has a few issues, but those could be
fixed, but why do we need 600 new lines of code?

- Hendrik
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to