[Mesa-dev] [PATCH 1/2] nir: add missing GLSL_TYPE_DOUBLE case in type_size()

[Brian Paul]

To silence compiler warning about unhandled switch case.
---
 src/glsl/nir/nir_lower_io.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/glsl/nir/nir_lower_io.c b/src/glsl/nir/nir_lower_io.c
index ddbc249..23499e5 100644
--- a/src/glsl/nir/nir_lower_io.c
+++ b/src/glsl/nir/nir_lower_io.c
@@ -49,6 +49,7 @@ type_size(const struct glsl_type *type)
case GLSL_TYPE_UINT:
case GLSL_TYPE_INT:
case GLSL_TYPE_FLOAT:
+   case GLSL_TYPE_DOUBLE:
case GLSL_TYPE_BOOL:
   return glsl_get_components(type);
case GLSL_TYPE_ARRAY:
-- 
1.9.1

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

[Mesa-dev] [PATCH 1/2] nir: add missing GLSL_TYPE_DOUBLE case in type_size()

[Brian Paul]

To silence compiler warning about unhandled switch case.
v2: move GLSL_TYPE_DOUBLE to the "not reached" section, per Ilia.
---
 src/glsl/nir/nir_lower_io.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/glsl/nir/nir_lower_io.c b/src/glsl/nir/nir_lower_io.c
index ddbc249..207f8da 100644
--- a/src/glsl/nir/nir_lower_io.c
+++ b/src/glsl/nir/nir_lower_io.c
@@ -69,6 +69,7 @@ type_size(const struct glsl_type *type)
   return 0;
case GLSL_TYPE_VOID:
case GLSL_TYPE_ERROR:
+   case GLSL_TYPE_DOUBLE:
   unreachable("not reached");
}
 
-- 
1.9.1

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

Re: [Mesa-dev] [PATCH 1/2] nir: add missing GLSL_TYPE_DOUBLE case in type_size()

[Ilia Mirkin]

Series is Reviewed-by: Ilia Mirkin 

On Thu, Feb 19, 2015 at 3:51 PM, Brian Paul  wrote:
> To silence compiler warning about unhandled switch case.
> v2: move GLSL_TYPE_DOUBLE to the "not reached" section, per Ilia.
> ---
>  src/glsl/nir/nir_lower_io.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/glsl/nir/nir_lower_io.c b/src/glsl/nir/nir_lower_io.c
> index ddbc249..207f8da 100644
> --- a/src/glsl/nir/nir_lower_io.c
> +++ b/src/glsl/nir/nir_lower_io.c
> @@ -69,6 +69,7 @@ type_size(const struct glsl_type *type)
>return 0;
> case GLSL_TYPE_VOID:
> case GLSL_TYPE_ERROR:
> +   case GLSL_TYPE_DOUBLE:
>unreachable("not reached");
> }
>
> --
> 1.9.1
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 1/2] nir: add missing GLSL_TYPE_DOUBLE case in type_size()

[Jason Ekstrand]

Reviewed-by: Jason Ekstrand 

On Thu, Feb 19, 2015 at 12:51 PM, Brian Paul  wrote:

> To silence compiler warning about unhandled switch case.
> v2: move GLSL_TYPE_DOUBLE to the "not reached" section, per Ilia.
> ---
>  src/glsl/nir/nir_lower_io.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/glsl/nir/nir_lower_io.c b/src/glsl/nir/nir_lower_io.c
> index ddbc249..207f8da 100644
> --- a/src/glsl/nir/nir_lower_io.c
> +++ b/src/glsl/nir/nir_lower_io.c
> @@ -69,6 +69,7 @@ type_size(const struct glsl_type *type)
>return 0;
> case GLSL_TYPE_VOID:
> case GLSL_TYPE_ERROR:
> +   case GLSL_TYPE_DOUBLE:
>unreachable("not reached");
> }
>
> --
> 1.9.1
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev