Hi, On Sun, Jun 12, 2011 at 6:04 AM, Anton Khirnov <[email protected]> wrote: > On Sat, 11 Jun 2011 22:01:33 -0400, "Ronald S. Bultje" <[email protected]> > wrote: >> On Fri, Jun 10, 2011 at 4:13 PM, Anton Khirnov <[email protected]> wrote: >> [..] >> >> Quoting from earlier: >> >> On Thu, Jun 9, 2011 at 9:05 PM, Ronald S. Bultje <[email protected]> wrote: >> > On Thu, Jun 9, 2011 at 10:27 AM, Anton Khirnov <[email protected]> wrote: >> >> + } else if (oldval && flags & AV_DICT_APPEND) { >> >> + int len = strlen(oldval) + strlen(value) + 1; >> >> + if (!(oldval = av_realloc(oldval, len))) >> >> + return AVERROR(ENOMEM); >> >> + av_strlcat(oldval, value, len); >> >> + m->elems[m->count].value = oldval; >> > >> > What's the exact use case here? Having comma's or so, or a generally >> > settable delimiter, may be a good idea. >> > > See the following patch (cmdutils). > It's used to emulate the current behavior, where you can > -flags +foo <..some other options..> -flags +bar > to set both foo and bar.
av_dict_*() is used by metadata also, and I can see wanting people to abuse this to set up a list of concatenated artists, or so. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
