aboutsummaryrefslogtreecommitdiffstats
path: root/libavutil
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@outlook.com>2022-09-26 21:05:00 +0200
committerAndreas Rheinhardt <andreas.rheinhardt@outlook.com>2022-09-30 14:33:08 +0200
commit5fe447bbb4a1dfebb6f19339764aeab0315ff17f (patch)
tree01fc63ee3ac3b58ab2ca467d000d448b02a0e2f3 /libavutil
parent571b670e7dfc90394ba1f3cf736f446976c832da (diff)
downloadffmpeg-5fe447bbb4a1dfebb6f19339764aeab0315ff17f.tar.gz
avutil/pixdesc: Move ff_check_pixfmt_descriptors() to its only user
Namely to lavu/tests/pixelutils.c. This way, this function will not be included into actual binaries any more. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Diffstat (limited to 'libavutil')
-rw-r--r--libavutil/internal.h2
-rw-r--r--libavutil/pixdesc.c49
-rw-r--r--libavutil/tests/pixelutils.c53
3 files changed, 52 insertions, 52 deletions
diff --git a/libavutil/internal.h b/libavutil/internal.h
index c9e30bc5e9..454c59aa50 100644
--- a/libavutil/internal.h
+++ b/libavutil/internal.h
@@ -187,6 +187,4 @@ static av_always_inline av_const int avpriv_mirror(int x, int w)
return x;
}
-void ff_check_pixfmt_descriptors(void);
-
#endif /* AVUTIL_INTERNAL_H */
diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c
index 4f95f88aba..ca3e204a0b 100644
--- a/libavutil/pixdesc.c
+++ b/libavutil/pixdesc.c
@@ -22,12 +22,10 @@
#include <stdio.h>
#include <string.h>
-#include "avassert.h"
#include "avstring.h"
#include "common.h"
#include "pixfmt.h"
#include "pixdesc.h"
-#include "internal.h"
#include "intreadwrite.h"
void av_read_image_line2(void *dst,
@@ -2912,53 +2910,6 @@ int av_pix_fmt_count_planes(enum AVPixelFormat pix_fmt)
return ret;
}
-void ff_check_pixfmt_descriptors(void){
- const AVPixFmtDescriptor *d, *last = NULL;
- int i;
-
- for (i = AV_PIX_FMT_NONE, d = NULL; i++, d = av_pix_fmt_desc_next(d);) {
- uint8_t fill[4][8+6+3] = {{0}};
- uint8_t *data[4] = {fill[0], fill[1], fill[2], fill[3]};
- int linesize[4] = {0,0,0,0};
- uint16_t tmp[2];
-
- av_assert0(d->name && d->name[0]);
- av_log(NULL, AV_LOG_INFO, "Checking: %s\n", d->name);
- av_assert0(d->log2_chroma_w <= 3);
- av_assert0(d->log2_chroma_h <= 3);
- av_assert0(d->nb_components <= 4);
- av_assert2(av_get_pix_fmt(d->name) == av_pix_fmt_desc_get_id(d));
-
- /* The following two checks as well as the one after the loop
- * would need to be changed if we changed the way the descriptors
- * are stored. */
- av_assert0(i == av_pix_fmt_desc_get_id(d));
- av_assert0(!last || last + 1 == d);
-
- for (int j = 0; j < FF_ARRAY_ELEMS(d->comp); j++) {
- const AVComponentDescriptor *c = &d->comp[j];
- if(j>=d->nb_components) {
- av_assert0(!c->plane && !c->step && !c->offset && !c->shift && !c->depth);
- continue;
- }
- if (d->flags & AV_PIX_FMT_FLAG_BITSTREAM) {
- av_assert0(c->step >= c->depth);
- } else {
- av_assert0(8*c->step >= c->depth);
- }
- if (d->flags & AV_PIX_FMT_FLAG_BAYER)
- continue;
- av_read_image_line(tmp, (void*)data, linesize, d, 0, 0, j, 2, 0);
- av_assert0(tmp[0] == 0 && tmp[1] == 0);
- tmp[0] = tmp[1] = (1ULL << c->depth) - 1;
- av_write_image_line(tmp, data, linesize, d, 0, 0, j, 2);
- }
- last = d;
- }
- av_assert0(i == AV_PIX_FMT_NB);
-}
-
-
enum AVPixelFormat av_pix_fmt_swap_endianness(enum AVPixelFormat pix_fmt)
{
const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(pix_fmt);
diff --git a/libavutil/tests/pixelutils.c b/libavutil/tests/pixelutils.c
index 927c8d9217..41663d3eda 100644
--- a/libavutil/tests/pixelutils.c
+++ b/libavutil/tests/pixelutils.c
@@ -18,15 +18,66 @@
#include <stdio.h>
+#include "libavutil/avassert.h"
#include "libavutil/internal.h"
+#include "libavutil/log.h"
#include "libavutil/mem.h"
+#include "libavutil/pixdesc.h"
#include "libavutil/pixelutils.c"
+#include "libavutil/pixfmt.h"
#define W1 320
#define H1 240
#define W2 640
#define H2 480
+static void check_pixfmt_descriptors(void)
+{
+ const AVPixFmtDescriptor *d, *last = NULL;
+ int i;
+
+ for (i = AV_PIX_FMT_NONE, d = NULL; i++, d = av_pix_fmt_desc_next(d);) {
+ uint8_t fill[4][8 + 6 + 3] = {{ 0 }};
+ uint8_t *data[4] = { fill[0], fill[1], fill[2], fill[3] };
+ int linesize[4] = { 0, 0, 0, 0 };
+ uint16_t tmp[2];
+
+ av_assert0(d->name && d->name[0]);
+ av_log(NULL, AV_LOG_INFO, "Checking: %s\n", d->name);
+ av_assert0(d->log2_chroma_w <= 3);
+ av_assert0(d->log2_chroma_h <= 3);
+ av_assert0(d->nb_components <= 4);
+ av_assert2(av_get_pix_fmt(d->name) == av_pix_fmt_desc_get_id(d));
+
+ /* The following two checks as well as the one after the loop
+ * would need to be changed if we changed the way the descriptors
+ * are stored. */
+ av_assert0(i == av_pix_fmt_desc_get_id(d));
+ av_assert0(!last || last + 1 == d);
+
+ for (int j = 0; j < FF_ARRAY_ELEMS(d->comp); j++) {
+ const AVComponentDescriptor *c = &d->comp[j];
+ if (j >= d->nb_components) {
+ av_assert0(!c->plane && !c->step && !c->offset && !c->shift && !c->depth);
+ continue;
+ }
+ if (d->flags & AV_PIX_FMT_FLAG_BITSTREAM) {
+ av_assert0(c->step >= c->depth);
+ } else {
+ av_assert0(8*c->step >= c->depth);
+ }
+ if (d->flags & AV_PIX_FMT_FLAG_BAYER)
+ continue;
+ av_read_image_line(tmp, (void*)data, linesize, d, 0, 0, j, 2, 0);
+ av_assert0(tmp[0] == 0 && tmp[1] == 0);
+ tmp[0] = tmp[1] = (1ULL << c->depth) - 1;
+ av_write_image_line(tmp, data, linesize, d, 0, 0, j, 2);
+ }
+ last = d;
+ }
+ av_assert0(i == AV_PIX_FMT_NB);
+}
+
static int run_single_test(const char *test,
const uint8_t *block1, ptrdiff_t stride1,
const uint8_t *block2, ptrdiff_t stride2,
@@ -87,7 +138,7 @@ int main(void)
goto end;
}
- ff_check_pixfmt_descriptors();
+ check_pixfmt_descriptors();
#define RANDOM_INIT(buf, size) do { \
int k; \