[FFmpeg-devel] [PATCH] lavu: Add av_dict_from_string()

[Alexander Strasser]

Andrey Utkin wrote:
> 2012/2/16 Alexander Strasser <eclipse7 at gmx.net>:
> >  Would it seem OK for you to use an array of character pointers or similar?
> > I guess your current interface would not allow passing through arbitrary
> > character strings.
> >
> >  Another advantage would be that a more generic interface would serve a wider
> > audience of applications in a better way (though some would need a little more
> > code, but maybe we could also provide that in a separate API if you think it
> > would be useful to have in lavu).
> 
> Thanks for pointing out the limitation of my procedure. Yep, option
> value can't contain comma and '='.
> But, if caller have an array of string pointers, then IMO he is at a
> single step from having it split by option name and value, so passing
> it to some librarian helper function seems having little sense.
> The only idea i have on extending capability of this function is
> support of escaping comma and = with backslash.

  Yes, it lowers the value of the utility function. I think I misread part of your
code/intention earlier, please pardon my oversight.

  Maybe it would be fine if we have backslash escaping supported. It needs a bit
of thought work and writing down the exact specs of what the function supports.

  Also could you explain to me how you use it in your application. That would help
me to reason about it.

  Alexander