[FFmpeg-devel] [PATCH] cmdutils: sort codec lists.

Michael Niedermayer michaelni at gmx.at
Sun Sep 23 19:38:48 CEST 2012


On Sun, Sep 23, 2012 at 11:05:32AM +0200, Nicolas George wrote:
> The lists are sorted first by type (video first)
> and then alphabetically by name.
> 
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
>  cmdutils.c |   44 ++++++++++++++++++++++++++++++++++++++++----
>  1 file changed, 40 insertions(+), 4 deletions(-)
> 
> 
> New version, with the lists for -encoders and -decoders sorted as well.
> 
> I do not think I can use Michael's compare function because AFAIK there is
> no guarantee that strcmp's return value is between -255 and +255; I can
> imagine big-endian implementations comparing two or four chars at once and
> returning a larger value.

(this or variants of it) LGTM


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

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- 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/20120923/7b31316c/attachment.asc>


More information about the ffmpeg-devel mailing list