[Mesa-dev] [PATCH 1/2] Eliminate several cases of multiplication in arguments to calloc

Carl Worth cworth at cworth.org
Wed Sep 3 18:43:00 PDT 2014


Matt Turner <mattst88 at gmail.com> writes:
>> -   configs = calloc(1, (num_modes + 1) * sizeof *configs);
>> +   configs = calloc((num_modes + 1), sizeof *configs);
>
> I'd drop the extra parentheses.

Thanks for reading carefully, Matt!

> With that, both are
>
> Reviewed-by: Matt Turner <mattst88 at gmail.com>

OK. These are both pushed now.

-Carl

-- 
carl.d.worth at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140903/ea5e22aa/attachment.sig>


More information about the mesa-dev mailing list