diff options
| author | Tim Foley <tfoleyNV@users.noreply.github.com> | 2017-10-09 13:38:33 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-10-09 13:38:33 -0700 |
| commit | 06b2192aa2bb6698c863388c5d085ba5b1f28374 (patch) | |
| tree | 27a959d883adad48652ca9461291fddb64b5a9b9 /tests/preprocessor/define-redefine.slang.expected | |
| parent | fe5eef423389b82e7eb2586fb7a16b96afd004f2 (diff) | |
Preprocessor: fix `undef` and redefinition (#204)
* Preprocessor: fix `undef` and redefinition
The logic for `undef` directives was failing to suppress macro expansion when reading the name to un-define, and so it wasn't actually working at all. We didn't notice this because we didn't have a test case, and users hadn't tried it.
The logic for `define` had a similar bug, which meant that any attempt to define an already-defined macro would fail with a cryptic error, rather than raising the intended warning.
Test cases have been added for both issues, along with the fixes.
* fixup: add expected output for tests added
Diffstat (limited to 'tests/preprocessor/define-redefine.slang.expected')
| -rw-r--r-- | tests/preprocessor/define-redefine.slang.expected | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/tests/preprocessor/define-redefine.slang.expected b/tests/preprocessor/define-redefine.slang.expected new file mode 100644 index 000000000..2542c05d8 --- /dev/null +++ b/tests/preprocessor/define-redefine.slang.expected @@ -0,0 +1,7 @@ +result code = 0 +standard error = { +tests/preprocessor/define-redefine.slang(10): warning 15400: redefinition of macro 'FOO' +tests/preprocessor/define-redefine.slang(6): note: see previous definition of 'FOO' +} +standard output = { +} |