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 cwo...@cworth.org 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 anuj.pho...@gmail.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


[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