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

Stefano Sabatini stefano.sabatini-lala
Fri May 1 20:02:40 CEST 2009


On date Saturday 2009-04-25 23:11:40 +0200, Stefano Sabatini encoded:
> Hi, as in subject.
> 
> The result of the test is:
> 
> [test @ 0xbfaf32a0]Setting options string ''
> 
> [test @ 0xbfaf32a0]Setting options string 'foo='
> [test @ 0xbfaf32a0]Setting value '' for key 'foo'
> [test @ 0xbfaf32a0]Unknown option key 'foo'
> [test @ 0xbfaf32a0]Error setting options string: 'foo='
> 
> [test @ 0xbfaf32a0]Setting options string 'foo'
> [test @ 0xbfaf32a0]No key or key/value separator found after key
> [test @ 0xbfaf32a0]Error setting options string: 'foo'
> 
> [test @ 0xbfaf32a0]Setting options string 'foo=val'
> [test @ 0xbfaf32a0]Setting value 'val' for key 'foo'
> [test @ 0xbfaf32a0]Unknown option key 'foo'
> [test @ 0xbfaf32a0]Error setting options string: 'foo=val'
> 
> [test @ 0xbfaf32a0]Setting options string 'toggle=1:foo'
> [test @ 0xbfaf32a0]Setting value '1' for key 'toggle'
> [test @ 0xbfaf32a0]No key or key/value separator found after key
> [test @ 0xbfaf32a0]Error setting options string: 'toggle=1:foo'
> 
> [test @ 0xbfaf32a0]Setting options string 'toggle=100'
> [test @ 0xbfaf32a0]Setting value '100' for key 'toggle'
> Value 100.000000 for parameter 'toggle' out of range
> [test @ 0xbfaf32a0]Invalid value '100' for key 'toggle'
> [test @ 0xbfaf32a0]Error setting options string: 'toggle=100'
> 
> [test @ 0xbfaf32a0]Setting options string 'flags=+mu-lame:num=42:toggle=0'
> [test @ 0xbfaf32a0]Setting value '+mu-lame' for key 'flags'
> [test @ 0xbfaf32a0]Setting value '42' for key 'num'
> [test @ 0xbfaf32a0]Setting value '0' for key 'toggle'
> 
> [test @ 0xbfaf32a0]Setting options string 'num=42:string=blahblah'
> [test @ 0xbfaf32a0]Setting value '42' for key 'num'
> [test @ 0xbfaf32a0]Setting value 'blahblah' for key 'string'
> 
> [test @ 0xbfaf32a0]Setting options string 'num=42:string=blahblah'
> [test @ 0xbfaf32a0]Setting value '42' for key 'num'
> [test @ 0xbfaf32a0]Setting value 'blahblah' for key 'string'

Patch updated, AV_DEFINE_CLASS patch included for convenience.

If you prefer we could drectly add the av_set_options_string() in
opt.[hc], and the test as a separate commit in opt.c.

Regards.
-- 
FFmpeg = Fantastic and Fast Multimedia Pitiless Extensive Guide
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lavfi-implement-define-class.patch
Type: text/x-diff
Size: 924 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090501/3ab1dd9b/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: parseutils-implement-set-options.patch
Type: text/x-diff
Size: 6035 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090501/3ab1dd9b/attachment-0001.patch>



More information about the ffmpeg-devel mailing list