[FFmpeg-devel] [PATCH] Implement av_set_options_string()

Michael Niedermayer michaelni
Fri May 8 03:11:07 CEST 2009


On Thu, May 07, 2009 at 09:22:01AM +0200, Stefano Sabatini wrote:
> On date Thursday 2009-05-07 02:02:58 +0200, Michael Niedermayer encoded:
[...]
> +int av_set_options_string(void *ctx, const char *opts,
> +                          const char *key_val_sep, const char *pairs_sep)
> +{
> +    int count = 0;
> +
> +    while (*opts) {

> +        if (parse_key_value_pair(ctx, &opts, key_val_sep, pairs_sep) < 0)
> +            return -1;

passing the error code through seems like a good idea ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090508/ecae824b/attachment.pgp>



More information about the ffmpeg-devel mailing list