diff options
author | wm4 <wm4@nowhere> | 2017-09-30 16:07:00 +0200 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2017-09-30 16:07:00 +0200 |
commit | f3512686eee6c1a7650e1e769169b4018ee4466d (patch) | |
tree | ec52b4651375f5c27e611c427b463e1591875f19 /video | |
parent | 4516707848159808c34e3dbb54c29fa7a5d41a13 (diff) | |
download | mpv-f3512686eee6c1a7650e1e769169b4018ee4466d.tar.bz2 mpv-f3512686eee6c1a7650e1e769169b4018ee4466d.tar.xz |
vaapi: replace error macro implementation
The current check_va_status() function could probably be argued to be
derived from the original VAAPI's patch check_status() function, thus
GPL-only. While I have my doubts that it applies to an idiom on this
level, it's better to replace it. Similar idea, different expression
equals no copyright association.
An earlier commit message promised this, but it was forgotten.
Diffstat (limited to 'video')
-rw-r--r-- | video/vaapi.c | 9 | ||||
-rw-r--r-- | video/vaapi.h | 6 |
2 files changed, 3 insertions, 12 deletions
diff --git a/video/vaapi.c b/video/vaapi.c index 055316bb38..6bedbbaa18 100644 --- a/video/vaapi.c +++ b/video/vaapi.c @@ -30,15 +30,6 @@ #include <libavutil/hwcontext.h> #include <libavutil/hwcontext_vaapi.h> -bool check_va_status(struct mp_log *log, VAStatus status, const char *msg) -{ - if (status != VA_STATUS_SUCCESS) { - mp_err(log, "%s: %s\n", msg, vaErrorStr(status)); - return false; - } - return true; -} - int va_get_colorspace_flag(enum mp_csp csp) { switch (csp) { diff --git a/video/vaapi.h b/video/vaapi.h index 2e438bebfb..41c9f7c1cb 100644 --- a/video/vaapi.h +++ b/video/vaapi.h @@ -36,9 +36,9 @@ struct mp_vaapi_ctx { void (*destroy_native_ctx)(void *native_ctx); }; -bool check_va_status(struct mp_log *log, VAStatus status, const char *msg); - -#define CHECK_VA_STATUS(ctx, msg) check_va_status((ctx)->log, status, msg) +#define CHECK_VA_STATUS(ctx, msg) \ + (status == VA_STATUS_SUCCESS ? true \ + : (MP_ERR(ctx, "%s failed (%s)\n", msg, vaErrorStr(status)), false)) int va_get_colorspace_flag(enum mp_csp csp); |