Merge commit 'f6b6d6ac42c0ee427fcd3fae4c398b2103b491b8'
authorMichael Niedermayer <michaelni@gmx.at>
Thu, 16 May 2013 13:21:31 +0000 (15:21 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Thu, 16 May 2013 13:21:31 +0000 (15:21 +0200)
* commit 'f6b6d6ac42c0ee427fcd3fae4c398b2103b491b8':
  vf_frei0r: use the name 's' for the pointer to the private context
  vf_format: use the name 's' for the pointer to the private context
  vf_fieldorder: use the name 's' for the pointer to the private context
  vf_fieldorder: cosmetics, remove an extra linebreak

Conflicts:
libavfilter/vf_frei0r.c

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

Simple merge
@@@ -50,10 -49,10 +50,10 @@@ typedef struct 
  
  static av_cold int init(AVFilterContext *ctx)
  {
-     FormatContext *format = ctx->priv;
+     FormatContext *s = ctx->priv;
      const char *cur, *sep;
      char             pix_fmt_name[AV_PIX_FMT_NAME_MAXSIZE];
 -    int              pix_fmt_name_len;
 +    int              pix_fmt_name_len, ret;
      enum AVPixelFormat pix_fmt;
  
      /* parse the list of formats */
  
          memcpy(pix_fmt_name, cur, pix_fmt_name_len);
          pix_fmt_name[pix_fmt_name_len] = 0;
 -        pix_fmt = av_get_pix_fmt(pix_fmt_name);
  
 -        if (pix_fmt == AV_PIX_FMT_NONE) {
 -            av_log(ctx, AV_LOG_ERROR, "Unknown pixel format: %s\n", pix_fmt_name);
 -            return -1;
 -        }
 +        if ((ret = ff_parse_pixel_format(&pix_fmt, pix_fmt_name, ctx)) < 0)
 +            return ret;
  
-         format->listed_pix_fmt_flags[pix_fmt] = 1;
+         s->listed_pix_fmt_flags[pix_fmt] = 1;
      }
  
      return 0;
@@@ -136,13 -136,10 +136,13 @@@ fail
  
  static int set_params(AVFilterContext *ctx, const char *params)
  {
-     Frei0rContext *frei0r = ctx->priv;
+     Frei0rContext *s = ctx->priv;
      int i;
  
-     for (i = 0; i < frei0r->plugin_info.num_params; i++) {
 +    if (!params)
 +        return 0;
 +
+     for (i = 0; i < s->plugin_info.num_params; i++) {
          f0r_param_info_t info;
          char *param;
          int ret;
@@@ -237,54 -231,24 +237,54 @@@ static av_cold int frei0r_init(AVFilter
          return AVERROR(EINVAL);
      }
  
 -    /* see: http://piksel.org/frei0r/1.2/spec/1.2/spec/group__pluglocations.html */
 +    /* see: http://frei0r.dyne.org/codedoc/html/group__pluglocations.html */
      if ((path = av_strdup(getenv("FREI0R_PATH")))) {
 +#ifdef _WIN32
 +        const char *separator = ";";
 +#else
 +        const char *separator = ":";
 +#endif
          char *p, *ptr = NULL;
 -        for (p = path; p = strtok_r(p, ":", &ptr); p = NULL)
 -            if (s->dl_handle = load_path(ctx, p, dl_name))
 +        for (p = path; p = av_strtok(p, separator, &ptr); p = NULL) {
 +            /* add additional trailing slash in case it is missing */
 +            char *p1 = av_asprintf("%s/", p);
 +            if (!p1) {
 +                ret = AVERROR(ENOMEM);
 +                goto check_path_end;
 +            }
-             ret = load_path(ctx, &frei0r->dl_handle, p1, dl_name);
++            ret = load_path(ctx, &s->dl_handle, p1, dl_name);
 +            av_free(p1);
 +            if (ret < 0)
 +                goto check_path_end;
-             if (frei0r->dl_handle)
++            if (s->dl_handle)
                  break;
 +        }
 +
 +    check_path_end:
          av_free(path);
 +        if (ret < 0)
 +            return ret;
      }
-     if (!frei0r->dl_handle && (path = getenv("HOME"))) {
+     if (!s->dl_handle && (path = getenv("HOME"))) {
 -        char prefix[1024];
 -        snprintf(prefix, sizeof(prefix), "%s/.frei0r-1/lib/", path);
 -        s->dl_handle = load_path(ctx, prefix, dl_name);
 +        char *prefix = av_asprintf("%s/.frei0r-1/lib/", path);
 +        if (!prefix)
 +            return AVERROR(ENOMEM);
-         ret = load_path(ctx, &frei0r->dl_handle, prefix, dl_name);
++        ret = load_path(ctx, &s->dl_handle, prefix, dl_name);
 +        av_free(prefix);
 +        if (ret < 0)
 +            return ret;
 +    }
-     if (!frei0r->dl_handle) {
-         ret = load_path(ctx, &frei0r->dl_handle, "/usr/local/lib/frei0r-1/", dl_name);
++    if (!s->dl_handle) {
++        ret = load_path(ctx, &s->dl_handle, "/usr/local/lib/frei0r-1/", dl_name);
 +        if (ret < 0)
 +            return ret;
 +    }
-     if (!frei0r->dl_handle) {
-         ret = load_path(ctx, &frei0r->dl_handle, "/usr/lib/frei0r-1/", dl_name);
++    if (!s->dl_handle) {
++        ret = load_path(ctx, &s->dl_handle, "/usr/lib/frei0r-1/", dl_name);
 +        if (ret < 0)
 +            return ret;
      }
-     if (!frei0r->dl_handle) {
 -    if (!s->dl_handle)
 -        s->dl_handle = load_path(ctx, "/usr/local/lib/frei0r-1/", dl_name);
 -    if (!s->dl_handle)
 -        s->dl_handle = load_path(ctx, "/usr/lib/frei0r-1/", dl_name);
+     if (!s->dl_handle) {
          av_log(ctx, AV_LOG_ERROR, "Could not find module '%s'\n", dl_name);
          return AVERROR(EINVAL);
      }
@@@ -351,10 -315,10 +351,10 @@@ static av_cold void uninit(AVFilterCont
  static int config_input_props(AVFilterLink *inlink)
  {
      AVFilterContext *ctx = inlink->dst;
-     Frei0rContext *frei0r = ctx->priv;
+     Frei0rContext *s = ctx->priv;
  
-     if (!(frei0r->instance = frei0r->construct(inlink->w, inlink->h))) {
+     if (!(s->instance = s->construct(inlink->w, inlink->h))) {
 -        av_log(ctx, AV_LOG_ERROR, "Impossible to load frei0r instance");
 +        av_log(ctx, AV_LOG_ERROR, "Impossible to load frei0r instance\n");
          return AVERROR(EINVAL);
      }
  
@@@ -465,30 -429,30 +465,30 @@@ static av_cold int source_init(AVFilter
          return AVERROR(EINVAL);
      }
  
-     if (av_parse_video_rate(&frame_rate_q, frei0r->framerate) < 0) {
-         av_log(ctx, AV_LOG_ERROR, "Invalid frame rate: '%s'\n", frei0r->framerate);
 -    if (av_parse_video_rate(&frame_rate_q, s->framerate) < 0 ||
 -        frame_rate_q.den <= 0 || frame_rate_q.num <= 0) {
++    if (av_parse_video_rate(&frame_rate_q, s->framerate) < 0) {
+         av_log(ctx, AV_LOG_ERROR, "Invalid frame rate: '%s'\n", s->framerate);
          return AVERROR(EINVAL);
      }
-     frei0r->time_base.num = frame_rate_q.den;
-     frei0r->time_base.den = frame_rate_q.num;
+     s->time_base.num = frame_rate_q.den;
+     s->time_base.den = frame_rate_q.num;
  
-     return frei0r_init(ctx, frei0r->dl_name, F0R_PLUGIN_TYPE_SOURCE);
+     return frei0r_init(ctx, s->dl_name, F0R_PLUGIN_TYPE_SOURCE);
  }
  
  static int source_config_props(AVFilterLink *outlink)
  {
      AVFilterContext *ctx = outlink->src;
-     Frei0rContext *frei0r = ctx->priv;
+     Frei0rContext *s = ctx->priv;
  
-     if (av_image_check_size(frei0r->w, frei0r->h, 0, ctx) < 0)
+     if (av_image_check_size(s->w, s->h, 0, ctx) < 0)
          return AVERROR(EINVAL);
-     outlink->w = frei0r->w;
-     outlink->h = frei0r->h;
-     outlink->time_base = frei0r->time_base;
+     outlink->w = s->w;
+     outlink->h = s->h;
+     outlink->time_base = s->time_base;
 +    outlink->sample_aspect_ratio = (AVRational){1,1};
  
-     if (!(frei0r->instance = frei0r->construct(outlink->w, outlink->h))) {
+     if (!(s->instance = s->construct(outlink->w, outlink->h))) {
 -        av_log(ctx, AV_LOG_ERROR, "Impossible to load frei0r instance");
 +        av_log(ctx, AV_LOG_ERROR, "Impossible to load frei0r instance\n");
          return AVERROR(EINVAL);
      }