[FFmpeg-devel] [PATCH] doc/developer: remove warning regarding long long

Ganesh Ajjanagadde gajjanag at mit.edu
Sun Nov 29 16:21:57 CET 2015


On Sun, Nov 29, 2015 at 10:14 AM, Nicolas George <george at nsup.org> wrote:
> Le nonidi 9 frimaire, an CCXXIV, Ganesh Ajjanagadde a écrit :
>> long long is already being used in quite a few places in FFmpeg. There
>> is no reason to list it as something to avoid.
>
> IMHO, most of these places should be replaced by more adapted types. With
> modern C, having long, long long or short in the code is usually sign that
> something is wrong because their size is not guaranteed at all and,
> especially long, varies a lot.

Fine, don't really mind since for the foreseeable future, all long
long instances may be replaced by int64_t. Just did not see a reason
to single it out as being so important in the surrounding context.
Patch dropped.

>
> Regards,
>
> --
>   Nicolas George
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list