[FFmpeg-cvslog] af_pan: comment a tricky piece of code.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Wed Feb 15 20:23:41 CET 2012


On Wed, Feb 15, 2012 at 08:12:36PM +0100, Nicolas George wrote:
> +        /* reject layouts that are not a single channel */
>          if (channel_id >= MAX_CHANNELS || layout0 != (int64_t)1 << channel_id)

If we ever used all 64 bits, (int64_t)1 << channel_id would actually be
a signed overflow.
Also 1ull is generally accepted as a shorthand I think (even though
the more proper one is UINT64_C(1) ).


More information about the ffmpeg-cvslog mailing list