[Mesa-dev] [PATCH] mesa: Fix return type of _mesa_get_format_bytes() (#37351)
Adam Jackson
ajax at redhat.com
Mon May 23 08:00:19 PDT 2011
On 5/20/11 8:50 PM, Jose Fonseca wrote:
>> I don't care which approach we go with. This will likely fix it
>> everywhere, but I'd like there to be a comment explaining why the
>> return
>> type must not be unsigned. Otherwise someone will come along in the
>> future and make it unsigned in the name of "code cleanup."
>
> I agree with Ian: Adam's patch is good, but a comment would be useful, as the implicit coercion to unsigned is both unfortunate and unintuitive.
Makes sense. Added a comment above _mesa_get_format_bytes and pushed,
thanks.
- ajax
More information about the mesa-dev
mailing list