[FFmpeg-devel] [PATCH] ffmpeg: raise level for message printed in case of auto-select pixel format

Nicolas George nicolas.george at normalesup.org
Thu Aug 1 12:30:52 CEST 2013


Le quartidi 14 thermidor, an CCXXI, Michael Niedermayer a écrit :
> Or stated differently the user told you to encode to h264 and that
> pretty much implies that he wants h264 that works for an UNSPECIFIED
> target or ANY target.
> like compiling C without extra switches generates a program that
> works on ANY reasonable variant of the host platform

This argument is interesting, but I believe it does not apply: the practical
difference between code produced by a C compiler with generic optimizations
or with arch-dependant optimizations is negligible for most users. The
difference between RGB -> 444 and RGB -> 420 is often noticeable.

I remind you once again: when RGB -> 420 was the default, we had a lot of
complaints about blurry edges.

Also, what I most strongly object is making an EXCEPTION for x264, because
exceptions are annoying BOTH for users and for developers.

If you want to make "baseline" profile the default for x264 (and make sure
it enforces the pixel format), that is fine by me, because it fits a general
design.

I still think that a warning "you will have either compatibility problems or
blurry edges, please choose one" is better, because it educates users about
something that can absolutely not be guessed by the program.

> like "hey i want to convert my ogg to mp3, target device huh i want to
> share with friends no idea what they or their friends will use it on

Why do you assume that instead of "I want to share with friends, they all
have recent phones but very little space on the memory card"? I assure you
that teenagers know exactly what phone all their friends have.

If you assume that compatibility is what users want, then of course
compatibility is what the default should be. But that is not an argument,
that is begging the question.

Regards,

-- 
  Nicolas George
-------------- 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/20130801/74cda257/attachment.asc>


More information about the ffmpeg-devel mailing list