diff options
author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-09-26 19:37:25 +0200 |
---|---|---|
committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-09-30 14:33:08 +0200 |
commit | 6d0a7e96e746a8a0fc03725a0f5e2c908c82b7c3 (patch) | |
tree | b6f9a3d3b6c907b8cf44f50aa9367715d397445c /libavutil/internal.h | |
parent | a1bfb5290e58afa3d38e4d3f986302a2668fbfbe (diff) | |
download | ffmpeg-6d0a7e96e746a8a0fc03725a0f5e2c908c82b7c3.tar.gz |
avutil/pixdesc: Remove always-false checks
ff_check_pixfmt_descriptors() was added in commit
20e99a9c10cdbe9ad659dce5bdec569d744f8219. At this time,
the values of enum AVPixelFormat were not contiguous;
instead there was a jump from 111 to 291 (or from 115
to 295 depending upon AV_PIX_FMT_ABI_GIT_MASTER).
ff_check_pixfmt_descriptors() accounts for this
by skipping empty descriptors. Yet this issue no longer
exists: There are no holes.
The check for said holes makes GCC believe that the name
can be NULL; because it is used as argument corresponding to
%s in a log statement, it therefore emits a warning
(since d75c4693fef51e8f0a1b88798530f4c5147ea906). Therefore
this commit simply removes these checks.
Also move the checks for name before the log statement.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Diffstat (limited to 'libavutil/internal.h')
0 files changed, 0 insertions, 0 deletions