[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