Merge commit '21bc44038491e525335ad763883b0288402ff6a0'
authorMichael Niedermayer <michaelni@gmx.at>
Wed, 5 Sep 2012 12:52:52 +0000 (14:52 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Wed, 5 Sep 2012 12:52:52 +0000 (14:52 +0200)
* commit '21bc44038491e525335ad763883b0288402ff6a0':
  avopt: Explicitly store rational option defaults in .dbl

Conflicts:
libavfilter/buffersrc.c
libavfilter/vsrc_testsrc.c

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

diff --combined libavfilter/buffersrc.c
@@@ -1,20 -1,20 +1,20 @@@
  /*
   * Copyright (c) 2008 Vitor Sessak
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 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
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -29,7 -29,6 +29,7 @@@
  #include "formats.h"
  #include "internal.h"
  #include "video.h"
 +#include "avcodec.h"
  
  #include "libavutil/audioconvert.h"
  #include "libavutil/common.h"
@@@ -42,15 -41,11 +42,15 @@@ typedef struct 
      const AVClass    *class;
      AVFifoBuffer     *fifo;
      AVRational        time_base;     ///< time_base to set in the output link
 +    AVRational        frame_rate;    ///< frame_rate to set in the output link
 +    unsigned          nb_failed_requests;
 +    unsigned          warning_limit;
  
      /* video only */
 -    int               h, w;
 +    int               w, h;
      enum PixelFormat  pix_fmt;
      AVRational        pixel_aspect;
 +    char              *sws_param;
  
      /* audio only */
      int sample_rate;
          return AVERROR(EINVAL);\
      }
  
 -int av_buffersrc_write_frame(AVFilterContext *buffer_filter, AVFrame *frame)
 +int av_buffersrc_add_frame(AVFilterContext *buffer_src,
 +                           const AVFrame *frame, int flags)
  {
 -    BufferSourceContext *c = buffer_filter->priv;
 -    AVFilterBufferRef *buf;
 +    AVFilterBufferRef *picref;
      int ret;
  
 -    if (!frame) {
 -        c->eof = 1;
 -        return 0;
 -    } else if (c->eof)
 -        return AVERROR(EINVAL);
 -
 -    if (!av_fifo_space(c->fifo) &&
 -        (ret = av_fifo_realloc2(c->fifo, av_fifo_size(c->fifo) +
 -                                         sizeof(buf))) < 0)
 -        return ret;
 -
 -    switch (buffer_filter->outputs[0]->type) {
 -    case AVMEDIA_TYPE_VIDEO:
 -        CHECK_VIDEO_PARAM_CHANGE(buffer_filter, c, frame->width, frame->height,
 -                                 frame->format);
 -        buf = ff_get_video_buffer(buffer_filter->outputs[0], AV_PERM_WRITE,
 -                                  c->w, c->h);
 -        if (!buf)
 -            return AVERROR(ENOMEM);
 -
 -        av_image_copy(buf->data, buf->linesize, frame->data, frame->linesize,
 -                      c->pix_fmt, c->w, c->h);
 -        break;
 -    case AVMEDIA_TYPE_AUDIO:
 -        CHECK_AUDIO_PARAM_CHANGE(buffer_filter, c, frame->sample_rate, frame->channel_layout,
 -                                 frame->format);
 -        buf = ff_get_audio_buffer(buffer_filter->outputs[0], AV_PERM_WRITE,
 -                                  frame->nb_samples);
 -        if (!buf)
 -            return AVERROR(ENOMEM);
 -
 -        av_samples_copy(buf->extended_data, frame->extended_data,
 -                        0, 0, frame->nb_samples,
 -                        av_get_channel_layout_nb_channels(frame->channel_layout),
 -                        frame->format);
 -        break;
 -    default:
 -        return AVERROR(EINVAL);
 -    }
 -
 -    avfilter_copy_frame_props(buf, frame);
 +    if (!frame) /* NULL for EOF */
 +        return av_buffersrc_add_ref(buffer_src, NULL, flags);
  
 -    if ((ret = av_fifo_generic_write(c->fifo, &buf, sizeof(buf), NULL)) < 0) {
 -        avfilter_unref_buffer(buf);
 -        return ret;
 -    }
 +    picref = avfilter_get_buffer_ref_from_frame(buffer_src->outputs[0]->type,
 +                                                frame, AV_PERM_WRITE);
 +    if (!picref)
 +        return AVERROR(ENOMEM);
 +    ret = av_buffersrc_add_ref(buffer_src, picref, flags);
 +    picref->buf->data[0] = NULL;
 +    avfilter_unref_buffer(picref);
 +    return ret;
 +}
  
 -    return 0;
 +int av_buffersrc_write_frame(AVFilterContext *buffer_filter, AVFrame *frame)
 +{
 +    return av_buffersrc_add_frame(buffer_filter, frame, 0);
  }
  
 -int av_buffersrc_buffer(AVFilterContext *s, AVFilterBufferRef *buf)
 +int av_buffersrc_add_ref(AVFilterContext *s, AVFilterBufferRef *buf, int flags)
  {
      BufferSourceContext *c = s->priv;
 +    AVFilterBufferRef *to_free = NULL;
      int ret;
  
      if (!buf) {
                                           sizeof(buf))) < 0)
          return ret;
  
 -    switch (s->outputs[0]->type) {
 -    case AVMEDIA_TYPE_VIDEO:
 -        CHECK_VIDEO_PARAM_CHANGE(s, c, buf->video->w, buf->video->h, buf->format);
 -        break;
 -    case AVMEDIA_TYPE_AUDIO:
 -        CHECK_AUDIO_PARAM_CHANGE(s, c, buf->audio->sample_rate, buf->audio->channel_layout,
 -                                 buf->format);
 -        break;
 -    default:
 -        return AVERROR(EINVAL);
 +    if (!(flags & AV_BUFFERSRC_FLAG_NO_CHECK_FORMAT)) {
 +        switch (s->outputs[0]->type) {
 +        case AVMEDIA_TYPE_VIDEO:
 +            CHECK_VIDEO_PARAM_CHANGE(s, c, buf->video->w, buf->video->h, buf->format);
 +            break;
 +        case AVMEDIA_TYPE_AUDIO:
 +            CHECK_AUDIO_PARAM_CHANGE(s, c, buf->audio->sample_rate, buf->audio->channel_layout,
 +                                     buf->format);
 +            break;
 +        default:
 +            return AVERROR(EINVAL);
 +        }
      }
 +    if (!(flags & AV_BUFFERSRC_FLAG_NO_COPY))
 +        to_free = buf = ff_copy_buffer_ref(s->outputs[0], buf);
 +    if(!buf)
 +        return -1;
  
 -    if ((ret = av_fifo_generic_write(c->fifo, &buf, sizeof(buf), NULL)) < 0)
 +    if ((ret = av_fifo_generic_write(c->fifo, &buf, sizeof(buf), NULL)) < 0) {
 +        avfilter_unref_buffer(to_free);
          return ret;
 +    }
 +    c->nb_failed_requests = 0;
 +    if (c->warning_limit &&
 +        av_fifo_size(c->fifo) / sizeof(buf) >= c->warning_limit) {
 +        av_log(s, AV_LOG_WARNING,
 +               "%d buffers queued in %s, something may be wrong.\n",
 +               c->warning_limit,
 +               (char *)av_x_if_null(s->name, s->filter->name));
 +        c->warning_limit *= 10;
 +    }
 +
 +    if ((flags & AV_BUFFERSRC_FLAG_PUSH))
 +        if ((ret = s->output_pads[0].request_frame(s->outputs[0])) < 0)
 +            return ret;
  
      return 0;
  }
  
-     { "time_base",      NULL, OFFSET(time_base),           AV_OPT_TYPE_RATIONAL,   { 0 }, 0, INT_MAX, FLAGS },
-     { "frame_rate",     NULL, OFFSET(frame_rate),          AV_OPT_TYPE_RATIONAL,   { 0 }, 0, INT_MAX, FLAGS },
 +#ifdef FF_API_BUFFERSRC_BUFFER
 +int av_buffersrc_buffer(AVFilterContext *s, AVFilterBufferRef *buf)
 +{
 +    return av_buffersrc_add_ref(s, buf, AV_BUFFERSRC_FLAG_NO_COPY);
 +}
 +#endif
 +
 +unsigned av_buffersrc_get_nb_failed_requests(AVFilterContext *buffer_src)
 +{
 +    return ((BufferSourceContext *)buffer_src->priv)->nb_failed_requests;
 +}
 +
 +#define OFFSET(x) offsetof(BufferSourceContext, x)
 +#define FLAGS AV_OPT_FLAG_FILTERING_PARAM|AV_OPT_FLAG_VIDEO_PARAM
 +static const AVOption buffer_options[] = {
-     { "pixel_aspect",   NULL, OFFSET(pixel_aspect),        AV_OPT_TYPE_RATIONAL,   { 0 }, 0, INT_MAX, FLAGS },
++    { "time_base",      NULL, OFFSET(time_base),           AV_OPT_TYPE_RATIONAL,   { .dbl = 0 }, 0, INT_MAX, FLAGS },
++    { "frame_rate",     NULL, OFFSET(frame_rate),          AV_OPT_TYPE_RATIONAL,   { .dbl = 0 }, 0, INT_MAX, FLAGS },
 +    { "video_size",     NULL, OFFSET(w),                   AV_OPT_TYPE_IMAGE_SIZE, .flags = FLAGS },
 +    { "pix_fmt",        NULL, OFFSET(pix_fmt),             AV_OPT_TYPE_PIXEL_FMT,  .flags = FLAGS },
++    { "pixel_aspect",   NULL, OFFSET(pixel_aspect),        AV_OPT_TYPE_RATIONAL,   { .dbl = 0 }, 0, INT_MAX, FLAGS },
 +    { "sws_param",      NULL, OFFSET(sws_param),           AV_OPT_TYPE_STRING,     .flags = FLAGS },
 +    { NULL },
 +};
 +#undef FLAGS
 +
 +AVFILTER_DEFINE_CLASS(buffer);
 +
  static av_cold int init_video(AVFilterContext *ctx, const char *args)
  {
      BufferSourceContext *c = ctx->priv;
 -    char pix_fmt_str[128];
 -    int n = 0;
 +    char pix_fmt_str[128], sws_param[256] = "", *colon, *equal;
 +    int ret, n = 0;
  
 -    if (!args ||
 -        (n = sscanf(args, "%d:%d:%127[^:]:%d:%d:%d:%d", &c->w, &c->h, pix_fmt_str,
 -                    &c->time_base.num, &c->time_base.den,
 -                    &c->pixel_aspect.num, &c->pixel_aspect.den)) != 7) {
 -        av_log(ctx, AV_LOG_ERROR, "Expected 7 arguments, but %d found in '%s'\n", n, args);
 +    c->class = &buffer_class;
 +
 +    if (!args) {
 +        av_log(ctx, AV_LOG_ERROR, "Arguments required\n");
          return AVERROR(EINVAL);
      }
 -    if ((c->pix_fmt = av_get_pix_fmt(pix_fmt_str)) == PIX_FMT_NONE) {
 -        char *tail;
 -        c->pix_fmt = strtol(pix_fmt_str, &tail, 10);
 -        if (*tail || c->pix_fmt < 0 || c->pix_fmt >= PIX_FMT_NB) {
 -            av_log(ctx, AV_LOG_ERROR, "Invalid pixel format string '%s'\n", pix_fmt_str);
 -            return AVERROR(EINVAL);
 -        }
 +    colon = strchr(args, ':');
 +    equal = strchr(args, '=');
 +    if (equal && (!colon || equal < colon)) {
 +        av_opt_set_defaults(c);
 +        ret = av_set_options_string(c, args, "=", ":");
 +        if (ret < 0)
 +            goto fail;
 +    } else {
 +    if ((n = sscanf(args, "%d:%d:%127[^:]:%d:%d:%d:%d:%255c", &c->w, &c->h, pix_fmt_str,
 +                    &c->time_base.num, &c->time_base.den,
 +                    &c->pixel_aspect.num, &c->pixel_aspect.den, sws_param)) < 7) {
 +        av_log(ctx, AV_LOG_ERROR, "Expected at least 7 arguments, but only %d found in '%s'\n", n, args);
 +        ret = AVERROR(EINVAL);
 +        goto fail;
      }
 +    av_log(ctx, AV_LOG_WARNING, "Flat options syntax is deprecated, use key=value pairs\n");
  
 -    if (!(c->fifo = av_fifo_alloc(sizeof(AVFilterBufferRef*))))
 -        return AVERROR(ENOMEM);
 +    if ((ret = ff_parse_pixel_format(&c->pix_fmt, pix_fmt_str, ctx)) < 0)
 +        goto fail;
 +    c->sws_param = av_strdup(sws_param);
 +    if (!c->sws_param) {
 +        ret = AVERROR(ENOMEM);
 +        goto fail;
 +    }
 +    }
  
 -    av_log(ctx, AV_LOG_VERBOSE, "w:%d h:%d pixfmt:%s\n", c->w, c->h, av_pix_fmt_descriptors[c->pix_fmt].name);
 +    if (!(c->fifo = av_fifo_alloc(sizeof(AVFilterBufferRef*)))) {
 +        ret = AVERROR(ENOMEM);
 +        goto fail;
 +    }
 +
 +    av_log(ctx, AV_LOG_VERBOSE, "w:%d h:%d pixfmt:%s tb:%d/%d fr:%d/%d sar:%d/%d sws_param:%s\n",
 +           c->w, c->h, av_pix_fmt_descriptors[c->pix_fmt].name,
 +           c->time_base.num, c->time_base.den, c->frame_rate.num, c->frame_rate.den,
 +           c->pixel_aspect.num, c->pixel_aspect.den, (char *)av_x_if_null(c->sws_param, ""));
 +    c->warning_limit = 100;
      return 0;
 +
 +fail:
 +    av_opt_free(c);
 +    return ret;
  }
  
 -#define OFFSET(x) offsetof(BufferSourceContext, x)
 -#define A AV_OPT_FLAG_AUDIO_PARAM
 -static const AVOption audio_options[] = {
 -    { "time_base",      NULL, OFFSET(time_base),           AV_OPT_TYPE_RATIONAL, { .dbl = 0 }, 0, INT_MAX, A },
 -    { "sample_rate",    NULL, OFFSET(sample_rate),         AV_OPT_TYPE_INT,      { .i64 = 0 }, 0, INT_MAX, A },
 -    { "sample_fmt",     NULL, OFFSET(sample_fmt_str),      AV_OPT_TYPE_STRING,             .flags = A },
 -    { "channel_layout", NULL, OFFSET(channel_layout_str),  AV_OPT_TYPE_STRING,             .flags = A },
 +#define FLAGS AV_OPT_FLAG_FILTERING_PARAM|AV_OPT_FLAG_AUDIO_PARAM
 +static const AVOption abuffer_options[] = {
-     { "time_base",      NULL, OFFSET(time_base),           AV_OPT_TYPE_RATIONAL, { 0 }, 0, INT_MAX, FLAGS },
++    { "time_base",      NULL, OFFSET(time_base),           AV_OPT_TYPE_RATIONAL, { .dbl = 0 }, 0, INT_MAX, FLAGS },
 +    { "sample_rate",    NULL, OFFSET(sample_rate),         AV_OPT_TYPE_INT,      { .i64 = 0 }, 0, INT_MAX, FLAGS },
 +    { "sample_fmt",     NULL, OFFSET(sample_fmt_str),      AV_OPT_TYPE_STRING, .flags = FLAGS },
 +    { "channel_layout", NULL, OFFSET(channel_layout_str),  AV_OPT_TYPE_STRING, .flags = FLAGS },
      { NULL },
  };
  
 -static const AVClass abuffer_class = {
 -    .class_name = "abuffer source",
 -    .item_name  = av_default_item_name,
 -    .option     = audio_options,
 -    .version    = LIBAVUTIL_VERSION_INT,
 -};
 +AVFILTER_DEFINE_CLASS(abuffer);
  
  static av_cold int init_audio(AVFilterContext *ctx, const char *args)
  {
      s->class = &abuffer_class;
      av_opt_set_defaults(s);
  
 -    if ((ret = av_set_options_string(s, args, "=", ":")) < 0) {
 -        av_log(ctx, AV_LOG_ERROR, "Error parsing options string: %s.\n", args);
 +    if ((ret = av_set_options_string(s, args, "=", ":")) < 0)
          goto fail;
 -    }
  
      s->sample_fmt = av_get_sample_fmt(s->sample_fmt_str);
      if (s->sample_fmt == AV_SAMPLE_FMT_NONE) {
 -        av_log(ctx, AV_LOG_ERROR, "Invalid sample format %s.\n",
 +        av_log(ctx, AV_LOG_ERROR, "Invalid sample format '%s'\n",
                 s->sample_fmt_str);
          ret = AVERROR(EINVAL);
          goto fail;
  
      s->channel_layout = av_get_channel_layout(s->channel_layout_str);
      if (!s->channel_layout) {
 -        av_log(ctx, AV_LOG_ERROR, "Invalid channel layout %s.\n",
 +        av_log(ctx, AV_LOG_ERROR, "Invalid channel layout '%s'\n",
                 s->channel_layout_str);
          ret = AVERROR(EINVAL);
          goto fail;
      if (!s->time_base.num)
          s->time_base = (AVRational){1, s->sample_rate};
  
 -    av_log(ctx, AV_LOG_VERBOSE, "tb:%d/%d samplefmt:%s samplerate: %d "
 -           "ch layout:%s\n", s->time_base.num, s->time_base.den, s->sample_fmt_str,
 +    av_log(ctx, AV_LOG_VERBOSE,
 +           "tb:%d/%d samplefmt:%s samplerate:%d chlayout:%s\n",
 +           s->time_base.num, s->time_base.den, s->sample_fmt_str,
             s->sample_rate, s->channel_layout_str);
 +    s->warning_limit = 100;
  
  fail:
      av_opt_free(s);
@@@ -304,7 -262,6 +304,7 @@@ static av_cold void uninit(AVFilterCont
      }
      av_fifo_free(s->fifo);
      s->fifo = NULL;
 +    av_freep(&s->sws_param);
  }
  
  static int query_formats(AVFilterContext *ctx)
@@@ -355,7 -312,6 +355,7 @@@ static int config_props(AVFilterLink *l
      }
  
      link->time_base = c->time_base;
 +    link->frame_rate = c->frame_rate;
      return 0;
  }
  
@@@ -368,7 -324,6 +368,7 @@@ static int request_frame(AVFilterLink *
      if (!av_fifo_size(c->fifo)) {
          if (c->eof)
              return AVERROR_EOF;
 +        c->nb_failed_requests++;
          return AVERROR(EAGAIN);
      }
      av_fifo_generic_read(c->fifo, &buf, sizeof(buf), NULL);
@@@ -416,7 -371,6 +416,7 @@@ AVFilter avfilter_vsrc_buffer = 
                                            .poll_frame      = poll_frame,
                                            .config_props    = config_props, },
                                          { .name = NULL}},
 +    .priv_class = &buffer_class,
  };
  
  AVFilter avfilter_asrc_abuffer = {
                                            .poll_frame      = poll_frame,
                                            .config_props    = config_props, },
                                          { .name = NULL}},
 +    .priv_class = &abuffer_class,
  };
diff --combined libavutil/opt.c
@@@ -2,20 -2,20 +2,20 @@@
   * AVOptions
   * Copyright (c) 2005 Michael Niedermayer <michaelni@gmx.at>
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 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
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
  #include "eval.h"
  #include "dict.h"
  #include "log.h"
 +#include "parseutils.h"
 +#include "pixdesc.h"
  #include "mathematics.h"
  
  #if FF_API_FIND_OPT
  //FIXME order them and do a bin search
  const AVOption *av_find_opt(void *v, const char *name, const char *unit, int mask, int flags)
  {
 -    AVClass *c= *(AVClass**)v; //FIXME silly way of storing AVClass
 -    const AVOption *o= c->option;
 +    const AVOption *o = NULL;
  
 -    for (; o && o->name; o++) {
 +    while ((o = av_next_option(v, o))) {
          if (!strcmp(o->name, name) && (!unit || (o->unit && !strcmp(o->unit, unit))) && (o->flags & mask) == flags)
              return o;
      }
@@@ -60,8 -59,7 +60,8 @@@ const AVOption *av_next_option(void *ob
  const AVOption *av_opt_next(void *obj, const AVOption *last)
  {
      AVClass *class = *(AVClass**)obj;
 -    if (!last && class->option[0].name) return class->option;
 +    if (!last && class->option && class->option[0].name)
 +        return class->option;
      if (last && last[1].name)           return ++last;
      return NULL;
  }
@@@ -77,7 -75,6 +77,7 @@@ static int read_number(const AVOption *
      case AV_OPT_TYPE_RATIONAL:  *intnum = ((AVRational*)dst)->num;
                                  *den    = ((AVRational*)dst)->den;
                                                          return 0;
 +    case AV_OPT_TYPE_CONST:     *num    = o->default_val.dbl; return 0;
      }
      return AVERROR(EINVAL);
  }
@@@ -85,7 -82,7 +85,7 @@@
  static int write_number(void *obj, const AVOption *o, void *dst, double num, int den, int64_t intnum)
  {
      if (o->max*den < num*intnum || o->min*den > num*intnum) {
 -        av_log(obj, AV_LOG_ERROR, "Value %lf for parameter '%s' out of range\n",
 +        av_log(obj, AV_LOG_ERROR, "Value %f for parameter '%s' out of range\n",
                 num*intnum/den, o->name);
          return AVERROR(ERANGE);
      }
@@@ -236,12 -233,11 +236,12 @@@ int av_set_string3(void *obj, const cha
  
  int av_opt_set(void *obj, const char *name, const char *val, int search_flags)
  {
 +    int ret;
      void *dst, *target_obj;
      const AVOption *o = av_opt_find2(obj, name, NULL, 0, search_flags, &target_obj);
      if (!o || !target_obj)
          return AVERROR_OPTION_NOT_FOUND;
 -    if (!val)
 +    if (!val && o->type != AV_OPT_TYPE_STRING)
          return AVERROR(EINVAL);
  
      dst = ((uint8_t*)target_obj) + o->offset;
      case AV_OPT_TYPE_FLOAT:
      case AV_OPT_TYPE_DOUBLE:
      case AV_OPT_TYPE_RATIONAL: return set_string_number(obj, o, val, dst);
 +    case AV_OPT_TYPE_IMAGE_SIZE:
 +        ret = av_parse_video_size(dst, ((int *)dst) + 1, val);
 +        if (ret < 0)
 +            av_log(obj, AV_LOG_ERROR, "Unable to parse option value \"%s\" as image size\n", val);
 +        return ret;
 +    case AV_OPT_TYPE_PIXEL_FMT:
 +        ret = av_get_pix_fmt(val);
 +        if (ret == PIX_FMT_NONE) {
 +            char *tail;
 +            ret = strtol(val, &tail, 0);
 +            if (*tail || (unsigned)ret >= PIX_FMT_NB) {
 +                av_log(obj, AV_LOG_ERROR, "Unable to parse option value \"%s\" as pixel format\n", val);
 +                return AVERROR(EINVAL);
 +            }
 +        }
 +        *(enum PixelFormat *)dst = ret;
 +        return 0;
      }
  
      av_log(obj, AV_LOG_ERROR, "Invalid option type.\n");
@@@ -383,7 -362,7 +383,7 @@@ int av_opt_set_bin(void *obj, const cha
   */
  const char *av_get_string(void *obj, const char *name, const AVOption **o_out, char *buf, int buf_len)
  {
 -    const AVOption *o = av_opt_find(obj, name, NULL, 0, 0);
 +    const AVOption *o = av_opt_find(obj, name, NULL, 0, AV_OPT_SEARCH_CHILDREN);
      void *dst;
      uint8_t *bin;
      int len, i;
      case AV_OPT_TYPE_FLOAT:     snprintf(buf, buf_len, "%f" , *(float  *)dst);break;
      case AV_OPT_TYPE_DOUBLE:    snprintf(buf, buf_len, "%f" , *(double *)dst);break;
      case AV_OPT_TYPE_RATIONAL:  snprintf(buf, buf_len, "%d/%d", ((AVRational*)dst)->num, ((AVRational*)dst)->den);break;
 +    case AV_OPT_TYPE_CONST:     snprintf(buf, buf_len, "%f" , o->default_val.dbl);break;
      case AV_OPT_TYPE_STRING:    return *(void**)dst;
      case AV_OPT_TYPE_BINARY:
          len = *(int*)(((uint8_t *)dst) + sizeof(uint8_t *));
@@@ -423,7 -401,7 +423,7 @@@ int av_opt_get(void *obj, const char *n
      uint8_t *bin, buf[128];
      int len, i, ret;
  
 -    if (!o || !target_obj)
 +    if (!o || !target_obj || (o->offset<=0 && o->type != AV_OPT_TYPE_CONST))
          return AVERROR_OPTION_NOT_FOUND;
  
      dst = (uint8_t*)target_obj + o->offset;
      case AV_OPT_TYPE_FLOAT:     ret = snprintf(buf, sizeof(buf), "%f" ,     *(float  *)dst);break;
      case AV_OPT_TYPE_DOUBLE:    ret = snprintf(buf, sizeof(buf), "%f" ,     *(double *)dst);break;
      case AV_OPT_TYPE_RATIONAL:  ret = snprintf(buf, sizeof(buf), "%d/%d",   ((AVRational*)dst)->num, ((AVRational*)dst)->den);break;
 +    case AV_OPT_TYPE_CONST:     ret = snprintf(buf, sizeof(buf), "%f" ,     o->default_val.dbl);break;
      case AV_OPT_TYPE_STRING:
          if (*(uint8_t**)dst)
              *out_val = av_strdup(*(uint8_t**)dst);
          for (i = 0; i < len; i++)
              snprintf(*out_val + i*2, 3, "%02X", bin[i]);
          return 0;
 +    case AV_OPT_TYPE_IMAGE_SIZE:
 +        ret = snprintf(buf, sizeof(buf), "%dx%d", ((int *)dst)[0], ((int *)dst)[1]);
 +        break;
 +    case AV_OPT_TYPE_PIXEL_FMT:
 +        ret = snprintf(buf, sizeof(buf), "%s", (char *)av_x_if_null(av_get_pix_fmt_name(*(enum PixelFormat *)dst), "?"));
 +        break;
      default:
          return AVERROR(EINVAL);
      }
@@@ -628,12 -599,6 +628,12 @@@ static void opt_list(void *obj, void *a
              case AV_OPT_TYPE_BINARY:
                  av_log(av_log_obj, AV_LOG_INFO, "%-7s ", "<binary>");
                  break;
 +            case AV_OPT_TYPE_IMAGE_SIZE:
 +                av_log(av_log_obj, AV_LOG_INFO, "%-7s ", "<image_size>");
 +                break;
 +            case AV_OPT_TYPE_PIXEL_FMT:
 +                av_log(av_log_obj, AV_LOG_INFO, "%-7s ", "<pix_fmt>");
 +                break;
              case AV_OPT_TYPE_CONST:
              default:
                  av_log(av_log_obj, AV_LOG_INFO, "%-7s ", "");
          }
          av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_ENCODING_PARAM) ? 'E' : '.');
          av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_DECODING_PARAM) ? 'D' : '.');
 +        av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_FILTERING_PARAM)? 'F' : '.');
          av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_VIDEO_PARAM   ) ? 'V' : '.');
          av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_AUDIO_PARAM   ) ? 'A' : '.');
          av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_SUBTITLE_PARAM) ? 'S' : '.');
@@@ -705,8 -669,6 +705,8 @@@ void av_opt_set_defaults2(void *s, int 
              }
              break;
              case AV_OPT_TYPE_STRING:
 +            case AV_OPT_TYPE_IMAGE_SIZE:
 +            case AV_OPT_TYPE_PIXEL_FMT:
                  av_opt_set(s, opt->name, opt->default_val.str, 0);
                  break;
              case AV_OPT_TYPE_BINARY:
@@@ -751,7 -713,7 +751,7 @@@ static int parse_key_value_pair(void *c
          return AVERROR(EINVAL);
      }
  
 -    av_log(ctx, AV_LOG_DEBUG, "Setting value '%s' for key '%s'\n", val, key);
 +    av_log(ctx, AV_LOG_DEBUG, "Setting entry with key '%s' to value '%s'\n", key, val);
  
      ret = av_opt_set(ctx, key, val, 0);
      if (ret == AVERROR_OPTION_NOT_FOUND)
@@@ -820,14 -782,9 +820,14 @@@ const AVOption *av_opt_find(void *obj, 
  const AVOption *av_opt_find2(void *obj, const char *name, const char *unit,
                               int opt_flags, int search_flags, void **target_obj)
  {
 -    const AVClass  *c = *(AVClass**)obj;
 +    const AVClass  *c;
      const AVOption *o = NULL;
  
 +    if(!obj)
 +        return NULL;
 +
 +    c= *(AVClass**)obj;
 +
      if (search_flags & AV_OPT_SEARCH_CHILDREN) {
          if (search_flags & AV_OPT_SEARCH_FAKE_OBJ) {
              const AVClass *child = NULL;
      while (o = av_opt_next(obj, o)) {
          if (!strcmp(o->name, name) && (o->flags & opt_flags) == opt_flags &&
              ((!unit && o->type != AV_OPT_TYPE_CONST) ||
 -             (unit  && o->unit && !strcmp(o->unit, unit)))) {
 +             (unit  && o->type == AV_OPT_TYPE_CONST && o->unit && !strcmp(o->unit, unit)))) {
              if (target_obj) {
                  if (!(search_flags & AV_OPT_SEARCH_FAKE_OBJ))
                      *target_obj = obj;
@@@ -873,14 -830,6 +873,14 @@@ const AVClass *av_opt_child_class_next(
      return NULL;
  }
  
 +void *av_opt_ptr(const AVClass *class, void *obj, const char *name)
 +{
 +    const AVOption *opt= av_opt_find2(&class, name, NULL, 0, AV_OPT_SEARCH_FAKE_OBJ, NULL);
 +    if(!opt)
 +        return NULL;
 +    return (uint8_t*)obj + opt->offset;
 +}
 +
  #ifdef TEST
  
  #undef printf
@@@ -893,8 -842,6 +893,8 @@@ typedef struct TestContex
      char *string;
      int flags;
      AVRational rational;
 +    int w, h;
 +    enum PixelFormat pix_fmt;
  } TestContext;
  
  #define OFFSET(x) offsetof(TestContext, x)
  static const AVOption test_options[]= {
  {"num",      "set num",        OFFSET(num),      AV_OPT_TYPE_INT,      {.i64 = 0},       0,        100                 },
  {"toggle",   "set toggle",     OFFSET(toggle),   AV_OPT_TYPE_INT,      {.i64 = 0},       0,        1                   },
- {"rational", "set rational",   OFFSET(rational), AV_OPT_TYPE_RATIONAL, {0},              0,        10                  },
+ {"rational", "set rational",   OFFSET(rational), AV_OPT_TYPE_RATIONAL, {.dbl = 0},  0,        10                  },
  {"string",   "set string",     OFFSET(string),   AV_OPT_TYPE_STRING,   {0},              CHAR_MIN, CHAR_MAX            },
  {"flags",    "set flags",      OFFSET(flags),    AV_OPT_TYPE_FLAGS,    {.i64 = 0},       0,        INT_MAX, 0, "flags" },
  {"cool",     "set cool flag ", 0,                AV_OPT_TYPE_CONST,    {.i64 = TEST_FLAG_COOL}, INT_MIN,  INT_MAX, 0, "flags" },
  {"lame",     "set lame flag ", 0,                AV_OPT_TYPE_CONST,    {.i64 = TEST_FLAG_LAME}, INT_MIN,  INT_MAX, 0, "flags" },
  {"mu",       "set mu flag ",   0,                AV_OPT_TYPE_CONST,    {.i64 = TEST_FLAG_MU},   INT_MIN,  INT_MAX, 0, "flags" },
 +{"size",     "set size",       OFFSET(w),        AV_OPT_TYPE_IMAGE_SIZE,{0},             0,        0                   },
 +{"pix_fmt",  "set pixfmt",     OFFSET(pix_fmt),  AV_OPT_TYPE_PIXEL_FMT,{0},              0,        0                   },
  {NULL},
  };
  
@@@ -934,7 -879,7 +934,7 @@@ int main(void
  
      printf("\nTesting av_set_options_string()\n");
      {
 -        TestContext test_ctx;
 +        TestContext test_ctx = { 0 };
          const char *options[] = {
              "",
              ":",
              "num=42 : string=blahblah",
              "rational=0 : rational=1/2 : rational=1/-1",
              "rational=-1/0",
 +            "size=1024x768",
 +            "size=pal",
 +            "size=bogus",
 +            "pix_fmt=yuv420p",
 +            "pix_fmt=2",
 +            "pix_fmt=bogus",
          };
  
          test_ctx.class = &test_class;
                  av_log(&test_ctx, AV_LOG_ERROR, "Error setting options string: '%s'\n", options[i]);
              printf("\n");
          }
 +        av_freep(&test_ctx.string);
      }
  
      return 0;