Nan hai Zou wrote:

[... git check-in info ...]

Nan,

Since you've merged the i965 GLSL support into Mesa/master, can you 
summarize what state it's in?  Is it fully functional, or are there some 
unfinished bits?

In any case, it's great to have this!

I've found one problem, however.  In shader/program.c in 
_mesa_new_shader() you added a call to ctx->Driver.NewProgram():

struct gl_program *
_mesa_new_program(GLcontext *ctx, GLenum target, GLuint id)
{
    if (ctx->Driver.NewProgram)
         return ctx->Driver.NewProgram(ctx, target, id);
[...]


This causes infinite recursive calls with software drivers since 
ctx->Driver.NewProgram is initialized to point to _mesa_new_program().

I'm going to disable this code.  Let me know what problem you were 
trying to fix with this change.

-Brian

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Mesa3d-dev mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to