Merge commit 'ccd70d9c1689990e5aef2de383199bbc7cf60d13'
authorMichael Niedermayer <michaelni@gmx.at>
Sat, 16 Feb 2013 07:48:45 +0000 (08:48 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Sat, 16 Feb 2013 07:48:45 +0000 (08:48 +0100)
* commit 'ccd70d9c1689990e5aef2de383199bbc7cf60d13':
  vf_yadif: factorize initializing the filtering callbacks

Conflicts:
libavfilter/vf_yadif.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavfilter/vf_yadif.c

diff --combined libavfilter/vf_yadif.c
@@@ -1,26 -1,26 +1,26 @@@
  /*
 - * Copyright (C) 2006-2010 Michael Niedermayer <michaelni@gmx.at>
 + * Copyright (C) 2006-2011 Michael Niedermayer <michaelni@gmx.at>
   *               2010      James Darnley <james.darnley@gmail.com>
   *
 - * This file is part of Libav.
 - *
 - * Libav is free software; you can redistribute it and/or modify
 + * FFmpeg is free software; you can redistribute it and/or modify
   * it under the terms of the GNU General Public License as published by
   * the Free Software Foundation; either version 2 of the License, or
   * (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
   * GNU General Public License for more details.
   *
   * You should have received a copy of the GNU General Public License along
 - * with Libav; if not, write to the Free Software Foundation, Inc.,
 + * with FFmpeg; if not, write to the Free Software Foundation, Inc.,
   * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
   */
  
 +#include "libavutil/avassert.h"
  #include "libavutil/cpu.h"
  #include "libavutil/common.h"
 +#include "libavutil/opt.h"
  #include "libavutil/pixdesc.h"
  #include "avfilter.h"
  #include "formats.h"
@@@ -124,7 -124,6 +124,7 @@@ static void filter(AVFilterContext *ctx
          int w = dstpic->video->w;
          int h = dstpic->video->h;
          int refs = yadif->cur->linesize[i];
 +        int absrefs = FFABS(refs);
          int df = (yadif->csp->comp[i].depth_minus1 + 8) / 8;
  
          if (i == 1 || i == 2) {
              h >>= yadif->csp->log2_chroma_h;
          }
  
 +        if(yadif->temp_line_size < absrefs) {
 +            av_free(yadif->temp_line);
 +            yadif->temp_line = av_mallocz(2*64 + 5*absrefs);
 +            yadif->temp_line_size = absrefs;
 +        }
 +
          for (y = 0; y < h; y++) {
              if ((y ^ parity) & 1) {
                  uint8_t *prev = &yadif->prev->data[i][y * refs];
                  uint8_t *next = &yadif->next->data[i][y * refs];
                  uint8_t *dst  = &dstpic->data[i][y * dstpic->linesize[i]];
                  int     mode  = y == 1 || y + 2 == h ? 2 : yadif->mode;
 +                int     prefs = y+1<h ? refs : -refs;
 +                int     mrefs =     y ?-refs :  refs;
 +
 +                if(y<=1 || y+2>=h) {
 +                    uint8_t *tmp = yadif->temp_line + 64 + 2*absrefs;
 +                    if(mode<2)
 +                        memcpy(tmp+2*mrefs, cur+2*mrefs, w*df);
 +                    memcpy(tmp+mrefs, cur+mrefs, w*df);
 +                    memcpy(tmp      , cur      , w*df);
 +                    if(prefs != mrefs) {
 +                        memcpy(tmp+prefs, cur+prefs, w*df);
 +                        if(mode<2)
 +                            memcpy(tmp+2*prefs, cur+2*prefs, w*df);
 +                    }
 +                    cur = tmp;
 +                }
 +
                  yadif->filter_line(dst, prev, cur, next, w,
 -                                   y + 1 < h ? refs : -refs,
 -                                   y ? -refs : refs,
 +                                   prefs, mrefs,
                                     parity ^ tff, mode);
              } else {
                  memcpy(&dstpic->data[i][y * dstpic->linesize[i]],
      emms_c();
  }
  
 -static AVFilterBufferRef *get_video_buffer(AVFilterLink *link, int perms,
 -                                           int w, int h)
 -{
 -    AVFilterBufferRef *picref;
 -    int width  = FFALIGN(w, 32);
 -    int height = FFALIGN(h + 2, 32);
 -    int i;
 -
 -    picref = ff_default_get_video_buffer(link, perms, width, height);
 -
 -    picref->video->w = w;
 -    picref->video->h = h;
 -
 -    for (i = 0; i < 3; i++)
 -        picref->data[i] += picref->linesize[i];
 -
 -    return picref;
 -}
 -
  static int return_frame(AVFilterContext *ctx, int is_second)
  {
      YADIFContext *yadif = ctx->priv;
          yadif->out->video->interlaced = 0;
      }
  
-     if (!yadif->csp)
-         yadif->csp = av_pix_fmt_desc_get(link->format);
-     if (yadif->csp->comp[0].depth_minus1 / 8 == 1)
-         yadif->filter_line = filter_line_c_16bit;
      filter(ctx, yadif->out, tff ^ !is_second, tff);
  
      if (is_second) {
@@@ -226,8 -217,6 +221,8 @@@ static int filter_frame(AVFilterLink *l
      AVFilterContext *ctx = link->dst;
      YADIFContext *yadif = ctx->priv;
  
 +    av_assert0(picref);
 +
      if (yadif->frame_pending)
          return_frame(ctx, 1);
  
      if (!yadif->cur)
          return 0;
  
 -    if (yadif->auto_enable && !yadif->cur->video->interlaced) {
 -        yadif->out  = avfilter_ref_buffer(yadif->cur, AV_PERM_READ);
 +    if (yadif->deint && !yadif->cur->video->interlaced) {
 +        yadif->out  = avfilter_ref_buffer(yadif->cur, ~AV_PERM_WRITE);
          if (!yadif->out)
              return AVERROR(ENOMEM);
  
      }
  
      if (!yadif->prev &&
 -        !(yadif->prev = avfilter_ref_buffer(yadif->cur, AV_PERM_READ)))
 +        !(yadif->prev = avfilter_ref_buffer(yadif->cur, ~AV_PERM_WRITE)))
          return AVERROR(ENOMEM);
  
      yadif->out = ff_get_video_buffer(ctx->outputs[0], PERM_RWP,
@@@ -287,8 -276,9 +282,8 @@@ static int request_frame(AVFilterLink *
  
          ret  = ff_request_frame(link->src->inputs[0]);
  
 -        if (ret == AVERROR_EOF && yadif->next) {
 -            AVFilterBufferRef *next =
 -                avfilter_ref_buffer(yadif->next, AV_PERM_READ);
 +        if (ret == AVERROR_EOF && yadif->cur) {
 +            AVFilterBufferRef *next = avfilter_ref_buffer(yadif->next, ~AV_PERM_WRITE);
  
              if (!next)
                  return AVERROR(ENOMEM);
      return 0;
  }
  
 -static int poll_frame(AVFilterLink *link)
 -{
 -    YADIFContext *yadif = link->src->priv;
 -    int ret, val;
 +#define OFFSET(x) offsetof(YADIFContext, x)
 +#define FLAGS AV_OPT_FLAG_VIDEO_PARAM|AV_OPT_FLAG_FILTERING_PARAM
  
 -    if (yadif->frame_pending)
 -        return 1;
 +#define CONST(name, help, val, unit) { name, help, 0, AV_OPT_TYPE_CONST, {.i64=val}, INT_MIN, INT_MAX, FLAGS, unit }
  
 -    val = ff_poll_frame(link->src->inputs[0]);
 -    if (val <= 0)
 -        return val;
 +static const AVOption yadif_options[] = {
 +    { "mode",   "specify the interlacing mode", OFFSET(mode), AV_OPT_TYPE_INT, {.i64=YADIF_MODE_SEND_FRAME}, 0, 3, FLAGS, "mode"},
 +    CONST("send_frame",           "send one frame for each frame",                                     YADIF_MODE_SEND_FRAME,           "mode"),
 +    CONST("send_field",           "send one frame for each field",                                     YADIF_MODE_SEND_FIELD,           "mode"),
 +    CONST("send_frame_nospatial", "send one frame for each frame, but skip spatial interlacing check", YADIF_MODE_SEND_FRAME_NOSPATIAL, "mode"),
 +    CONST("send_field_nospatial", "send one frame for each field, but skip spatial interlacing check", YADIF_MODE_SEND_FIELD_NOSPATIAL, "mode"),
  
 -    //FIXME change API to not requre this red tape
 -    if (val == 1 && !yadif->next) {
 -        if ((ret = ff_request_frame(link->src->inputs[0])) < 0)
 -            return ret;
 -        val = ff_poll_frame(link->src->inputs[0]);
 -        if (val <= 0)
 -            return val;
 -    }
 -    assert(yadif->next || !val);
 +    { "parity", "specify the assumed picture field parity", OFFSET(parity), AV_OPT_TYPE_INT, {.i64=YADIF_PARITY_AUTO}, -1, 1, FLAGS, "parity" },
 +    CONST("tff",  "assume top field first",    YADIF_PARITY_TFF,  "parity"),
 +    CONST("bff",  "assume bottom field first", YADIF_PARITY_BFF,  "parity"),
 +    CONST("auto", "auto detect parity",        YADIF_PARITY_AUTO, "parity"),
  
 -    if (yadif->auto_enable && yadif->next && !yadif->next->video->interlaced)
 -        return val;
 +    { "deint", "specify which frames to deinterlace", OFFSET(deint), AV_OPT_TYPE_INT, {.i64=YADIF_DEINT_ALL}, 0, 1, FLAGS, "deint" },
 +    CONST("all",        "deinterlace all frames",                       YADIF_DEINT_ALL,         "deint"),
 +    CONST("interlaced", "only deinterlace frames marked as interlaced", YADIF_DEINT_INTERLACED,  "deint"),
  
 -    return val * ((yadif->mode&1)+1);
 -}
 +    {NULL},
 +};
 +
 +AVFILTER_DEFINE_CLASS(yadif);
  
  static av_cold void uninit(AVFilterContext *ctx)
  {
      YADIFContext *yadif = ctx->priv;
  
 -    if (yadif->prev) avfilter_unref_bufferp(&yadif->prev);
 -    if (yadif->cur ) avfilter_unref_bufferp(&yadif->cur );
 -    if (yadif->next) avfilter_unref_bufferp(&yadif->next);
 +    avfilter_unref_bufferp(&yadif->prev);
 +    avfilter_unref_bufferp(&yadif->cur );
 +    avfilter_unref_bufferp(&yadif->next);
 +    av_freep(&yadif->temp_line); yadif->temp_line_size = 0;
 +    av_opt_free(yadif);
  }
  
  static int query_formats(AVFilterContext *ctx)
          AV_NE( AV_PIX_FMT_GRAY16BE, AV_PIX_FMT_GRAY16LE ),
          AV_PIX_FMT_YUV440P,
          AV_PIX_FMT_YUVJ440P,
 +        AV_NE( AV_PIX_FMT_YUV420P9BE,  AV_PIX_FMT_YUV420P9LE ),
 +        AV_NE( AV_PIX_FMT_YUV422P9BE,  AV_PIX_FMT_YUV422P9LE ),
 +        AV_NE( AV_PIX_FMT_YUV444P9BE,  AV_PIX_FMT_YUV444P9LE ),
          AV_NE( AV_PIX_FMT_YUV420P10BE, AV_PIX_FMT_YUV420P10LE ),
          AV_NE( AV_PIX_FMT_YUV422P10BE, AV_PIX_FMT_YUV422P10LE ),
          AV_NE( AV_PIX_FMT_YUV444P10BE, AV_PIX_FMT_YUV444P10LE ),
 +        AV_NE( AV_PIX_FMT_YUV420P12BE, AV_PIX_FMT_YUV420P12LE ),
 +        AV_NE( AV_PIX_FMT_YUV422P12BE, AV_PIX_FMT_YUV422P12LE ),
 +        AV_NE( AV_PIX_FMT_YUV444P12BE, AV_PIX_FMT_YUV444P12LE ),
 +        AV_NE( AV_PIX_FMT_YUV420P14BE, AV_PIX_FMT_YUV420P14LE ),
 +        AV_NE( AV_PIX_FMT_YUV422P14BE, AV_PIX_FMT_YUV422P14LE ),
 +        AV_NE( AV_PIX_FMT_YUV444P14BE, AV_PIX_FMT_YUV444P14LE ),
          AV_NE( AV_PIX_FMT_YUV420P16BE, AV_PIX_FMT_YUV420P16LE ),
          AV_NE( AV_PIX_FMT_YUV422P16BE, AV_PIX_FMT_YUV422P16LE ),
          AV_NE( AV_PIX_FMT_YUV444P16BE, AV_PIX_FMT_YUV444P16LE ),
          AV_PIX_FMT_YUVA420P,
 +        AV_PIX_FMT_YUVA422P,
 +        AV_PIX_FMT_YUVA444P,
          AV_PIX_FMT_NONE
      };
  
  static av_cold int init(AVFilterContext *ctx, const char *args)
  {
      YADIFContext *yadif = ctx->priv;
 +    static const char *shorthand[] = { "mode", "parity", "deint", NULL };
 +    int ret;
  
-     yadif->csp = NULL;
 -    yadif->mode = 0;
 -    yadif->parity = -1;
 -    yadif->auto_enable = 0;
 +    yadif->class = &yadif_class;
 +    av_opt_set_defaults(yadif);
  
 -    if (args)
 -        sscanf(args, "%d:%d:%d",
 -               &yadif->mode, &yadif->parity, &yadif->auto_enable);
 +    if ((ret = av_opt_set_from_string(yadif, args, shorthand, "=", ":")) < 0)
 +        return ret;
  
-     yadif->filter_line = filter_line_c;
-     if (ARCH_X86)
-         ff_yadif_init_x86(yadif);
 -    av_log(ctx, AV_LOG_VERBOSE, "mode:%d parity:%d auto_enable:%d\n",
 -           yadif->mode, yadif->parity, yadif->auto_enable);
 +    av_log(ctx, AV_LOG_VERBOSE, "mode:%d parity:%d deint:%d\n",
 +           yadif->mode, yadif->parity, yadif->deint);
  
      return 0;
  }
  
  static int config_props(AVFilterLink *link)
  {
-     YADIFContext *yadif = ctx->priv;
 +    AVFilterContext *ctx = link->src;
+     YADIFContext *s = link->src->priv;
  
      link->time_base.num = link->src->inputs[0]->time_base.num;
      link->time_base.den = link->src->inputs[0]->time_base.den * 2;
      link->w             = link->src->inputs[0]->w;
      link->h             = link->src->inputs[0]->h;
  
-     if(yadif->mode&1)
++    if(s->mode&1)
 +        link->frame_rate = av_mul_q(link->src->inputs[0]->frame_rate, (AVRational){2,1});
 +
 +    if (link->w < 3 || link->h < 3) {
 +        av_log(ctx, AV_LOG_ERROR, "Video of less than 3 columns or lines is not supported\n");
 +        return AVERROR(EINVAL);
 +    }
 +
+     s->csp = av_pix_fmt_desc_get(link->format);
+     if (s->csp->comp[0].depth_minus1 / 8 == 1) {
+         s->filter_line = filter_line_c_16bit;
+     } else {
+         s->filter_line = filter_line_c;
+         if (ARCH_X86)
+             ff_yadif_init_x86(s);
+     }
      return 0;
  }
  
@@@ -433,8 -406,8 +431,8 @@@ static const AVFilterPad avfilter_vf_ya
      {
          .name             = "default",
          .type             = AVMEDIA_TYPE_VIDEO,
 -        .get_video_buffer = get_video_buffer,
          .filter_frame     = filter_frame,
 +        .min_perms        = AV_PERM_PRESERVE,
      },
      { NULL }
  };
@@@ -443,6 -416,7 +441,6 @@@ static const AVFilterPad avfilter_vf_ya
      {
          .name          = "default",
          .type          = AVMEDIA_TYPE_VIDEO,
 -        .poll_frame    = poll_frame,
          .request_frame = request_frame,
          .config_props  = config_props,
      },
  
  AVFilter avfilter_vf_yadif = {
      .name          = "yadif",
 -    .description   = NULL_IF_CONFIG_SMALL("Deinterlace the input image"),
 +    .description   = NULL_IF_CONFIG_SMALL("Deinterlace the input image."),
  
      .priv_size     = sizeof(YADIFContext),
      .init          = init,
      .query_formats = query_formats,
  
      .inputs    = avfilter_vf_yadif_inputs,
 -
      .outputs   = avfilter_vf_yadif_outputs,
 +
 +    .priv_class = &yadif_class,
  };