[Ffmpeg-devel] [PATCH] add all and none options for partitions config

Michael Niedermayer michaelni
Tue Mar 27 15:56:07 CEST 2007


Hi

On Tue, Mar 27, 2007 at 09:28:38PM +0800, Limin Wang wrote:
> Hi,
> 
> 
> > > Good idea, however it can't work as expected after tried.  On the other hand,
> > > even if it works, it's hacker mode only since we can't get any help
> > > information by "-h".
> > 
> > hmm, then what about a "all" ~0 / "none" 0 but with NULL unit so its not
> > limited to partitions?
> Sorry, What's the NULL unit? replace 0 with NULL? I haven't catch your point.

your patch added AVOptions with unit="partitions" so its specific to 
partitions, which is bad

either add it with unit=NULL and if needed fix the AVOption code or

or better fix av_set_string() so it parses all&none like max/min/default

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

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070327/5ee204cf/attachment.pgp>



More information about the ffmpeg-devel mailing list