[Mesa-dev] [PATCH] Add test to ensure compiler won't allow #undef for some builtins

2014-06-11 Thread Carl Worth
Currently verifying that an #undef of __FILE__, __LINE__, or __VERSION__ will
generate an error.
---
 src/glsl/glcpp/tests/120-undef-builtin.c  | 3 +++
 src/glsl/glcpp/tests/120-undef-builtin.c.expected | 7 +++
 2 files changed, 10 insertions(+)
 create mode 100644 src/glsl/glcpp/tests/120-undef-builtin.c
 create mode 100644 src/glsl/glcpp/tests/120-undef-builtin.c.expected

diff --git a/src/glsl/glcpp/tests/120-undef-builtin.c 
b/src/glsl/glcpp/tests/120-undef-builtin.c
new file mode 100644
index 000..49e7696
--- /dev/null
+++ b/src/glsl/glcpp/tests/120-undef-builtin.c
@@ -0,0 +1,3 @@
+#undef __LINE__
+#undef __FILE__
+#undef __VERSION__
diff --git a/src/glsl/glcpp/tests/120-undef-builtin.c.expected 
b/src/glsl/glcpp/tests/120-undef-builtin.c.expected
new file mode 100644
index 000..d6981dc
--- /dev/null
+++ b/src/glsl/glcpp/tests/120-undef-builtin.c.expected
@@ -0,0 +1,7 @@
+0:1(2): preprocessor error: Built-in (pre-defined) macro names can not be 
undefined.
+0:2(1): preprocessor error: Built-in (pre-defined) macro names can not be 
undefined.
+0:3(1): preprocessor error: Built-in (pre-defined) macro names can not be 
undefined.
+
+
+
+
-- 
2.0.0

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


Re: [Mesa-dev] [PATCH] Add test to ensure compiler won't allow #undef for some builtins

2014-06-16 Thread Anuj Phogat
On Wed, Jun 11, 2014 at 5:32 PM, Carl Worth  wrote:
> Currently verifying that an #undef of __FILE__, __LINE__, or __VERSION__ will
> generate an error.
> ---
>  src/glsl/glcpp/tests/120-undef-builtin.c  | 3 +++
>  src/glsl/glcpp/tests/120-undef-builtin.c.expected | 7 +++
>  2 files changed, 10 insertions(+)
>  create mode 100644 src/glsl/glcpp/tests/120-undef-builtin.c
>  create mode 100644 src/glsl/glcpp/tests/120-undef-builtin.c.expected
>
> diff --git a/src/glsl/glcpp/tests/120-undef-builtin.c 
> b/src/glsl/glcpp/tests/120-undef-builtin.c
> new file mode 100644
> index 000..49e7696
> --- /dev/null
> +++ b/src/glsl/glcpp/tests/120-undef-builtin.c
> @@ -0,0 +1,3 @@
> +#undef __LINE__
> +#undef __FILE__
> +#undef __VERSION__
> diff --git a/src/glsl/glcpp/tests/120-undef-builtin.c.expected 
> b/src/glsl/glcpp/tests/120-undef-builtin.c.expected
> new file mode 100644
> index 000..d6981dc
> --- /dev/null
> +++ b/src/glsl/glcpp/tests/120-undef-builtin.c.expected
> @@ -0,0 +1,7 @@
> +0:1(2): preprocessor error: Built-in (pre-defined) macro names can not be 
> undefined.
> +0:2(1): preprocessor error: Built-in (pre-defined) macro names can not be 
> undefined.
> +0:3(1): preprocessor error: Built-in (pre-defined) macro names can not be 
> undefined.
> +
> +
> +
> +
> --
> 2.0.0
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

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