[FFmpeg-devel] [PATCH] lavu, lavc, lswr: do not hardcode AV_SAMPLE_FMT_NB value when setting sample format max value

Michael Niedermayer michaelni at gmx.at
Sun Dec 22 15:43:14 CET 2013


On Mon, Dec 16, 2013 at 04:50:55PM +0100, Stefano Sabatini wrote:
> The constant may change in libavutil but the library may be compiled
> against an older version, thus rejecting a value which is otherwise
> supported by the new libavutil.
> 
> INT_MAX is used here to denote the max allowed value for a sample format.
> 
> The opt-test code is changed to provide a valid reference example.

maybe ok

note though that there is no gurantee that new sample formats would
neccessarily be supported by the code that supports all current ones
(an actual example of this would probably be the addition of planar
 at a time when all code assumed packed)

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Into a blind darkness they enter who follow after the Ignorance,
they as if into a greater darkness enter who devote themselves
to the Knowledge alone. -- Isha Upanishad
-------------- 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/20131222/ec7e30fd/attachment.asc>


More information about the ffmpeg-devel mailing list