[FFmpeg-devel] [PATCH] lavu/opt: add av_opt_ accessors for pixel/format/image size options

Stefano Sabatini stefasab at gmail.com
Sun Nov 11 15:14:33 CET 2012


On date Wednesday 2012-11-07 23:58:07 +0100, Stefano Sabatini encoded:
> On date Tuesday 2012-11-06 13:31:53 +0100, Michael Niedermayer encoded:
> > On Tue, Nov 06, 2012 at 01:09:38AM +0100, Stefano Sabatini wrote:
[...] 
> > Iam not sure a public function pair for each type added is a good
> > idea.
> 
> This is basically following what is already done in the defied
> interface.
>
> > But if we go this route, at least the code should check the types,
> > i would not want to have a flags or pixel format read as
> > AVSampleFormat without warning
> 
> Fixed.

Possible advantages of this approach: safer setting of options, more
sensical feedback. Cons: bloat.

Ping (see also attached patch to see a possible application).
-- 
FFmpeg = Fundamental Foolish Merciful Plastic Exploitable Game
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-lavu-opt-add-av_opt_-accessors-for-pixel-format-imag.patch
Type: text/x-diff
Size: 6727 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121111/d62c433b/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-lswr-swresample-use-AV_OPT_TYPE_SAMPLE_FMT-for-sampl.patch
Type: text/x-diff
Size: 4515 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121111/d62c433b/attachment-0001.bin>


More information about the ffmpeg-devel mailing list