On 7/12/2016 7:52 PM, Moritz Barsnick wrote:
> On Tue, Jul 12, 2016 at 18:36:20 -0300, James Almer wrote:
>> +    uint32_t invcnt = !((granule >> 30) & 3);
> 
> If it's just for storing a 0/1 (bool, basically), wouldn't you use the
> more unspecific type "int"? (Not sure whether it matters at all.)
> 
> Moritz

Yeah, int should suffice. I used uint32_t for alignment purposes
with the other variables, but then i added the comment and that
became superfluous i guess.

Changed locally.

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to