Round image line sizes up to 32. Seems to help with LHS image corruption with both...
[dcpomatic.git] / src / lib / image.cc
index 2df7636af689a14e6a7e175ca677cd2bf8791972..c8849303cac99e8ed01c1e6f6d5e80bffade6706 100644 (file)
@@ -57,6 +57,7 @@ Image::lines (int n) const
                }
                break;
        case PIX_FMT_RGB24:
+       case PIX_FMT_RGBA:
                return size().height;
        default:
                assert (false);
@@ -73,6 +74,7 @@ Image::components () const
        case PIX_FMT_YUV420P:
                return 3;
        case PIX_FMT_RGB24:
+       case PIX_FMT_RGBA:
                return 1;
        default:
                assert (false);
@@ -81,11 +83,36 @@ Image::components () const
        return 0;
 }
 
+shared_ptr<Image>
+Image::scale (Size out_size, Scaler const * scaler) const
+{
+       assert (scaler);
+
+       shared_ptr<Image> scaled (new SimpleImage (pixel_format(), out_size));
+
+       struct SwsContext* scale_context = sws_getContext (
+               size().width, size().height, pixel_format(),
+               out_size.width, out_size.height, pixel_format(),
+               scaler->ffmpeg_id (), 0, 0, 0
+               );
+
+       sws_scale (
+               scale_context,
+               data(), line_size(),
+               0, size().height,
+               scaled->data (), scaled->line_size ()
+               );
+
+       sws_freeContext (scale_context);
+
+       return scaled;
+}
+
 /** Scale this image to a given size and convert it to RGB.
  *  @param out_size Output image size in pixels.
  *  @param scaler Scaler to use.
  */
-shared_ptr<RGBFrameImage>
+shared_ptr<Image>
 Image::scale_and_convert_to_rgb (Size out_size, int padding, Scaler const * scaler) const
 {
        assert (scaler);
@@ -93,7 +120,7 @@ Image::scale_and_convert_to_rgb (Size out_size, int padding, Scaler const * scal
        Size content_size = out_size;
        content_size.width -= (padding * 2);
 
-       shared_ptr<RGBFrameImage> rgb (new RGBFrameImage (content_size));
+       shared_ptr<Image> rgb (new SimpleImage (PIX_FMT_RGB24, content_size));
 
        struct SwsContext* scale_context = sws_getContext (
                size().width, size().height, pixel_format(),
@@ -114,7 +141,7 @@ Image::scale_and_convert_to_rgb (Size out_size, int padding, Scaler const * scal
           scheme of things.
        */
        if (padding > 0) {
-               shared_ptr<RGBFrameImage> padded_rgb (new RGBFrameImage (out_size));
+               shared_ptr<Image> padded_rgb (new SimpleImage (PIX_FMT_RGB24, out_size));
                padded_rgb->make_black ();
 
                /* XXX: we are cheating a bit here; we know the frame is RGB so we can
@@ -140,10 +167,10 @@ Image::scale_and_convert_to_rgb (Size out_size, int padding, Scaler const * scal
  *  @param pp Flags for the required set of post processes.
  *  @return Post-processed image.
  */
-shared_ptr<PostProcessImage>
+shared_ptr<Image>
 Image::post_process (string pp) const
 {
-       shared_ptr<PostProcessImage> out (new PostProcessImage (PIX_FMT_YUV420P, size ()));
+       shared_ptr<Image> out (new SimpleImage (PIX_FMT_YUV420P, size ()));
        
        pp_mode* mode = pp_get_mode_by_name_and_quality (pp.c_str (), PP_QUALITY_MAX);
        pp_context* context = pp_get_context (size().width, size().height, PP_FORMAT_420 | PP_CPU_CAPS_MMX2);
@@ -180,6 +207,42 @@ Image::make_black ()
        }
 }
 
+void
+Image::alpha_blend (shared_ptr<Image> other, Position position)
+{
+       /* Only implemented for RGBA onto RGB24 so far */
+       assert (_pixel_format == PIX_FMT_RGB24 && other->pixel_format() == PIX_FMT_RGBA);
+
+       int start_tx = position.x;
+       int start_ox = 0;
+
+       if (start_tx < 0) {
+               start_ox = -start_tx;
+               start_tx = 0;
+       }
+
+       int start_ty = position.y;
+       int start_oy = 0;
+
+       if (start_ty < 0) {
+               start_oy = -start_ty;
+               start_ty = 0;
+       }
+
+       for (int ty = start_ty, oy = start_oy; ty < size().height && oy < other->size().height; ++ty, ++oy) {
+               uint8_t* tp = data()[0] + ty * line_size()[0] + position.x * 3;
+               uint8_t* op = other->data()[0] + oy * other->line_size()[0];
+               for (int tx = start_tx, ox = start_ox; tx < size().width && ox < other->size().width; ++tx, ++ox) {
+                       float const alpha = float (op[3]) / 255;
+                       tp[0] = (tp[0] * (1 - alpha)) + op[0] * alpha;
+                       tp[1] = (tp[1] * (1 - alpha)) + op[1] * alpha;
+                       tp[2] = (tp[2] * (1 - alpha)) + op[2] * alpha;
+                       tp += 3;
+                       op += 4;
+               }
+       }
+}
+
 /** Construct a SimpleImage of a given size and format, allocating memory
  *  as required.
  *
@@ -190,12 +253,29 @@ SimpleImage::SimpleImage (PixelFormat p, Size s)
        : Image (p)
        , _size (s)
 {
-       _data = (uint8_t **) av_malloc (components() * sizeof (uint8_t *));
-       _line_size = (int *) av_malloc (components() * sizeof (int));
+       _data = (uint8_t **) av_malloc (4 * sizeof (uint8_t *));
+       _data[0] = _data[1] = _data[2] = _data[3] = 0;
+       _line_size = (int *) av_malloc (4);
+       _line_size[0] = _line_size[1] = _line_size[2] = _line_size[3] = 0;
+
+       switch (p) {
+       case PIX_FMT_RGB24:
+               _line_size[0] = round_up (s.width * 3, 32);
+               break;
+       case PIX_FMT_RGBA:
+               _line_size[0] = round_up (s.width * 4, 32);
+               break;
+       case PIX_FMT_YUV420P:
+               _line_size[0] = round_up (s.width, 32);
+               _line_size[1] = round_up (s.width / 2, 32);
+               _line_size[2] = round_up (s.width / 2, 32);
+               break;
+       default:
+               assert (false);
+       }
        
        for (int i = 0; i < components(); ++i) {
-               _data[i] = 0;
-               _line_size[i] = 0;
+               _data[i] = (uint8_t *) av_malloc (_line_size[i] * lines (i));
        }
 }
 
@@ -210,17 +290,6 @@ SimpleImage::~SimpleImage ()
        av_free (_line_size);
 }
 
-/** Set the size in bytes of each horizontal line of a given component.
- *  @param i Component index.
- *  @param s Size of line in bytes.
- */
-void
-SimpleImage::set_line_size (int i, int s)
-{
-       _line_size[i] = s;
-       _data[i] = (uint8_t *) av_malloc (s * lines (i));
-}
-
 uint8_t **
 SimpleImage::data () const
 {
@@ -313,44 +382,3 @@ RGBFrameImage::size () const
 {
        return _size;
 }
-
-PostProcessImage::PostProcessImage (PixelFormat p, Size s)
-       : Image (p)
-       , _size (s)
-{
-       _data = new uint8_t*[4];
-       _line_size = new int[4];
-       
-       for (int i = 0; i < 4; ++i) {
-               _data[i] = (uint8_t *) av_malloc (s.width * s.height);
-               _line_size[i] = s.width;
-       }
-}
-
-PostProcessImage::~PostProcessImage ()
-{
-       for (int i = 0; i < 4; ++i) {
-               av_free (_data[i]);
-       }
-       
-       delete[] _data;
-       delete[] _line_size;
-}
-
-uint8_t **
-PostProcessImage::data () const
-{
-       return _data;
-}
-
-int *
-PostProcessImage::line_size () const
-{
-       return _line_size;
-}
-
-Size
-PostProcessImage::size () const
-{
-       return _size;
-}