[FFmpeg-cvslog] Merge commit '85770f2a2651497861ed938efcd0df3696ff5e45'

[Stefano Sabatini]

On date Sunday 2011-05-01 00:27:06 +0200, Michael Niedermayer wrote:
> ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Sun May  1 00:21:56 2011 +0200| [ffb5a0d533498102c31aa131bc91a4cce868b0a8] | committer: Michael Niedermayer
> 
> Merge commit '85770f2a2651497861ed938efcd0df3696ff5e45'
> 
> * commit '85770f2a2651497861ed938efcd0df3696ff5e45':
>   AVOptions: make default_val a union, as proposed in AVOption2.
>   Move ff_dynarray_add to lavu and make it public.
>   lavf: remove duplicate assignment in avformat_alloc_context.
>   lavf: use designated initializers for AVClasses.
>   options: simplify av_find_opt by using av_next_option.
> 
> Merged-by: Michael Niedermayer <michaelni at gmx.at>
> 
> > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=ffb5a0d533498102c31aa131bc91a4cce868b0a8
> ---

Why did you remove .i64 and .q from the default_val union?

For the rational it is more safe/logical not assume sizeof(double) >=
sizeof(int64_t), for the second it may be convenient to express a
default by using a rational rather than its double representation, for
documentation/convenience purposes.

Also I suppose we agreed that public API changes should be posted for
review before being pushed.