Merge commit '564b4591bbe223bdc5f36a1131eaef103f23f5d0'
authorDerek Buitenhuis <derek.buitenhuis@gmail.com>
Tue, 19 Apr 2016 18:17:50 +0000 (19:17 +0100)
committerDerek Buitenhuis <derek.buitenhuis@gmail.com>
Tue, 19 Apr 2016 18:19:53 +0000 (19:19 +0100)
* commit '564b4591bbe223bdc5f36a1131eaef103f23f5d0':
  opt: Add av_opt_copy()

Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
1  2 
libavutil/opt.c
libavutil/version.h

diff --cc libavutil/opt.c
@@@ -1579,15 -766,10 +1579,15 @@@ static int opt_size(enum AVOptionType t
      case AV_OPT_TYPE_DOUBLE:    return sizeof(double);
      case AV_OPT_TYPE_FLOAT:     return sizeof(float);
      case AV_OPT_TYPE_STRING:    return sizeof(uint8_t*);
 +    case AV_OPT_TYPE_VIDEO_RATE:
      case AV_OPT_TYPE_RATIONAL:  return sizeof(AVRational);
      case AV_OPT_TYPE_BINARY:    return sizeof(uint8_t*) + sizeof(int);
 +    case AV_OPT_TYPE_IMAGE_SIZE:return sizeof(int[2]);
 +    case AV_OPT_TYPE_PIXEL_FMT: return sizeof(enum AVPixelFormat);
 +    case AV_OPT_TYPE_SAMPLE_FMT:return sizeof(enum AVSampleFormat);
 +    case AV_OPT_TYPE_COLOR:     return 4;
      }
-     return 0;
+     return AVERROR(EINVAL);
  }
  
  int av_opt_copy(void *dst, const void *src)
              *(int*)(field_dst8 + 1) = len;
          } else if (o->type == AV_OPT_TYPE_CONST) {
              // do nothing
 +        } else if (o->type == AV_OPT_TYPE_DICT) {
 +            AVDictionary **sdict = (AVDictionary **) field_src;
 +            AVDictionary **ddict = (AVDictionary **) field_dst;
 +            if (*sdict != *ddict)
 +                av_dict_free(ddict);
 +            *ddict = NULL;
 +            av_dict_copy(ddict, *sdict, 0);
 +            if (av_dict_count(*sdict) != av_dict_count(*ddict))
 +                ret = AVERROR(ENOMEM);
          } else {
-             memcpy(field_dst, field_src, opt_size(o->type));
+             int size = opt_size(o->type);
+             if (size < 0)
+                 ret = size;
+             else
+                 memcpy(field_dst, field_src, size);
          }
      }
      return ret;
@@@ -63,9 -53,9 +63,9 @@@
   * @{
   */
  
 -#define LIBAVUTIL_VERSION_MAJOR 55
 -#define LIBAVUTIL_VERSION_MINOR 10
 -#define LIBAVUTIL_VERSION_MICRO  0
 +#define LIBAVUTIL_VERSION_MAJOR  55
 +#define LIBAVUTIL_VERSION_MINOR  22
- #define LIBAVUTIL_VERSION_MICRO 100
++#define LIBAVUTIL_VERSION_MICRO 101
  
  #define LIBAVUTIL_VERSION_INT   AV_VERSION_INT(LIBAVUTIL_VERSION_MAJOR, \
                                                 LIBAVUTIL_VERSION_MINOR, \