[FFmpeg-devel] [PATCH] cmdutils: apply option to codec and format contexts if possible.

[Michael Niedermayer]

On Thu, Oct 18, 2012 at 09:32:24AM +0000, Paul B Mahol wrote:
> On 10/18/12, Michael Niedermayer <michaelni at gmx.at> wrote:
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  cmdutils.c |   19 +++++++++++++------
> >  1 file changed, 13 insertions(+), 6 deletions(-)
> >
> 
> What if option is set twice, Is there way to give different
> value for different contexts? Yes that feature may not be
> much useful but whatever....

there is, you can prefix the option by a,v,s to route it to
audio/video/subtitle codec contexts.
without prefix it goes to all including format. (that allows to set
each individually in theory)

I can add a similar prefix to route only to format contexts
if theres some usecase for this ?
I just dont want to polute the namespace and complicate things before
there is a usecase.


> 
> There should be some kind of messages that option
> is going to be used for both contexts (under verbose or
> something like that).

added

patch applied

thanks

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121022/e3a34930/attachment.asc>