Add pixel format 0 (AV_PIX_FMT_YUV420P) to make_part_black().
authorCarl Hetherington <cth@carlh.net>
Sun, 2 May 2021 23:38:11 +0000 (01:38 +0200)
committerCarl Hetherington <cth@carlh.net>
Sun, 2 May 2021 23:39:32 +0000 (01:39 +0200)
Remainder of fix for #1984.

src/lib/image.cc
test/image_test.cc

index 63ae34c..c4312cb 100644 (file)
@@ -392,6 +392,17 @@ Image::swap_16 (uint16_t v)
 void
 Image::make_part_black (int const start, int const width)
 {
+       auto y_part = [&]() {
+               int const bpp = bytes_per_pixel(0);
+               int const h = sample_size(0).height;
+               int const s = stride()[0];
+               auto p = data()[0];
+               for (int y = 0; y < h; ++y) {
+                       memset (p + start * bpp, 0, width * bpp);
+                       p += s;
+               }
+       };
+
        switch (_pixel_format) {
        case AV_PIX_FMT_RGB24:
        case AV_PIX_FMT_ARGB:
@@ -413,20 +424,28 @@ Image::make_part_black (int const start, int const width)
                }
                break;
        }
-       case AV_PIX_FMT_YUV422P10LE:
+       case AV_PIX_FMT_YUV420P:
        {
-               int const bpp_0 = bytes_per_pixel(0);
-               int const h_0 = sample_size(0).height;
-               int const stride_0 = stride()[0];
-               auto p = data()[0];
-               for (int y = 0; y < h_0; ++y) {
-                       memset (p + start * bpp_0, 0xff, width * bpp_0);
-                       p += stride_0;
+               y_part ();
+               for (int i = 1; i < 3; ++i) {
+                       auto p = data()[i];
+                       int const h = sample_size(i).height;
+                       for (int y = 0; y < h; ++y) {
+                               for (int x = start / 2; x < (start + width) / 2; ++x) {
+                                       p[x] = eight_bit_uv;
+                               }
+                               p += stride()[i];
+                       }
                }
+               break;
+       }
+       case AV_PIX_FMT_YUV422P10LE:
+       {
+               y_part ();
                for (int i = 1; i < 3; ++i) {
                        auto p = reinterpret_cast<int16_t*>(data()[i]);
-                       int const lines = sample_size(i).height;
-                       for (int y = 0; y < lines; ++y) {
+                       int const h = sample_size(i).height;
+                       for (int y = 0; y < h; ++y) {
                                for (int x = start / 2; x < (start + width) / 2; ++x) {
                                        p[x] = ten_bit_uv;
                                }
index bbaf9bd..0993be6 100644 (file)
@@ -514,6 +514,7 @@ BOOST_AUTO_TEST_CASE (make_part_black_test)
                AV_PIX_FMT_RGBA,
                AV_PIX_FMT_ABGR,
                AV_PIX_FMT_BGRA,
+               AV_PIX_FMT_YUV420P,
                AV_PIX_FMT_YUV422P10LE,
        };