[FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

[Michael Niedermayer]

On Thu, Apr 07, 2016 at 11:44:20AM -0400, nablet developer wrote:
> Signed-off-by: nablet developer <sdk at nablet.com>
> ---
>  libavcodec/qsv.c          | 64 +++++++++++++++++++++--------------------------
>  libavcodec/qsv.h          | 53 +++++++++++++++++++++++++++++++++++++++
>  libavcodec/qsv_api.c      | 26 +++++++++++++++++++
>  libavcodec/qsv_internal.h | 15 +----------
>  libavcodec/qsvdec.c       | 13 +++++-----
>  libavcodec/qsvdec.h       |  3 ++-
>  libavcodec/qsvenc.c       | 16 ++++++------
>  libavcodec/qsvenc.h       |  2 +-
>  8 files changed, 125 insertions(+), 67 deletions(-)

commit message is missing the explanation for "why?"


> 
> diff --git a/libavcodec/qsv.c b/libavcodec/qsv.c
> index 4c8e6b0..81d1f0c 100644
> --- a/libavcodec/qsv.c
> +++ b/libavcodec/qsv.c
> @@ -18,14 +18,19 @@
>   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
>   */
>  
> -#include <mfx/mfxvideo.h>
> -#include <mfx/mfxplugin.h>
> -
>  #include <stdio.h>
>  #include <string.h>
>  
>  #include "libavutil/avstring.h"
>  #include "libavutil/error.h"
> +#include "libavutil/log.h"
> +
> +#include "qsv.h"
> +
> +#if CONFIG_QSV

conditional compilation belongs in the Makefile

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160411/a2b315ce/attachment.sig>