Re: [Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-08-01 Thread Juha-Pekka Heikkila
On 01.08.2014 06:39, Jason Ekstrand wrote: This adds the API entrypoint, error checking logic, and a driver hook for the ARB_copy_image extension. v2: Fix a typo in ARB_copy_image.xml and add it to the makefile v3: Put ARB_copy_image.xml in the right place alphebetically in the makefile

Re: [Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-08-01 Thread Neil Roberts
There is one line that is 116 characters long (!) but apart from that: Reviewed-by: Neil Roberts n...@linux.intel.com Regards, - Neil ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-08-01 Thread Jason Ekstrand
On Fri, Aug 1, 2014 at 2:56 AM, Juha-Pekka Heikkila juhapekka.heikk...@gmail.com wrote: On 01.08.2014 06:39, Jason Ekstrand wrote: This adds the API entrypoint, error checking logic, and a driver hook for the ARB_copy_image extension. v2: Fix a typo in ARB_copy_image.xml and add it to

[Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-07-31 Thread Jason Ekstrand
This adds the API entrypoint, error checking logic, and a driver hook for the ARB_copy_image extension. Signed-off-by: Jason Ekstrand jason.ekstr...@intel.com --- src/mapi/glapi/gen/ARB_copy_image.xml | 28 +++ src/mapi/glapi/gen/gl_API.xml | 2 +- src/mapi/glapi/gen/gl_genexec.py

Re: [Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-07-31 Thread Ilia Mirkin
On Thu, Jul 31, 2014 at 2:27 PM, Jason Ekstrand ja...@jlekstrand.net wrote: This adds the API entrypoint, error checking logic, and a driver hook for the ARB_copy_image extension. Signed-off-by: Jason Ekstrand jason.ekstr...@intel.com --- src/mapi/glapi/gen/ARB_copy_image.xml | 28 +++

Re: [Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-07-31 Thread Jason Ekstrand
On Thu, Jul 31, 2014 at 11:38 AM, Ilia Mirkin imir...@alum.mit.edu wrote: On Thu, Jul 31, 2014 at 2:27 PM, Jason Ekstrand ja...@jlekstrand.net wrote: This adds the API entrypoint, error checking logic, and a driver hook for the ARB_copy_image extension. Signed-off-by: Jason Ekstrand

[Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-07-31 Thread Jason Ekstrand
This adds the API entrypoint, error checking logic, and a driver hook for the ARB_copy_image extension. v2: Fix a typo in ARB_copy_image.xml and add it to the makefile Signed-off-by: Jason Ekstrand jason.ekstr...@intel.com --- src/mapi/glapi/gen/ARB_copy_image.xml | 28 +++

Re: [Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-07-31 Thread Matt Turner
On Thu, Jul 31, 2014 at 4:47 PM, Jason Ekstrand ja...@jlekstrand.net wrote: diff --git a/src/mapi/glapi/gen/Makefile.am b/src/mapi/glapi/gen/Makefile.am index 212731f..f0bf34a 100644 --- a/src/mapi/glapi/gen/Makefile.am +++ b/src/mapi/glapi/gen/Makefile.am @@ -114,6 +114,7 @@ API_XML = \

Re: [Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-07-31 Thread Marek Olšák
Hi Jason, The prefix of the commit message should be mesa: , because it modifies mesa/main. Marek On Fri, Aug 1, 2014 at 1:47 AM, Jason Ekstrand ja...@jlekstrand.net wrote: This adds the API entrypoint, error checking logic, and a driver hook for the ARB_copy_image extension. v2: Fix a typo

[Mesa-dev] [PATCH 1/5] Add GL API support for ARB_copy_image

2014-07-31 Thread Jason Ekstrand
This adds the API entrypoint, error checking logic, and a driver hook for the ARB_copy_image extension. v2: Fix a typo in ARB_copy_image.xml and add it to the makefile v3: Put ARB_copy_image.xml in the right place alphebetically in the makefile and update the commit message Signed-off-by: Jason