Hi Samuel, Iago,

On 05/08/15 09:30, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez <sigles...@igalia.com>
> 
> v2:
> - Add ShaderStorageBlockBinding to static_data.py
> 
Why (see comment below) ?


v4:
 - Ship ARB_shader_storage_buffer_object.xml in the tarball.

Seems like other patches are also missing v4 revision history :|


> --- a/src/mapi/glapi/gen/static_data.py
> +++ b/src/mapi/glapi/gen/static_data.py
> @@ -1305,6 +1305,7 @@ functions = [
>     "ShaderBinary",
>     "ShaderSource",
>     "ShaderSourceARB",
> +   "ShaderStorageBlockBinding",
This list is for functions exported by libGL.so, which imho should not
change (ever). If there is there a program that depends on this static
symbol (and the binary drivers expose it), it should be noted in the
commit message.

Thanks
Emil

P.S. Have mercy on us, don't resent the 50+ series again ;)

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

Reply via email to