On Fri, Jul 08, 2011 at 08:28:19 (CEST), Anton Khirnov wrote:
> ---
> libavutil/dict.h | 39 ++++++++++++++++++++++++++++++++++++---
> 1 files changed, 36 insertions(+), 3 deletions(-)
>
> diff --git a/libavutil/dict.h b/libavutil/dict.h
> index 397ce38..7adac68 100644
> --- a/libavutil/dict.h
> +++ b/libavutil/dict.h
> @@ -25,10 +25,42 @@
> #ifndef AVUTIL_DICT_H
> #define AVUTIL_DICT_H
>
> +/**
> + * @defgroup dict_api Public dictionary API
For consistency with "Public Metadata API" in the module index, capitalize
'Dictionary'.
> + * @{
> + * Dictionaries are used for storing key:value pairs. To create
> + * an AVDictionary, simply pass an address of a NULL pointer to
> + * av_dict_set(). NULL can be used as an empty dictionary wherever
> + * a pointer to an AVDictionary is required.
> + * Use av_dict_get() to retrieve an entry or iterate over all
> + * entries and finally av_dict_free() to free the dictionary
> + * and all its contents.
> + *
> + * @code
> + * AVDictionary *d = NULL; // "create" an empty dictionary
> + * av_dict_set(&d, "foo", "bar", 0); // add an entry
> + *
> + * char *k = av_strdup("key"); // if your strings are already
> allocated,
> + * char *v = av_strdup("value"); // you can avoid copying them like
> this
> + * av_dict_set(&d, k, v, AV_DICT_DONT_STRDUP_KEY | AV_DICT_DONT_STRDUP_VAL);
> + *
> + * AVDictionaryEntry *t = NULL;
> + * while (t = av_dict_get(d, "", t, AV_DICT_IGNORE_SUFFIX)) {
> + * <....> // iterate over all entries in d
> + * }
> + *
> + * av_dict_free(&d);
> + * @endcode
> + *
> + * @}
> + */
> +
We should have much more of such code examples. Thanks for writing them!
> #define AV_DICT_MATCH_CASE 1
> #define AV_DICT_IGNORE_SUFFIX 2
> -#define AV_DICT_DONT_STRDUP_KEY 4
> -#define AV_DICT_DONT_STRDUP_VAL 8
> +#define AV_DICT_DONT_STRDUP_KEY 4 /**< Take ownership of a key that's been
> + allocated with av_malloc() and
> children. */
> +#define AV_DICT_DONT_STRDUP_VAL 8 /**< Take ownership of a value that's
> been
> + allocated with av_malloc() and
> chilren. */
> #define AV_DICT_DONT_OVERWRITE 16 ///< Don't overwrite existing entries.
> #define AV_DICT_APPEND 32 /**< If the entry already exists, append
> to it. Note that no
> delimiter is added, the strings are
> simply concatenated. */
> @@ -74,7 +106,8 @@ int av_dict_set(AVDictionary **pm, const char *key, const
> char *value, int flags
> void av_dict_copy(AVDictionary **dst, AVDictionary *src, int flags);
>
> /**
> - * Free all the memory allocated for an AVDictionary struct.
> + * Free all the memory allocated for an AVDictionary struct
> + * and all keys and values.
> */
> void av_dict_free(AVDictionary **m);
Great!
(patch OK)
--
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel