On Saturday, February 15, 2014, Anton Khirnov <[email protected]> wrote:

> ---
>  doc/APIchanges      |    4 ++++
>  libavutil/opt.c     |   14 +++++++++++---
>  libavutil/opt.h     |    9 +++++++++
>  libavutil/version.h |    4 ++--
>  4 files changed, 26 insertions(+), 5 deletions(-)
>
> diff --git a/doc/APIchanges b/doc/APIchanges
> index ba7ebd1..3803aed 100644
> --- a/doc/APIchanges
> +++ b/doc/APIchanges
> @@ -13,6 +13,10 @@ libavutil:     2013-12-xx
>
>  API changes, most recent first:
>
> +2014-xx-xx - xxxxxxx - lavu 53.03.0  - opt.h
> +  Add AV_OPT_FLAG_EXPORT and AV_OPT_FLAG_READONLY to mark options meant
> (only)
> +  for reading.


 The patch looks ok, but I don't understand why you are splitting the
flags. After all caller has access to all avoptions, so in a way they are
all exported already. Or am I missing something?
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to