diff mbox

[FFmpeg-devel,v1,5/5] avcodec/v210enc: define DEFINE_V210_ENCODE_FUNC to remove duplicate code

Message ID 1567007116-9088-6-git-send-email-lance.lmwang@gmail.com
State Superseded
Headers show

Commit Message

Lance Wang Aug. 28, 2019, 3:45 p.m. UTC
From: Limin Wang <lance.lmwang@gmail.com>

Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
---
 libavcodec/v210enc.c | 171 +++++++++++++++++++++------------------------------
 1 file changed, 69 insertions(+), 102 deletions(-)

Comments

Michael Niedermayer Aug. 30, 2019, 6:11 p.m. UTC | #1
On Wed, Aug 28, 2019 at 11:45:16PM +0800, lance.lmwang@gmail.com wrote:
> From: Limin Wang <lance.lmwang@gmail.com>
> 
> Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
> ---
>  libavcodec/v210enc.c | 171 +++++++++++++++++++++------------------------------
>  1 file changed, 69 insertions(+), 102 deletions(-)
> 
> diff --git a/libavcodec/v210enc.c b/libavcodec/v210enc.c
> index 2400538..b40c7ae 100644
> --- a/libavcodec/v210enc.c
> +++ b/libavcodec/v210enc.c
> @@ -89,15 +89,77 @@ FF_ENABLE_DEPRECATION_WARNINGS
>      return 0;
>  }
>  
> +#define DEFINE_V210_ENCODE_FUNC(nbits, depth)                                 \
> +static void v210_enc_##depth(AVCodecContext *avctx,                           \
> +        uint8_t *dst, const AVFrame *pic)                                     \

puting this in a seperate file which is included twice would avoid
the rather large mutiline macro
also that would match how such functions for multiple depths are
handled in other codecs

thx

[...]
Lance Wang Aug. 31, 2019, 7:26 a.m. UTC | #2
On Fri, Aug 30, 2019 at 08:11:17PM +0200, Michael Niedermayer wrote:
> On Wed, Aug 28, 2019 at 11:45:16PM +0800, lance.lmwang@gmail.com wrote:
> > From: Limin Wang <lance.lmwang@gmail.com>
> > 
> > Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
> > ---
> >  libavcodec/v210enc.c | 171 +++++++++++++++++++++------------------------------
> >  1 file changed, 69 insertions(+), 102 deletions(-)
> > 
> > diff --git a/libavcodec/v210enc.c b/libavcodec/v210enc.c
> > index 2400538..b40c7ae 100644
> > --- a/libavcodec/v210enc.c
> > +++ b/libavcodec/v210enc.c
> > @@ -89,15 +89,77 @@ FF_ENABLE_DEPRECATION_WARNINGS
> >      return 0;
> >  }
> >  
> > +#define DEFINE_V210_ENCODE_FUNC(nbits, depth)                                 \
> > +static void v210_enc_##depth(AVCodecContext *avctx,                           \
> > +        uint8_t *dst, const AVFrame *pic)                                     \
> 
> puting this in a seperate file which is included twice would avoid
> the rather large mutiline macro
> also that would match how such functions for multiple depths are
> handled in other codecs

That's OK, I'll add a template file for the function.


> 
> thx
> 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> The smallest minority on earth is the individual. Those who deny 
> individual rights cannot claim to be defenders of minorities. - Ayn Rand



> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
Lance Wang Sept. 6, 2019, 2:41 p.m. UTC | #3
On Fri, Aug 30, 2019 at 08:11:17PM +0200, Michael Niedermayer wrote:
> On Wed, Aug 28, 2019 at 11:45:16PM +0800, lance.lmwang@gmail.com wrote:
> > From: Limin Wang <lance.lmwang@gmail.com>
> > 
> > Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
> > ---
> >  libavcodec/v210enc.c | 171 +++++++++++++++++++++------------------------------
> >  1 file changed, 69 insertions(+), 102 deletions(-)
> > 
> > diff --git a/libavcodec/v210enc.c b/libavcodec/v210enc.c
> > index 2400538..b40c7ae 100644
> > --- a/libavcodec/v210enc.c
> > +++ b/libavcodec/v210enc.c
> > @@ -89,15 +89,77 @@ FF_ENABLE_DEPRECATION_WARNINGS
> >      return 0;
> >  }
> >  
> > +#define DEFINE_V210_ENCODE_FUNC(nbits, depth)                                 \
> > +static void v210_enc_##depth(AVCodecContext *avctx,                           \
> > +        uint8_t *dst, const AVFrame *pic)                                     \
> 
> puting this in a seperate file which is included twice would avoid
> the rather large mutiline macro
> also that would match how such functions for multiple depths are
> handled in other codecs

Michael, I have updated the patch with template file to replace macro in the thread,
please help to review them. I plan to add multithread function if it's OK to apply.

> 
> thx
> 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> The smallest minority on earth is the individual. Those who deny 
> individual rights cannot claim to be defenders of minorities. - Ayn Rand



> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
diff mbox

Patch

diff --git a/libavcodec/v210enc.c b/libavcodec/v210enc.c
index 2400538..b40c7ae 100644
--- a/libavcodec/v210enc.c
+++ b/libavcodec/v210enc.c
@@ -89,15 +89,77 @@  FF_ENABLE_DEPRECATION_WARNINGS
     return 0;
 }
 
+#define DEFINE_V210_ENCODE_FUNC(nbits, depth)                                 \
+static void v210_enc_##depth(AVCodecContext *avctx,                           \
+        uint8_t *dst, const AVFrame *pic)                                     \
+{                                                                             \
+    V210EncContext *s = avctx->priv_data;                                     \
+    int aligned_width = ((avctx->width + 47) / 48) * 48;                      \
+    int stride = aligned_width * 8 / 3;                                       \
+    int line_padding = stride - ((avctx->width * 8 + 11) / 12) * 4;           \
+    int h, w;                                                                 \
+    int bytes = nbits / 8;                                                    \
+    const uint##nbits##_t *y = (const uint##nbits##_t *)pic->data[0];         \
+    const uint##nbits##_t *u = (const uint##nbits##_t *)pic->data[1];         \
+    const uint##nbits##_t *v = (const uint##nbits##_t *)pic->data[2];         \
+    const int sample_size = 6 * s->sample_factor_##depth;                     \
+    const int sample_w    = avctx->width / sample_size;                       \
+                                                                              \
+    for (h = 0; h < avctx->height; h++) {                                     \
+        uint32_t val;                                                         \
+        w = sample_w * sample_size;                                           \
+        s->pack_line_##depth (y, u, v, dst, w);                               \
+                                                                              \
+        y += w;                                                               \
+        u += w >> 1;                                                          \
+        v += w >> 1;                                                          \
+        dst += sample_w * 16 * s->sample_factor_##depth;                      \
+                                                                              \
+        for (; w < avctx->width - 5; w += 6) {                                \
+            WRITE_PIXELS(u, y, v, depth);                                     \
+            WRITE_PIXELS(y, u, y, depth);                                     \
+            WRITE_PIXELS(v, y, u, depth);                                     \
+            WRITE_PIXELS(y, v, y, depth);                                     \
+        }                                                                     \
+        if (w < avctx->width - 1) {                                           \
+            WRITE_PIXELS(u, y, v, depth);                                     \
+                                                                              \
+            val = CLIP(*y++, depth);                                          \
+            if (w == avctx->width - 2) {                                      \
+                AV_WL32(dst, val);                                            \
+                dst += 4;                                                     \
+            }                                                                 \
+        }                                                                     \
+        if (w < avctx->width - 3) {                                           \
+            val |= (CLIP(*u++, depth) << (20-depth)) |                        \
+                    (CLIP(*y++, depth) << (30-depth));                        \
+            AV_WL32(dst, val);                                                \
+            dst += 4;                                                         \
+                                                                              \
+            val = CLIP(*v++, depth) << (10-depth) |                           \
+                    (CLIP(*y++, depth) << (20-depth));                        \
+            AV_WL32(dst, val);                                                \
+            dst += 4;                                                         \
+        }                                                                     \
+                                                                              \
+        memset(dst, 0, line_padding);                                         \
+        dst += line_padding;                                                  \
+        y += pic->linesize[0] / bytes - avctx->width;                         \
+        u += pic->linesize[1] / bytes - avctx->width / 2;                     \
+        v += pic->linesize[2] / bytes - avctx->width / 2;                     \
+    }                                                                         \
+}
+
+DEFINE_V210_ENCODE_FUNC(8, 8)
+DEFINE_V210_ENCODE_FUNC(16, 10);
+
 static int encode_frame(AVCodecContext *avctx, AVPacket *pkt,
                         const AVFrame *pic, int *got_packet)
 {
-    V210EncContext *s = avctx->priv_data;
     int aligned_width = ((avctx->width + 47) / 48) * 48;
     int stride = aligned_width * 8 / 3;
-    int line_padding = stride - ((avctx->width * 8 + 11) / 12) * 4;
     AVFrameSideData *side_data;
-    int h, w, ret;
+    int ret;
     uint8_t *dst;
 
     ret = ff_alloc_packet2(avctx, pkt, avctx->height * stride, avctx->height * stride);
@@ -107,105 +169,10 @@  static int encode_frame(AVCodecContext *avctx, AVPacket *pkt,
     }
     dst = pkt->data;
 
-    if (pic->format == AV_PIX_FMT_YUV422P10) {
-        const uint16_t *y = (const uint16_t *)pic->data[0];
-        const uint16_t *u = (const uint16_t *)pic->data[1];
-        const uint16_t *v = (const uint16_t *)pic->data[2];
-
-        const int sample_size = 6 * s->sample_factor_10;
-        const int sample_w    = avctx->width / sample_size;
-
-        for (h = 0; h < avctx->height; h++) {
-            uint32_t val;
-            w = sample_w * sample_size;
-            s->pack_line_10(y, u, v, dst, w);
-
-            y += w;
-            u += w >> 1;
-            v += w >> 1;
-            dst += sample_w * 16 * s->sample_factor_10;
-
-            for (; w < avctx->width - 5; w += 6) {
-                WRITE_PIXELS(u, y, v, 10);
-                WRITE_PIXELS(y, u, y, 10);
-                WRITE_PIXELS(v, y, u, 10);
-                WRITE_PIXELS(y, v, y, 10);
-            }
-            if (w < avctx->width - 1) {
-                WRITE_PIXELS(u, y, v, 10);
-
-                val = CLIP(*y++, 10);
-                if (w == avctx->width - 2) {
-                    AV_WL32(dst, val);
-                    dst += 4;
-                }
-            }
-            if (w < avctx->width - 3) {
-                val |= (CLIP(*u++, 10) << (20-10)) | (CLIP(*y++, 10) << (30-10));
-                AV_WL32(dst, val);
-                dst += 4;
-
-                val = CLIP(*v++, 10) << (10-10) | (CLIP(*y++, 10) << (20-10));
-                AV_WL32(dst, val);
-                dst += 4;
-            }
-
-            memset(dst, 0, line_padding);
-            dst += line_padding;
-            y += pic->linesize[0] / 2 - avctx->width;
-            u += pic->linesize[1] / 2 - avctx->width / 2;
-            v += pic->linesize[2] / 2 - avctx->width / 2;
-        }
-    } else if(pic->format == AV_PIX_FMT_YUV422P) {
-        const uint8_t *y = pic->data[0];
-        const uint8_t *u = pic->data[1];
-        const uint8_t *v = pic->data[2];
-
-        const int sample_size = 6 * s->sample_factor_8;
-        const int sample_w    = avctx->width / sample_size;
-
-        for (h = 0; h < avctx->height; h++) {
-            uint32_t val;
-            w = sample_w * sample_size;
-            s->pack_line_8(y, u, v, dst, w);
-
-            y += w;
-            u += w >> 1;
-            v += w >> 1;
-            dst += sample_w * 16 * s->sample_factor_8;
-
-            for (; w < avctx->width - 5; w += 6) {
-                WRITE_PIXELS(u, y, v, 8);
-                WRITE_PIXELS(y, u, y, 8);
-                WRITE_PIXELS(v, y, u, 8);
-                WRITE_PIXELS(y, v, y, 8);
-            }
-            if (w < avctx->width - 1) {
-                WRITE_PIXELS(u, y, v, 8);
-
-                val = CLIP(*y++, 8) << (10-8);
-                if (w == avctx->width - 2) {
-                    AV_WL32(dst, val);
-                    dst += 4;
-                }
-            }
-            if (w < avctx->width - 3) {
-                val |= (CLIP(*u++, 8) << (20-8)) | (CLIP(*y++, 8) << (30-8));
-                AV_WL32(dst, val);
-                dst += 4;
-
-                val = (CLIP(*v++, 8) << (10-8)) | (CLIP(*y++, 8) << (20-8));
-                AV_WL32(dst, val);
-                dst += 4;
-            }
-            memset(dst, 0, line_padding);
-            dst += line_padding;
-
-            y += pic->linesize[0] - avctx->width;
-            u += pic->linesize[1] - avctx->width / 2;
-            v += pic->linesize[2] - avctx->width / 2;
-        }
-    }
+    if (pic->format == AV_PIX_FMT_YUV422P10)
+        v210_enc_10(avctx, dst, pic);
+    else if(pic->format == AV_PIX_FMT_YUV422P)
+        v210_enc_8(avctx, dst, pic);
 
     side_data = av_frame_get_side_data(pic, AV_FRAME_DATA_A53_CC);
     if (side_data && side_data->size) {