[FFmpeg-devel] [PATCH 3/3] random_seed: Replace a VLA with a heap alloc
derek.buitenhuis at gmail.com
Fri Sep 7 19:42:44 CEST 2012
On 07/09/2012 1:35 PM, Michael Niedermayer wrote:
> i would suggest
> uint8_t tmp;
> av_assert0(sizeof(tmp) >= av_sha_size)
> this would be simpler and completely avoid variable allocation
Wouldn't this be duplicating av_sha_size? I'm not against it, but
in general, I am wary of duplicated numbers like this.
BTW, why is av_sha_size not a constant/define anyway?
More information about the ffmpeg-devel