[Mesa-dev] [PATCH 4/4] mesa: Implement glGetFragDataLocation

Kenneth Graunke kenneth at whitecape.org
Wed Nov 9 20:39:17 PST 2011

On 11/09/2011 02:20 PM, Ian Romanick wrote:
> You mean, drop the 'var->location < FRAG_RESULT_DATA0' from the
> if-statement below? That is probably true. It was a copy-and-paste from
> GetAttributeLocation. The check *is* necessary there because there is no
> 'strncmp("gl_", name)' in that function. GetAttributeLocation can get
> away with that because there is no shared locations between built-in and
> user-defined. In the GetFragDataLocation, gl_FragData[] and user-defined
> variables overlap.
> How would you feel about a follow-on patch that makes the two functions
> consistent with each other by removing the check from both loops and
> adding the strcmp outside the loop in GetAttributeLocation?

I like that idea.  Avoiding built-ins by strncmp'ing with "gl_" is 
really self-explanatory.

More information about the mesa-dev mailing list