On Sat, 11 Jun 2011 22:01:33 -0400, "Ronald S. Bultje" <[email protected]> wrote: > Hi, > > 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. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
