diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2023-08-31 20:53:22 +0200 |
|---|---|---|
| committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2023-09-04 11:04:45 +0200 |
| commit | 03a60b2b3020af0aa07db3f9b76fafaa2235883e (patch) | |
| tree | 2c7ae0be0597c94046c38a3cf5d619b84d924036 /libavfilter | |
| parent | d84ee80c04d3704c571025364f73c7d0208c7c6a (diff) | |
| download | ffmpeg-03a60b2b3020af0aa07db3f9b76fafaa2235883e.tar.gz | |
avfilter/vf_colorspace: Remove redundant emms_c()
It is not even clear what these emms_c() are supposed to achieve:
create_filtergraph() (where it may be called) does not call
ASM functions itself; it merely sets some function pointers.
Furthermore, there are no colorspacedsp functions using MMX
(checked by checkasm which does not use declare_new_emms()).
Finally, checking whether to issue emms_c() is overblown anyway.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Diffstat (limited to 'libavfilter')
| -rw-r--r-- | libavfilter/vf_colorspace.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c index 852dc11c6c..2a30434401 100644 --- a/libavfilter/vf_colorspace.c +++ b/libavfilter/vf_colorspace.c @@ -400,7 +400,7 @@ static int create_filtergraph(AVFilterContext *ctx, ColorSpaceContext *s = ctx->priv; const AVPixFmtDescriptor *in_desc = av_pix_fmt_desc_get(in->format); const AVPixFmtDescriptor *out_desc = av_pix_fmt_desc_get(out->format); - int emms = 0, m, n, o, res, fmt_identical, redo_yuv2rgb = 0, redo_rgb2yuv = 0; + int m, n, o, res, fmt_identical, redo_yuv2rgb = 0, redo_rgb2yuv = 0; #define supported_depth(d) ((d) == 8 || (d) == 10 || (d) == 12) #define supported_subsampling(lcw, lch) \ @@ -494,7 +494,6 @@ static int create_filtergraph(AVFilterContext *ctx, s->lrgb2lrgb_coeffs[m][n][o] = s->lrgb2lrgb_coeffs[m][n][0]; } - emms = 1; } } @@ -542,7 +541,6 @@ static int create_filtergraph(AVFilterContext *ctx, res = fill_gamma_table(s); if (res < 0) return res; - emms = 1; } if (!s->in_lumacoef) { @@ -625,7 +623,6 @@ static int create_filtergraph(AVFilterContext *ctx, av_assert2(s->yuv2rgb_coeffs[0][0][0] == s->yuv2rgb_coeffs[2][0][0]); s->yuv2rgb = s->dsp.yuv2rgb[(in_desc->comp[0].depth - 8) >> 1] [in_desc->log2_chroma_h + in_desc->log2_chroma_w]; - emms = 1; } if (redo_rgb2yuv) { @@ -656,7 +653,6 @@ static int create_filtergraph(AVFilterContext *ctx, [out_desc->log2_chroma_h + out_desc->log2_chroma_w]; s->rgb2yuv_fsb = s->dsp.rgb2yuv_fsb[(out_desc->comp[0].depth - 8) >> 1] [out_desc->log2_chroma_h + out_desc->log2_chroma_w]; - emms = 1; } if (s->yuv2yuv_fastmode && (redo_yuv2rgb || redo_rgb2yuv)) { @@ -683,9 +679,6 @@ static int create_filtergraph(AVFilterContext *ctx, } } - if (emms) - emms_c(); - return 0; } |