Re: [Mesa-dev] [PATCH 1/4] i965: Remove spurious casts in copy_image_with_memcpy()

2014-11-19 Thread Chad Versace
On Wed 19 Nov 2014, Matt Turner wrote: On Wed, Nov 19, 2014 at 3:35 AM, Kristian Høgsberg wrote: On Tue, Nov 18, 2014 at 9:02 PM, Chad Versace wrote: If a pointer points to raw, untyped memory and is never dereferenced, then declare it as 'void*' instead of casting it to 'void*'. Signed-off-

Re: [Mesa-dev] [PATCH 1/4] i965: Remove spurious casts in copy_image_with_memcpy()

2014-11-19 Thread Matt Turner
On Wed, Nov 19, 2014 at 3:35 AM, Kristian Høgsberg wrote: > On Tue, Nov 18, 2014 at 9:02 PM, Chad Versace > wrote: >> If a pointer points to raw, untyped memory and is never dereferenced, >> then declare it as 'void*' instead of casting it to 'void*'. >> >> Signed-off-by: Chad Versace >> --- >>

Re: [Mesa-dev] [PATCH 1/4] i965: Remove spurious casts in copy_image_with_memcpy()

2014-11-19 Thread Kristian Høgsberg
On Tue, Nov 18, 2014 at 9:02 PM, Chad Versace wrote: > If a pointer points to raw, untyped memory and is never dereferenced, > then declare it as 'void*' instead of casting it to 'void*'. > > Signed-off-by: Chad Versace > --- > src/mesa/drivers/dri/i965/intel_copy_image.c | 8 > 1 file

[Mesa-dev] [PATCH 1/4] i965: Remove spurious casts in copy_image_with_memcpy()

2014-11-19 Thread Chad Versace
If a pointer points to raw, untyped memory and is never dereferenced, then declare it as 'void*' instead of casting it to 'void*'. Signed-off-by: Chad Versace --- src/mesa/drivers/dri/i965/intel_copy_image.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mesa/dri

[Mesa-dev] [PATCH 1/4] i965: Remove spurious casts in copy_image_with_memcpy()

2014-11-19 Thread Chad Versace
If a pointer points to raw, untyped memory and is never dereferenced, then declare it as 'void*' instead of casting it to 'void*'. Signed-off-by: Chad Versace --- src/mesa/drivers/dri/i965/intel_copy_image.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mesa/dri