On 04/23/2013 09:15 PM, Jordan Justen wrote:
On Tue, Apr 23, 2013 at 5:41 PM, Kenneth Graunke <kenn...@whitecape.org> wrote:
On 04/19/2013 10:54 AM, Eric Anholt wrote:

Removes 75/78 state-dependent recompiles in GLB2.7 (the remaining 3 are
due to FBO-rendering size predictions).


AFAICT ClampColor is not deprecated in OpenGL Core.  It doesn't exist in ES,
though.

If you change it to _mesa_is_desktop_gl(ctx), you can have a R-b.

glspec31.20090528, E.1 Profiles and Deprecated Features of OpenGL 3.0:

"Note that the FrontFace and ClampColor commands are not deprecated,
as they still affect other non-deprecated functionality; however, the Clam-
pColor targets CLAMP_VERTEX_COLOR and CLAMP_FRAGMENT_COLOR are
deprecated."

Does that mean Eric's original change is okay?

-Jordan

You're correct, of course (it's been a long day). ClampColor still exists but only to toggle clamping on readback, not vertex/fragment color clamping.

So, sorry Eric, your original patch gets a:
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Thanks Jordan.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to