summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2018-04-19 19:55:13 +0200
committerwm4 <wm4@nowhere>2018-04-20 12:37:34 +0200
commitb3baff399ebfd28b3402905fce32066263d7632d (patch)
tree4c7a497e480a72d1da4734cce602dc0262e5e667
parentc490b3f8ea88c028e8f23e412ec015c34e591e9c (diff)
downloadmpv-b3baff399ebfd28b3402905fce32066263d7632d.tar.bz2
mpv-b3baff399ebfd28b3402905fce32066263d7632d.tar.xz
vo_lavc: remove pointless uint32_t type for int values
params->w/h are int, and the further use of these variables are int. The uint32_t is probably some refactoring artifact.
-rw-r--r--video/out/vo_lavc.c9
1 files changed, 2 insertions, 7 deletions
diff --git a/video/out/vo_lavc.c b/video/out/vo_lavc.c
index 4b69231bfd..b32e405747 100644
--- a/video/out/vo_lavc.c
+++ b/video/out/vo_lavc.c
@@ -94,8 +94,8 @@ static int reconfig(struct vo *vo, struct mp_image_params *params)
struct priv *vc = vo->priv;
enum AVPixelFormat pix_fmt = imgfmt2pixfmt(params->imgfmt);
AVRational aspect = {params->p_w, params->p_h};
- uint32_t width = params->w;
- uint32_t height = params->h;
+ int width = params->w;
+ int height = params->h;
if (!vc || vc->shutdown)
return -1;
@@ -103,11 +103,6 @@ static int reconfig(struct vo *vo, struct mp_image_params *params)
pthread_mutex_lock(&vo->encode_lavc_ctx->lock);
if (vc->stream) {
- /* NOTE:
- * in debug builds we get a "comparison between signed and unsigned"
- * warning here. We choose to ignore that; just because ffmpeg currently
- * uses a plain 'int' for these struct fields, it doesn't mean it always
- * will */
if (width == vc->codec->width &&
height == vc->codec->height) {
if (aspect.num != vc->codec->sample_aspect_ratio.num ||