Fix Windows newlines.
[mesa.git] / src / mesa / shader / shaderobjects.c
index ac8ff1af487ddcdc5d47cd65f4ac9e649184c348..f26ff12adf12c7536845b8e4c09ba8fca7619eb3 100644 (file)
@@ -1,8 +1,8 @@
 /*
  * Mesa 3-D graphics library
- * Version:  6.3
+ * Version:  6.5
  *
- * Copyright (C) 2004-2005  Brian Paul   All Rights Reserved.
+ * Copyright (C) 2004-2006  Brian Paul   All Rights Reserved.
  *
  * Permission is hereby granted, free of charge, to any person obtaining a
  * copy of this software and associated documentation files (the "Software"),
 
 
 #include "glheader.h"
-#include "shaderobjects.h"
-#include "shaderobjects_3dlabs.h"
 #include "context.h"
-#include "macros.h"
 #include "hash.h"
+#include "shaderobjects.h"
+#include "shaderobjects_3dlabs.h"
 
 
-void GLAPIENTRY
-_mesa_DeleteObjectARB (GLhandleARB obj)
-{
-       GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_generic_intf **gen;
+#if FEATURE_ARB_shader_objects
 
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, obj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
+#define RELEASE_GENERIC(x)\
+   (**x)._unknown.Release ((struct gl2_unknown_intf **) (x))
 
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glDeleteObjectARB");
-               return;
-       }
+#define RELEASE_CONTAINER(x)\
+   (**x)._generic._unknown.Release ((struct gl2_unknown_intf **) (x))
 
-       gen = (struct gl2_generic_intf **) (**unk).QueryInterface (unk, UIID_GENERIC);
-       if (gen == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glDeleteObjectARB");
-               return;
-       }
+#define RELEASE_PROGRAM(x)\
+   (**x)._container._generic._unknown.Release ((struct gl2_unknown_intf **) (x))
 
-       (**gen).Delete (gen);
-       (**gen)._unknown.Release ((struct gl2_unknown_intf **) gen);
+#define RELEASE_SHADER(x)\
+   (**x)._generic._unknown.Release ((struct gl2_unknown_intf **) (x))
+
+static struct gl2_unknown_intf **
+lookup_handle (GLcontext *ctx, GLhandleARB handle, enum gl2_uiid uiid, const char *function)
+{
+   struct gl2_unknown_intf **unk;
+
+   /*
+    * Note: _mesa_HashLookup() requires non-zero input values, so the passed-in handle value
+    *       must be checked beforehand.
+    */
+   if (handle == 0) {
+      _mesa_error (ctx, GL_INVALID_VALUE, function);
+      return NULL;
+   }
+   _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
+   unk = (struct gl2_unknown_intf **) (_mesa_HashLookup (ctx->Shared->GL2Objects, handle));
+   _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
+   if (unk == NULL)
+      _mesa_error (ctx, GL_INVALID_VALUE, function);
+   else {
+      unk = (**unk).QueryInterface (unk, uiid);
+      if (unk == NULL)
+         _mesa_error (ctx, GL_INVALID_OPERATION, function);
+   }
+   return unk;
+}
+
+#define GET_GENERIC(x, handle, function)\
+   struct gl2_generic_intf **x = (struct gl2_generic_intf **)\
+                                 lookup_handle (ctx, handle, UIID_GENERIC, function);
+
+#define GET_CONTAINER(x, handle, function)\
+   struct gl2_container_intf **x = (struct gl2_container_intf **)\
+                                   lookup_handle (ctx, handle, UIID_CONTAINER, function);
+
+#define GET_PROGRAM(x, handle, function)\
+   struct gl2_program_intf **x = (struct gl2_program_intf **)\
+                                 lookup_handle (ctx, handle, UIID_PROGRAM, function);
+
+#define GET_SHADER(x, handle, function)\
+   struct gl2_shader_intf **x = (struct gl2_shader_intf **)\
+                                lookup_handle (ctx, handle, UIID_SHADER, function);
+
+#define GET_LINKED_PROGRAM(x, handle, function)\
+   GET_PROGRAM(x, handle, function);\
+   if (x != NULL && (**x).GetLinkStatus (x) == GL_FALSE) {\
+      RELEASE_PROGRAM(x);\
+      x = NULL;\
+      _mesa_error (ctx, GL_INVALID_OPERATION, function);\
+   }
+
+#define GET_CURRENT_LINKED_PROGRAM(x, function)\
+   struct gl2_program_intf **x = NULL;\
+   if (ctx->ShaderObjects.CurrentProgram == NULL)\
+      _mesa_error (ctx, GL_INVALID_OPERATION, function);\
+   else {\
+      x = ctx->ShaderObjects.CurrentProgram;\
+      if (x != NULL && (**x).GetLinkStatus (x) == GL_FALSE) {\
+         x = NULL;\
+         _mesa_error (ctx, GL_INVALID_OPERATION, function);\
+      }\
+   }
+
+#define IS_NAME_WITH_GL_PREFIX(x) ((x)[0] == 'g' && (x)[1] == 'l' && (x)[2] == '_')
+
+
+GLvoid GLAPIENTRY
+_mesa_DeleteObjectARB (GLhandleARB obj)
+{
+   if (obj != 0)
+   {
+      GET_CURRENT_CONTEXT(ctx);
+      GET_GENERIC(gen, obj, "glDeleteObjectARB");
+
+      if (gen != NULL)
+      {
+         (**gen).Delete (gen);
+         RELEASE_GENERIC(gen);
+      }
+   }
 }
 
 GLhandleARB GLAPIENTRY
@@ -72,59 +138,38 @@ _mesa_GetHandleARB (GLenum pname)
 
        switch (pname)
        {
-       case GL_PROGRAM_OBJECT_ARB:
-               if (ctx->ShaderObjects.current_program != NULL)
-                       return (**ctx->ShaderObjects.current_program)._container._generic.GetName (
-                               (struct gl2_generic_intf **) ctx->ShaderObjects.current_program);
-               break;
+   case GL_PROGRAM_OBJECT_ARB:
+      {
+         struct gl2_program_intf **pro = ctx->ShaderObjects.CurrentProgram;
+
+         if (pro != NULL)
+            return (**pro)._container._generic.GetName ((struct gl2_generic_intf **) (pro));
+      }
+      break;
+       default:
+               _mesa_error (ctx, GL_INVALID_ENUM, "glGetHandleARB");
        }
 
        return 0;
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_DetachObjectARB (GLhandleARB containerObj, GLhandleARB attachedObj)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unkc, **unka;
-       struct gl2_container_intf **con;
-       struct gl2_generic_intf **att;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unkc = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, containerObj);
-       unka = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, attachedObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unkc == NULL || unka == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glDetachObjectARB");
-               return;
-       }
+       GET_CONTAINER(con, containerObj, "glDetachObjectARB");
 
-       con = (struct gl2_container_intf **) (**unkc).QueryInterface (unkc, UIID_CONTAINER);
-       if (con == NULL)
+       if (con != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glDetachObjectARB");
-               return;
-       }
-
-       att = (struct gl2_generic_intf **) (**unka).QueryInterface (unka, UIID_GENERIC);
-       if (att == NULL)
-       {
-               (**con)._generic._unknown.Release ((struct gl2_unknown_intf **) con);
-               _mesa_error (ctx, GL_INVALID_VALUE, "glDetachObjectARB");
-               return;
-       }
+               GET_GENERIC(att, attachedObj, "glDetachObjectARB");
 
-       if ((**con).Detach (con, att) == GL_FALSE)
-       {
-               (**con)._generic._unknown.Release ((struct gl2_unknown_intf **) con);
-               (**att)._unknown.Release ((struct gl2_unknown_intf **) att);
-               return;
+               if (att != NULL)
+               {
+                       (**con).Detach (con, att);
+                       RELEASE_GENERIC(att);
+               }
+               RELEASE_CONTAINER(con);
        }
-
-       (**con)._generic._unknown.Release ((struct gl2_unknown_intf **) con);
-       (**att)._unknown.Release ((struct gl2_unknown_intf **) att);
 }
 
 GLhandleARB GLAPIENTRY
@@ -133,46 +178,47 @@ _mesa_CreateShaderObjectARB (GLenum shaderType)
        return _mesa_3dlabs_create_shader_object (shaderType);
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_ShaderSourceARB (GLhandleARB shaderObj, GLsizei count, const GLcharARB **string,
                                           const GLint *length)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_shader_intf **sha;
        GLint *offsets;
        GLsizei i;
        GLcharARB *source;
+       GET_SHADER(sha, shaderObj, "glShaderSourceARB");
 
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, shaderObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glShaderSourceARB");
+       if (sha == NULL)
                return;
-       }
 
-       sha = (struct gl2_shader_intf **) (**unk).QueryInterface (unk, UIID_SHADER);
-       if (sha == NULL)
+       if (string == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glShaderSourceARB");
+               RELEASE_SHADER(sha);
+               _mesa_error (ctx, GL_INVALID_VALUE, "glShaderSourceARB");
                return;
        }
 
-       /* this array holds offsets of where the appropriate string ends, thus the last
-       element will be set to the total length of the source code */
+       /*
+        * This array holds offsets of where the appropriate string ends, thus the last
+        * element will be set to the total length of the source code.
+        */
        offsets = (GLint *) _mesa_malloc (count * sizeof (GLint));
        if (offsets == NULL)
        {
-               (**sha)._generic._unknown.Release ((struct gl2_unknown_intf **) sha);
+               RELEASE_SHADER(sha);
                _mesa_error (ctx, GL_OUT_OF_MEMORY, "glShaderSourceARB");
                return;
        }
 
        for (i = 0; i < count; i++)
        {
+               if (string[i] == NULL)
+               {
+                       _mesa_free ((GLvoid *) offsets);
+                       RELEASE_SHADER(sha);
+                       _mesa_error (ctx, GL_INVALID_VALUE, "glShaderSourceARB");
+                       return;
+               }
                if (length == NULL || length[i] < 0)
                        offsets[i] = _mesa_strlen (string[i]);
                else
@@ -185,8 +231,8 @@ _mesa_ShaderSourceARB (GLhandleARB shaderObj, GLsizei count, const GLcharARB **s
        source = (GLcharARB *) _mesa_malloc ((offsets[count - 1] + 1) * sizeof (GLcharARB));
        if (source == NULL)
        {
-               _mesa_free ((void *) offsets);
-               (**sha)._generic._unknown.Release ((struct gl2_unknown_intf **) sha);
+               _mesa_free ((GLvoid *) offsets);
+               RELEASE_SHADER(sha);
                _mesa_error (ctx, GL_OUT_OF_MEMORY, "glShaderSourceARB");
                return;
        }
@@ -199,446 +245,575 @@ _mesa_ShaderSourceARB (GLhandleARB shaderObj, GLsizei count, const GLcharARB **s
        source[offsets[count - 1]] = '\0';
 
        (**sha).SetSource (sha, source, offsets, count);
-       (**sha)._generic._unknown.Release ((struct gl2_unknown_intf **) sha);
+       RELEASE_SHADER(sha);
 }
 
-void  GLAPIENTRY
+GLvoid  GLAPIENTRY
 _mesa_CompileShaderARB (GLhandleARB shaderObj)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_shader_intf **sha;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, shaderObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glCompileShaderARB");
-               return;
-       }
+       GET_SHADER(sha, shaderObj, "glCompileShaderARB");
 
-       sha = (struct gl2_shader_intf **) (**unk).QueryInterface (unk, UIID_SHADER);
-       if (sha == NULL)
+       if (sha != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glCompileShaderARB");
-               return;
+               (**sha).Compile (sha);
+               RELEASE_SHADER(sha);
        }
-
-       (**sha).Compile (sha);
-       (**sha)._generic._unknown.Release ((struct gl2_unknown_intf **) sha);
 }
 
 GLhandleARB GLAPIENTRY
-_mesa_CreateProgramObjectARB (void)
+_mesa_CreateProgramObjectARB (GLvoid)
 {
        return _mesa_3dlabs_create_program_object ();
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_AttachObjectARB (GLhandleARB containerObj, GLhandleARB obj)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unkc, **unka;
-       struct gl2_container_intf **con;
-       struct gl2_generic_intf **att;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unkc = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, containerObj);
-       unka = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, obj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unkc == NULL || unka == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glAttachObjectARB");
-               return;
-       }
+       GET_CONTAINER(con, containerObj, "glAttachObjectARB");
 
-       con = (struct gl2_container_intf **) (**unkc).QueryInterface (unkc, UIID_CONTAINER);
-       if (con == NULL)
+       if (con != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glAttachObjectARB");
-               return;
-       }
+               GET_GENERIC(att, obj, "glAttachObjectARB");
 
-       att = (struct gl2_generic_intf **) (**unka).QueryInterface (unka, UIID_GENERIC);
-       if (att == NULL)
-       {
-               (**con)._generic._unknown.Release ((struct gl2_unknown_intf **) con);
-               _mesa_error (ctx, GL_INVALID_VALUE, "glAttachObjectARB");
-               return;
-       }
-
-       if (!(**con).Attach (con, att))
-       {
-               (**con)._generic._unknown.Release ((struct gl2_unknown_intf **) con);
-               (**att)._unknown.Release ((struct gl2_unknown_intf **) att);
-               return;
+               if (att != NULL)
+               {
+                       (**con).Attach (con, att);
+                       RELEASE_GENERIC(att);
+               }
+               RELEASE_CONTAINER(con);
        }
-
-       (**con)._generic._unknown.Release ((struct gl2_unknown_intf **) con);
-       (**att)._unknown.Release ((struct gl2_unknown_intf **) att);
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_LinkProgramARB (GLhandleARB programObj)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_program_intf **pro;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glLinkProgramARB");
-               return;
-       }
+       GET_PROGRAM(pro, programObj, "glLinkProgramARB");
 
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
-       if (pro == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glLinkProgramARB");
-               return;
-       }
-
-       if (pro == ctx->ShaderObjects.current_program)
+       if (pro != NULL)
        {
-               /* TODO re-install executable program */
+               (**pro).Link (pro);
+               if (pro == ctx->ShaderObjects.CurrentProgram)
+               {
+                       if ((**pro).GetLinkStatus (pro))
+                               _mesa_UseProgramObjectARB (programObj);
+                       else
+                               _mesa_UseProgramObjectARB (0);
+               }
+               RELEASE_PROGRAM(pro);
        }
-
-       (**pro).Link (pro);
-       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_UseProgramObjectARB (GLhandleARB programObj)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_program_intf **pro;
+   struct gl2_program_intf **program = NULL;
 
-       if (programObj == 0)
-       {
-               pro = NULL;
-       }
-       else
-       {
-               struct gl2_unknown_intf **unk;
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
 
-               _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-               unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-               _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-               if (unk == NULL)
-               {
-                       _mesa_error (ctx, GL_INVALID_VALUE, "glUseProgramObjectARB");
-                       return;
-               }
+       if (programObj != 0)
+       {
+               GET_PROGRAM(pro, programObj, "glUseProgramObjectARB");
 
-               pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
                if (pro == NULL)
-               {
-                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUseProgramObjectARB");
                        return;
-               }
 
                if ((**pro).GetLinkStatus (pro) == GL_FALSE)
                {
-                       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
+                       RELEASE_PROGRAM(pro);
                        _mesa_error (ctx, GL_INVALID_OPERATION, "glUseProgramObjectARB");
                        return;
                }
-       }
 
-       if (ctx->ShaderObjects.current_program != NULL)
+               program = pro;
+
+               ctx->ShaderObjects._VertexShaderPresent = (**pro).IsShaderPresent (pro, GL_VERTEX_SHADER_ARB);
+               ctx->ShaderObjects._FragmentShaderPresent = (**pro).IsShaderPresent (pro,
+                       GL_FRAGMENT_SHADER_ARB);
+       }
+       else
        {
-               (**ctx->ShaderObjects.current_program)._container._generic._unknown.Release (
-                       (struct gl2_unknown_intf **) ctx->ShaderObjects.current_program);
+               ctx->ShaderObjects._VertexShaderPresent = GL_FALSE;
+               ctx->ShaderObjects._FragmentShaderPresent = GL_FALSE;
        }
 
-       ctx->ShaderObjects.current_program = pro;
+       if (ctx->ShaderObjects.CurrentProgram != NULL)
+               RELEASE_PROGRAM(ctx->ShaderObjects.CurrentProgram);
+       ctx->ShaderObjects.CurrentProgram = program;
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_ValidateProgramARB (GLhandleARB programObj)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_program_intf **pro;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
+       GET_PROGRAM(pro, programObj, "glValidateProgramARB");
 
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glValidateProgramARB");
-               return;
-       }
-
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
-       if (pro == NULL)
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glValidateProgramARB");
-               return;
+               (**pro).Validate (pro);
+               RELEASE_PROGRAM(pro);
        }
-
-       (**pro).Validate (pro);
-       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
 }
 
-/*
-Errors TODO
-
-    The error INVALID_OPERATION is generated by the Uniform*ARB if the
-    number of values loaded results in exceeding the declared extent of a
-    uniform.
-
-    The error INVALID_OPERATION is generated by the Uniform*ARB commands if
-    the size does not match the size of the uniform declared in the shader.
-
-    The error INVALID_OPERATION is generated by the Uniform*ARB commands if
-    the type does not match the type of the uniform declared in the shader,
-    if the uniform is not of type Boolean.
-
-    The error INVALID_OPERATION is generated by the Uniform*ARB commands if
-    <location> does not exist for the program object currently in use.
-
-    The error INVALID_OPERATION is generated if a uniform command other than
-    Uniform1i{v}ARB is used to load a sampler value.
-
-
-*/
-
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform1fARB (GLint location, GLfloat v0)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform1fARB");
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform1fARB");
-               return;
+               if (!(**pro).WriteUniform (pro, location, 1, &v0, GL_FLOAT))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform1fARB");
        }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform2fARB (GLint location, GLfloat v0, GLfloat v1)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform2fARB");
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform2fARB");
-               return;
+               GLfloat v[2];
+               v[0] = v0;
+               v[1] = v1;
+
+               if (!(**pro).WriteUniform (pro, location, 1, v, GL_FLOAT_VEC2))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform2fARB");
        }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform3fARB (GLint location, GLfloat v0, GLfloat v1, GLfloat v2)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform3fARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform3fARB");
-               return;
+               GLfloat v[3];
+               v[0] = v0;
+               v[1] = v1;
+               v[2] = v2;
+
+               if (!(**pro).WriteUniform (pro, location, 1, v, GL_FLOAT_VEC3))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform3fARB");
        }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform4fARB (GLint location, GLfloat v0, GLfloat v1, GLfloat v2, GLfloat v3)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform4fARB");
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform4fARB");
-               return;
+               GLfloat v[4];
+               v[0] = v0;
+               v[1] = v1;
+               v[2] = v2;
+               v[3] = v3;
+
+               if (!(**pro).WriteUniform (pro, location, 1, v, GL_FLOAT_VEC4))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform4fARB");
        }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform1iARB (GLint location, GLint v0)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform1iARB");
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform1iARB");
-               return;
+               if (!(**pro).WriteUniform (pro, location, 1, &v0, GL_INT))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform1iARB");
        }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform2iARB (GLint location, GLint v0, GLint v1)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform2iARB");
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform2iARB");
-               return;
+               GLint v[2];
+               v[0] = v0;
+               v[1] = v1;
+
+               if (!(**pro).WriteUniform (pro, location, 1, v, GL_INT_VEC2))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform2iARB");
        }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform3iARB (GLint location, GLint v0, GLint v1, GLint v2)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform3iARB");
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform3iARB");
-               return;
+               GLint v[3];
+               v[0] = v0;
+               v[1] = v1;
+               v[2] = v2;
+
+               if (!(**pro).WriteUniform (pro, location, 1, v, GL_INT_VEC3))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform3iARB");
        }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform4iARB (GLint location, GLint v0, GLint v1, GLint v2, GLint v3)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform4iARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform4iARB");
-               return;
+               GLint v[4];
+               v[0] = v0;
+               v[1] = v1;
+               v[2] = v2;
+               v[3] = v3;
+
+               if (!(**pro).WriteUniform (pro, location, 1, v, GL_INT_VEC4))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform4iARB");
        }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform1fvARB (GLint location, GLsizei count, const GLfloat *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform1fvARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform1fvARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniform1fvARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (!(**pro).WriteUniform (pro, location, count, value, GL_FLOAT))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform1fvARB");
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform2fvARB (GLint location, GLsizei count, const GLfloat *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform2fvARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform2fvARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniform2fvARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (!(**pro).WriteUniform (pro, location, count, value, GL_FLOAT_VEC2))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform2fvARB");
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform3fvARB (GLint location, GLsizei count, const GLfloat *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform3fvARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform3fvARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniform3fvARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (!(**pro).WriteUniform (pro, location, count, value, GL_FLOAT_VEC3))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform3fvARB");
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform4fvARB (GLint location, GLsizei count, const GLfloat *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform4fvARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform4fvARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniform4fvARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (!(**pro).WriteUniform (pro, location, count, value, GL_FLOAT_VEC4))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform4fvARB");
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform1ivARB (GLint location, GLsizei count, const GLint *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform1ivARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform1ivARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniform1ivARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (!(**pro).WriteUniform (pro, location, count, value, GL_INT))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform1ivARB");
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform2ivARB (GLint location, GLsizei count, const GLint *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform2ivARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform2ivARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniform2ivARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (!(**pro).WriteUniform (pro, location, count, value, GL_INT_VEC2))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform2ivARB");
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform3ivARB (GLint location, GLsizei count, const GLint *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform3ivARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform3ivARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniform3ivARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (!(**pro).WriteUniform (pro, location, count, value, GL_INT_VEC3))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform3ivARB");
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_Uniform4ivARB (GLint location, GLsizei count, const GLint *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniform4ivARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform4ivARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniform4ivARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (!(**pro).WriteUniform (pro, location, count, value, GL_INT_VEC4))
+                       _mesa_error (ctx, GL_INVALID_OPERATION, "glUniform4ivARB");
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_UniformMatrix2fvARB (GLint location, GLsizei count, GLboolean transpose, const GLfloat *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniformMatrix2fvARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniformMatrix2fvARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniformMatrix2fvARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (transpose)
+               {
+                       GLfloat *trans, *pt;
+                       const GLfloat *pv;
+
+                       trans = (GLfloat *) _mesa_malloc (count * 4 * sizeof (GLfloat));
+                       if (trans == NULL)
+                       {
+                               _mesa_error (ctx, GL_OUT_OF_MEMORY, "glUniformMatrix2fvARB");
+                               return;
+                       }
+                       for (pt = trans, pv = value; pt != trans + count * 4; pt += 4, pv += 4)
+                       {
+                               pt[0] = pv[0];
+                               pt[1] = pv[2];
+                               pt[2] = pv[1];
+                               pt[3] = pv[3];
+                       }
+                       if (!(**pro).WriteUniform (pro, location, count, trans, GL_FLOAT_MAT2))
+                               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniformMatrix2fvARB");
+                       _mesa_free (trans);
+               }
+               else
+               {
+                       if (!(**pro).WriteUniform (pro, location, count, value, GL_FLOAT_MAT2))
+                               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniformMatrix2fvARB");
+               }
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_UniformMatrix3fvARB (GLint location, GLsizei count, GLboolean transpose, const GLfloat *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniformMatrix3fvARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniformMatrix3fvARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniformMatrix3fvARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (transpose)
+               {
+                       GLfloat *trans, *pt;
+                       const GLfloat *pv;
+
+                       trans = (GLfloat *) _mesa_malloc (count * 9 * sizeof (GLfloat));
+                       if (trans == NULL)
+                       {
+                               _mesa_error (ctx, GL_OUT_OF_MEMORY, "glUniformMatrix3fvARB");
+                               return;
+                       }
+                       for (pt = trans, pv = value; pt != trans + count * 9; pt += 9, pv += 9)
+                       {
+                               pt[0] = pv[0];
+                               pt[1] = pv[3];
+                               pt[2] = pv[6];
+                               pt[3] = pv[1];
+                               pt[4] = pv[4];
+                               pt[5] = pv[7];
+                               pt[6] = pv[2];
+                               pt[7] = pv[5];
+                               pt[8] = pv[8];
+                       }
+                       if (!(**pro).WriteUniform (pro, location, count, trans, GL_FLOAT_MAT3))
+                               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniformMatrix3fvARB");
+                       _mesa_free (trans);
+               }
+               else
+               {
+                       if (!(**pro).WriteUniform (pro, location, count, value, GL_FLOAT_MAT3))
+                               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniformMatrix3fvARB");
+               }
+       }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_UniformMatrix4fvARB (GLint location, GLsizei count, GLboolean transpose, const GLfloat *value)
 {
        GET_CURRENT_CONTEXT(ctx);
+       GET_CURRENT_LINKED_PROGRAM(pro, "glUniformMatrix4fvARB");
 
-       if (ctx->ShaderObjects.current_program == NULL)
+       if (value == NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniformMatrix4fvARB");
+               _mesa_error (ctx, GL_INVALID_VALUE, "glUniformMatrix4fvARB");
                return;
        }
+
+       FLUSH_VERTICES(ctx, _NEW_PROGRAM);
+
+       if (pro != NULL)
+       {
+               if (transpose)
+               {
+                       GLfloat *trans, *pt;
+                       const GLfloat *pv;
+
+                       trans = (GLfloat *) _mesa_malloc (count * 16 * sizeof (GLfloat));
+                       if (trans == NULL)
+                       {
+                               _mesa_error (ctx, GL_OUT_OF_MEMORY, "glUniformMatrix4fvARB");
+                               return;
+                       }
+                       for (pt = trans, pv = value; pt != trans + count * 16; pt += 16, pv += 16)
+                       {
+                               _math_transposef (pt, pv);
+                       }
+                       if (!(**pro).WriteUniform (pro, location, count, trans, GL_FLOAT_MAT4))
+                               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniformMatrix4fvARB");
+                       _mesa_free (trans);
+               }
+               else
+               {
+                       if (!(**pro).WriteUniform (pro, location, count, value, GL_FLOAT_MAT4))
+                               _mesa_error (ctx, GL_INVALID_OPERATION, "glUniformMatrix4fvARB");
+               }
+       }
 }
 
 static GLboolean
@@ -646,536 +821,393 @@ _mesa_get_object_parameter (GLhandleARB obj, GLenum pname, GLvoid *params, GLboo
        GLint *size)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_generic_intf **gen;
-       struct gl2_shader_intf **sha;
-       struct gl2_program_intf **pro;
        GLint *ipar = (GLint *) params;
-       /*GLfloat *fpar = (GLfloat *) params;*/
-       GLboolean success = GL_TRUE;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, obj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetObjectParameterivARB");
-               return GL_FALSE;
-       }
-
-       gen = (struct gl2_generic_intf **) (**unk).QueryInterface (unk, UIID_GENERIC);
-       if (gen == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetObjectParameterivARB");
-               return GL_FALSE;
-       }
-
-       sha = (struct gl2_shader_intf **) (**unk).QueryInterface (unk, UIID_SHADER);
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
 
        /* set default values */
        *integral = GL_TRUE;    /* indicates param type, TRUE: GLint, FALSE: GLfloat */
-       *size = 1;                              /* param array size */ 
+       *size = 1;                              /* param array size */
 
        switch (pname)
        {
        case GL_OBJECT_TYPE_ARB:
-               *ipar = (**gen).GetType (gen);
-               break;
-       case GL_OBJECT_SUBTYPE_ARB:
-               if (sha != NULL)
-                       *ipar = (**sha).GetSubType (sha);
-               else {
-                       _mesa_error (ctx, GL_INVALID_OPERATION, "glGetObjectParameterivARB");
-                       success = GL_FALSE;
-               }
-               break;
        case GL_OBJECT_DELETE_STATUS_ARB:
-               *ipar = (**gen).GetDeleteStatus (gen);
+       case GL_OBJECT_INFO_LOG_LENGTH_ARB:
+               {
+                       GET_GENERIC(gen, obj, "glGetObjectParameterivARB");
+
+                       if (gen == NULL)
+                               return GL_FALSE;
+
+                       switch (pname)
+                       {
+                       case GL_OBJECT_TYPE_ARB:
+                               *ipar = (**gen).GetType (gen);
+                               break;
+                       case GL_OBJECT_DELETE_STATUS_ARB:
+                               *ipar = (**gen).GetDeleteStatus (gen);
+                               break;
+         case GL_OBJECT_INFO_LOG_LENGTH_ARB:
+            *ipar = (**gen).GetInfoLogLength (gen);
+            break;
+                       }
+
+                       RELEASE_GENERIC(gen);
+               }
                break;
+       case GL_OBJECT_SUBTYPE_ARB:
        case GL_OBJECT_COMPILE_STATUS_ARB:
-               if (sha != NULL)
-                       *ipar = (**sha).GetCompileStatus (sha);
-               else {
-                       _mesa_error (ctx, GL_INVALID_OPERATION, "glGetObjectParameterivARB");
-                       success = GL_FALSE;
+       case GL_OBJECT_SHADER_SOURCE_LENGTH_ARB:
+               {
+                       GET_SHADER(sha, obj, "glGetObjectParameterivARB");
+
+                       if (sha == NULL)
+                               return GL_FALSE;
+
+                       switch (pname)
+                       {
+                       case GL_OBJECT_SUBTYPE_ARB:
+                               *ipar = (**sha).GetSubType (sha);
+                               break;
+                       case GL_OBJECT_COMPILE_STATUS_ARB:
+                               *ipar = (**sha).GetCompileStatus (sha);
+                               break;
+                       case GL_OBJECT_SHADER_SOURCE_LENGTH_ARB:
+                               {
+                                       const GLcharARB *src = (**sha).GetSource (sha);
+
+                                       if (src == NULL)
+                                               *ipar = 0;
+                                       else
+                                               *ipar = _mesa_strlen (src) + 1;
+                               }
+                               break;
+                       }
+
+                       RELEASE_SHADER(sha);
                }
                break;
        case GL_OBJECT_LINK_STATUS_ARB:
-               if (pro != NULL)
-                       *ipar = (**pro).GetLinkStatus (pro);
-               else {
-                       _mesa_error (ctx, GL_INVALID_OPERATION, "glGetObjectParameterivARB");
-                       success = GL_FALSE;
-               }
-               break;
        case GL_OBJECT_VALIDATE_STATUS_ARB:
-               if (pro != NULL)
-                       *ipar = (**pro).GetValidateStatus (pro);
-               else {
-                       _mesa_error (ctx, GL_INVALID_OPERATION, "glGetObjectParameterivARB");
-                       success = GL_FALSE;
-               }
-               break;
-       case GL_OBJECT_INFO_LOG_LENGTH_ARB:
-               {
-                       const GLcharARB *info = (**gen).GetInfoLog (gen);
-                       if (info == NULL)
-                               *ipar = 0;
-                       else
-                               *ipar = _mesa_strlen (info) + 1;
-               }
-               break;
        case GL_OBJECT_ATTACHED_OBJECTS_ARB:
-               if (pro != NULL)
-                       *ipar = (**pro)._container.GetAttachedCount ((struct gl2_container_intf **) pro);
-               else {
-                       _mesa_error (ctx, GL_INVALID_OPERATION, "glGetObjectParameterivARB");
-                       success = GL_FALSE;
-               }
-               break;
        case GL_OBJECT_ACTIVE_UNIFORMS_ARB:
-               if (pro != NULL)
-                       *ipar = 0;      /* TODO */
-               else {
-                       _mesa_error (ctx, GL_INVALID_OPERATION, "glGetObjectParameterivARB");
-                       success = GL_FALSE;
-               }
-               break;
        case GL_OBJECT_ACTIVE_UNIFORM_MAX_LENGTH_ARB:
-               if (pro != NULL)
-                       *ipar = 0;      /* TODO */
-               else {
-                       _mesa_error (ctx, GL_INVALID_OPERATION, "glGetObjectParameterivARB");
-                       success = GL_FALSE;
-               }
-               break;
-       case GL_OBJECT_SHADER_SOURCE_LENGTH_ARB:
-               if (sha != NULL) {
-                       const GLcharARB *src = (**sha).GetSource (sha);
-                       if (src == NULL)
-                               *ipar = 0;
-                       else
-                               *ipar = _mesa_strlen (src) + 1;
-               } else {
-                       _mesa_error (ctx, GL_INVALID_OPERATION, "glGetObjectParameterivARB");
-                       success = GL_FALSE;
+               {
+                       GET_PROGRAM(pro, obj, "glGetObjectParameterivARB");
+
+                       if (pro == NULL)
+                               return GL_FALSE;
+
+                       switch (pname)
+                       {
+                       case GL_OBJECT_LINK_STATUS_ARB:
+                               *ipar = (**pro).GetLinkStatus (pro);
+                               break;
+                       case GL_OBJECT_VALIDATE_STATUS_ARB:
+                               *ipar = (**pro).GetValidateStatus (pro);
+                               break;
+         case GL_OBJECT_ATTACHED_OBJECTS_ARB:
+            *ipar = (**pro)._container.GetAttachedCount ((struct gl2_container_intf **) (pro));
+            break;
+                       case GL_OBJECT_ACTIVE_UNIFORMS_ARB:
+                               *ipar = (**pro).GetActiveUniformCount (pro);
+                               break;
+                       case GL_OBJECT_ACTIVE_UNIFORM_MAX_LENGTH_ARB:
+                               *ipar = (**pro).GetActiveUniformMaxLength (pro);
+                               break;
+                       case GL_OBJECT_ACTIVE_ATTRIBUTES_ARB:
+                               *ipar = (**pro).GetActiveAttribCount (pro);
+                               break;
+                       case GL_OBJECT_ACTIVE_ATTRIBUTE_MAX_LENGTH_ARB:
+                               *ipar = (**pro).GetActiveAttribMaxLength (pro);
+                               break;
+                       }
+
+                       RELEASE_PROGRAM(pro);
                }
                break;
        default:
                _mesa_error (ctx, GL_INVALID_ENUM, "glGetObjectParameterivARB");
-               success = GL_FALSE;
-               break;
+               return GL_FALSE;
        }
 
-       (**gen)._unknown.Release ((struct gl2_unknown_intf **) gen);
-       if (sha != NULL)
-               (**sha)._generic._unknown.Release ((struct gl2_unknown_intf **) sha);
-       if (pro != NULL)
-               (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
-
-       return success;
+       return GL_TRUE;
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_GetObjectParameterfvARB (GLhandleARB obj, GLenum pname, GLfloat *params)
 {
+       GET_CURRENT_CONTEXT(ctx);
        GLboolean integral;
-       GLint size, i;
+       GLint size;
+
+       if (params == NULL)
+       {
+               _mesa_error (ctx, GL_INVALID_VALUE, "glGetObjectParameterfvARB");
+               return;
+       }
 
        assert (sizeof (GLfloat) == sizeof (GLint));
 
-       if (_mesa_get_object_parameter (obj, pname, (GLvoid *) params, &integral, &size) != GL_FALSE)
-               if (integral != GL_FALSE)
+       if (_mesa_get_object_parameter (obj, pname, (GLvoid *) params, &integral, &size))
+               if (integral)
+               {
+                       GLint i;
+
                        for (i = 0; i < size; i++)
                                params[i] = (GLfloat) ((GLint *) params)[i];
+               }
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_GetObjectParameterivARB (GLhandleARB obj, GLenum pname, GLint *params)
 {
+       GET_CURRENT_CONTEXT(ctx);
        GLboolean integral;
-       GLint size, i;
+       GLint size;
+
+       if (params == NULL)
+       {
+               _mesa_error (ctx, GL_INVALID_VALUE, "glGetObjectParameterivARB");
+               return;
+       }
 
        assert (sizeof (GLfloat) == sizeof (GLint));
 
-       if (_mesa_get_object_parameter (obj, pname, (GLvoid *) params, &integral, &size) != GL_FALSE)
-               if (integral == GL_FALSE)
+       if (_mesa_get_object_parameter (obj, pname, (GLvoid *) params, &integral, &size))
+               if (!integral)
+               {
+                       GLint i;
+
                        for (i = 0; i < size; i++)
                                params[i] = (GLint) ((GLfloat *) params)[i];
+               }
 }
 
-static void
-_mesa_get_string (const GLcharARB *src, GLsizei maxLength, GLsizei *length, GLcharARB *str)
+
+/**
+ * Copy string from <src> to <dst>, up to maxLength characters, returning
+ * length of <dst> in <length>.
+ * \param src  the strings source
+ * \param maxLength  max chars to copy
+ * \param length  returns numberof chars copied
+ * \param dst  the string destination
+ */
+static GLvoid
+copy_string(const GLcharARB *src, GLsizei maxLength, GLsizei *length,
+            GLcharARB *dst)
 {
        GLsizei len;
-
-       if (src == NULL)
-               src = "";
-
-       len = _mesa_strlen (src);
-       if (len > maxLength)
-       {
-               len = maxLength;
-               /* allocate space for null termination */
-               if (len > 0)
-                       len--;
-       }
-
-       _mesa_memcpy (str, src, len * sizeof (GLcharARB));
+       for (len = 0; len < maxLength - 1 && src && src[len]; len++)
+               dst[len] = src[len];
        if (maxLength > 0)
-               str[len] = '\0';
-
-       if (length != NULL)
+               dst[len] = 0;
+       if (length)
                *length = len;
 }
 
-void GLAPIENTRY
+
+GLvoid GLAPIENTRY
 _mesa_GetInfoLogARB (GLhandleARB obj, GLsizei maxLength, GLsizei *length, GLcharARB *infoLog)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_generic_intf **gen;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, obj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetInfoLogARB");
-               return;
-       }
+       GET_GENERIC(gen, obj, "glGetInfoLogARB");
 
-       gen = (struct gl2_generic_intf **) (**unk).QueryInterface (unk, UIID_GENERIC);
        if (gen == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetInfoLogARB");
                return;
-       }
 
-       _mesa_get_string ((**gen).GetInfoLog (gen), maxLength, length, infoLog);
-
-       (**gen)._unknown.Release ((struct gl2_unknown_intf **) gen);
+       if (infoLog == NULL)
+               _mesa_error (ctx, GL_INVALID_VALUE, "glGetInfoLogARB");
+   else {
+      GLsizei actualsize = (**gen).GetInfoLogLength (gen);
+      if (actualsize > maxLength)
+         actualsize = maxLength;
+               (**gen).GetInfoLog (gen, actualsize, infoLog);
+      if (length != NULL)
+         *length = (actualsize > 0) ? actualsize - 1 : 0;
+   }
+       RELEASE_GENERIC(gen);
 }
 
-void GLAPIENTRY
-_mesa_GetAttachedObjectsARB (GLhandleARB containerObj, GLsizei maxCount, GLsizei *count, GLhandleARB *obj)
+GLvoid GLAPIENTRY
+_mesa_GetAttachedObjectsARB (GLhandleARB containerObj, GLsizei maxCount, GLsizei *count,
+                                                        GLhandleARB *obj)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_container_intf **con;
-       GLsizei cnt, i;
+       GET_CONTAINER(con, containerObj, "glGetAttachedObjectsARB");
 
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, containerObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetAttachedObjectsARB");
-               return;
-       }
-
-       con = (struct gl2_container_intf **) (**unk).QueryInterface (unk, UIID_CONTAINER);
        if (con == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetAttachedObjectsARB");
                return;
-       }
-
-       cnt = (**con).GetAttachedCount (con);
-       if (cnt > maxCount)
-               cnt = maxCount;
 
-       for (i = 0; i < cnt; i++)
+       if (obj == NULL)
+               _mesa_error (ctx, GL_INVALID_VALUE, "glGetAttachedObjectsARB");
+       else
        {
-               struct gl2_generic_intf **x = (**con).GetAttached (con, i);
-               obj[i] = (**x).GetName (x);
-               (**x)._unknown.Release ((struct gl2_unknown_intf **) x);
-       }
+               GLsizei cnt, i;
 
-       (**con)._generic._unknown.Release ((struct gl2_unknown_intf **) con);
+               cnt = (**con).GetAttachedCount (con);
+               if (cnt > maxCount)
+                       cnt = maxCount;
+               if (count != NULL)
+                       *count = cnt;
 
-       if (count != NULL)
-               *count = cnt;
+               for (i = 0; i < cnt; i++)
+               {
+         struct gl2_generic_intf **x = (**con).GetAttached (con, i);
+                       obj[i] = (**x).GetName (x);
+                       RELEASE_GENERIC(x);
+               }
+       }
+       RELEASE_CONTAINER(con);
 }
 
 GLint GLAPIENTRY
 _mesa_GetUniformLocationARB (GLhandleARB programObj, const GLcharARB *name)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_program_intf **pro;
        GLint loc = -1;
+       GET_LINKED_PROGRAM(pro, programObj, "glGetUniformLocationARB");
 
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetUniformLocationARB");
-               return -1;
-       }
-
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
        if (pro == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetUniformLocationARB");
                return -1;
-       }
 
-       if ((**pro).GetLinkStatus (pro) == GL_FALSE)
+       if (name == NULL)
+               _mesa_error (ctx, GL_INVALID_VALUE, "glGetUniformLocationARB");
+       else
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetUniformLocationARB");
-               (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
-               return -1;
+               if (!IS_NAME_WITH_GL_PREFIX(name))
+                       loc = (**pro).GetUniformLocation (pro, name);
        }
-
-       /* TODO */
-
-       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
+       RELEASE_PROGRAM(pro);
        return loc;
 }
 
-void GLAPIENTRY
-_mesa_GetActiveUniformARB (GLhandleARB programObj, GLuint index, GLsizei maxLength, GLsizei *length, GLint *size, GLenum *type, GLcharARB *name)
+GLvoid GLAPIENTRY
+_mesa_GetActiveUniformARB (GLhandleARB programObj, GLuint index, GLsizei maxLength, GLsizei *length,
+       GLint *size, GLenum *type, GLcharARB *name)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_program_intf **pro;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
+       GET_PROGRAM(pro, programObj, "glGetActiveUniformARB");
 
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetActiveUniformARB");
-               return;
-       }
-
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
        if (pro == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetActiveUniformARB");
                return;
-       }
 
-/*     if (index >= val (OBJECT_ACTIVE_ATTRIBUTES_ARB))
-       {
+       if (size == NULL || type == NULL || name == NULL)
                _mesa_error (ctx, GL_INVALID_VALUE, "glGetActiveUniformARB");
-               (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
-               return;
-       }*/
-
-       /* TODO */
-
-       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
+       else
+       {
+               if (index < (**pro).GetActiveUniformCount (pro))
+                       (**pro).GetActiveUniform (pro, index, maxLength, length, size, type, name);
+               else
+                       _mesa_error (ctx, GL_INVALID_VALUE, "glGetActiveUniformARB");
+       }
+       RELEASE_PROGRAM(pro);
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_GetUniformfvARB (GLhandleARB programObj, GLint location, GLfloat *params)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_program_intf **pro;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetUniformfvARB");
-               return;
-       }
-
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
-       if (pro == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetUniformfvARB");
-               return;
-       }
+       GET_LINKED_PROGRAM(pro, programObj, "glGetUniformfvARB");
 
-       if ((**pro).GetLinkStatus (pro) == GL_FALSE)
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetUniformfvARB");
-               (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
-               return;
+               /* TODO */
+               RELEASE_PROGRAM(pro);
        }
-
-       /* TODO validate location (OPERATION) */
-
-       /* TODO */
-
-       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_GetUniformivARB (GLhandleARB programObj, GLint location, GLint *params)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_program_intf **pro;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetUniformivARB");
-               return;
-       }
+       GET_LINKED_PROGRAM(pro, programObj, "glGetUniformivARB");
 
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
-       if (pro == NULL)
+       if (pro != NULL)
        {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetUniformivARB");
-               return;
+               /* TODO */
+               RELEASE_PROGRAM(pro);
        }
-
-       if ((**pro).GetLinkStatus (pro) == GL_FALSE)
-       {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetUniformivARB");
-               (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
-               return;
-       }
-
-       /* TODO validate location (GL_INVALID_OPERATION) */
-
-       /* TODO */
-
-       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
 }
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_GetShaderSourceARB (GLhandleARB obj, GLsizei maxLength, GLsizei *length, GLcharARB *source)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_shader_intf **sha;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, obj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetShaderSourceARB");
-               return;
-       }
+       GET_SHADER(sha, obj, "glGetShaderSourceARB");
 
-       sha = (struct gl2_shader_intf **) (**unk).QueryInterface (unk, UIID_SHADER);
        if (sha == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_OPERATION, "glGetShaderSourceARB");
                return;
-       }
-
-       _mesa_get_string ((**sha).GetSource (sha), maxLength, length, source);
 
-       (**sha)._generic._unknown.Release ((struct gl2_unknown_intf **) sha);
+       if (source == NULL)
+               _mesa_error (ctx, GL_INVALID_VALUE, "glGetShaderSourceARB");
+       else
+               copy_string ((**sha).GetSource (sha), maxLength, length, source);
+       RELEASE_SHADER(sha);
 }
 
 /* GL_ARB_vertex_shader */
 
-void GLAPIENTRY
+GLvoid GLAPIENTRY
 _mesa_BindAttribLocationARB (GLhandleARB programObj, GLuint index, const GLcharARB *name)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_program_intf **pro;
+       GET_PROGRAM(pro, programObj, "glBindAttribLocationARB");
 
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glBindAttribLocationARB");
-               return;
-       }
-
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
        if (pro == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glBindAttribLocationARB");
                return;
-       }
-
-       /* TODO */
 
-       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
+       if (name == NULL || index >= MAX_VERTEX_ATTRIBS)
+               _mesa_error (ctx, GL_INVALID_VALUE, "glBindAttribLocationARB");
+       else if (IS_NAME_WITH_GL_PREFIX(name))
+               _mesa_error (ctx, GL_INVALID_OPERATION, "glBindAttribLocationARB");
+       else
+               (**pro).OverrideAttribBinding (pro, index, name);
+       RELEASE_PROGRAM(pro);
 }
 
-void GLAPIENTRY
-_mesa_GetActiveAttribARB (GLhandleARB programObj, GLuint index, GLsizei maxLength, GLsizei *length, GLint *size, GLenum *type, GLcharARB *name)
+GLvoid GLAPIENTRY
+_mesa_GetActiveAttribARB (GLhandleARB programObj, GLuint index, GLsizei maxLength, GLsizei *length,
+                          GLint *size, GLenum *type, GLcharARB *name)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_program_intf **pro;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetActiveAttribARB");
-               return;
-       }
+       GET_PROGRAM(pro, programObj, "glGetActiveAttribARB");
 
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
        if (pro == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetActiveAttribARB");
                return;
-       }
 
-       /* TODO */
-
-       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
+       if (name == NULL || index >= (**pro).GetActiveAttribCount (pro))
+               _mesa_error (ctx, GL_INVALID_VALUE, "glGetActiveAttribARB");
+       else
+               (**pro).GetActiveAttrib (pro, index, maxLength, length, size, type, name);
+       RELEASE_PROGRAM(pro);
 }
 
 GLint GLAPIENTRY
 _mesa_GetAttribLocationARB (GLhandleARB programObj, const GLcharARB *name)
 {
        GET_CURRENT_CONTEXT(ctx);
-       struct gl2_unknown_intf **unk;
-       struct gl2_program_intf **pro;
-       GLint loc = 0;
-
-       _glthread_LOCK_MUTEX (ctx->Shared->Mutex);
-       unk = (struct gl2_unknown_intf **) _mesa_HashLookup (ctx->Shared->GL2Objects, programObj);
-       _glthread_UNLOCK_MUTEX (ctx->Shared->Mutex);
-
-       if (unk == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetAttribLocationARB");
-               return 0;
-       }
+       GLint loc = -1;
+       GET_LINKED_PROGRAM(pro, programObj, "glGetAttribLocationARB");
 
-       pro = (struct gl2_program_intf **) (**unk).QueryInterface (unk, UIID_PROGRAM);
        if (pro == NULL)
-       {
-               _mesa_error (ctx, GL_INVALID_VALUE, "glGetAttribLocationARB");
-               return 0;
-       }
-
-       /* TODO */
+               return -1;
 
-       (**pro)._container._generic._unknown.Release ((struct gl2_unknown_intf **) pro);
+       if (name == NULL)
+               _mesa_error (ctx, GL_INVALID_VALUE, "glGetAttribLocationARB");
+       else if (!IS_NAME_WITH_GL_PREFIX(name))
+               loc = (**pro).GetAttribLocation (pro, name);
+       RELEASE_PROGRAM(pro);
        return loc;
 }
 
-void
+#endif
+
+GLvoid
 _mesa_init_shaderobjects (GLcontext *ctx)
 {
-       ctx->ShaderObjects.current_program = NULL;
+       ctx->ShaderObjects.CurrentProgram = NULL;
+       ctx->ShaderObjects._FragmentShaderPresent = GL_FALSE;
+       ctx->ShaderObjects._VertexShaderPresent = GL_FALSE;
 
        _mesa_init_shaderobjects_3dlabs (ctx);
 }