[Mesa-stable] [Mesa-dev] [PATCH] i965: Ignore 'centroid' interpolation qualifier in case of persample shading

Anuj Phogat anuj.phogat at gmail.com
Fri Jan 31 14:03:19 PST 2014


On Fri, Jan 31, 2014 at 12:55 PM, Carl Worth <cworth at cworth.org> wrote:
> Anuj Phogat <anuj.phogat at gmail.com> writes:
>> I missed this change in commit f5cfb4a. It fixes the incorrect
>> rendering caused in Dolphin Emulator.
>
> It's too late for my review to change anything, but since I was recently
> adding this commit (and the previous one being touched-up here) to my
> ignore list, I noticed something that I think worth mentioning.
>
> Your original commit had a good commit message:
>
> i965: Ignore 'centroid' interpolation qualifier in case of persample shading
>
> But then after you noticed a mistake, the subsequent commit has the same
> commit message. This confused me at first when I saw two different
> commits in the history with the same message.
>
> In this particular case, the commit message does still seem to be
> descriptive of the change. So I can't really say that it's an
> inappropriate commit message.
>
> But it still might have helped to at least have the message be distinct.
>
> Maybe something like:
>
> i965: Another case to ignore 'centroid' qualifier with persample shading
>
I agree. It is little confusing. I'll take care in future.
I've squashed the two patches for mesa-stable.
http://lists.freedesktop.org/archives/mesa-stable/2014-January/000694.html
> But like I said, it's too late to change anything, and it's not really a
> big deal.
>
> -Carl
>
> --
> carl.d.worth at intel.com


More information about the mesa-stable mailing list