diff options
| author |  wm4 <wm4@nowhere> | 2018-04-19 19:55:13 +0200 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2018-04-20 12:37:34 +0200 |
| commit | b3baff399ebfd28b3402905fce32066263d7632d (patch) | |
| tree | 4c7a497e480a72d1da4734cce602dc0262e5e667 | |
| parent | c490b3f8ea88c028e8f23e412ec015c34e591e9c (diff) | |
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.c | 9 |
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 || |