On 01/12/2016 05:48 AM, Ganesh Ajjanagadde wrote:
You're free to make another patch, or if perhaps I should do it.
If something is inherently 32 bits (e.g obtained by reading 4 bytes),
then please don't make such a patch.
Seems to be the case here, and so I would nack such a patch:
color_start is obtained by an avio_rb32, keeping as uint32_t is
cleaner.
No we'll skip it in my book. I don't see reason enough to change it.
Thanks for your input, Ganesh.
Mats
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
