PP: Partially address issue #738: handle premature end of argument when token pasting.
authorJohn Kessenich <cepheus@frii.com>
Wed, 1 Mar 2017 01:04:47 +0000 (18:04 -0700)
committerJohn Kessenich <cepheus@frii.com>
Wed, 1 Mar 2017 01:06:39 +0000 (18:06 -0700)
Test/baseResults/tokenPaste.vert.out
Test/tokenPaste.vert
glslang/Include/revision.h
glslang/MachineIndependent/preprocessor/Pp.cpp

index 9a06c34..4bab57b 100755 (executable)
@@ -5,7 +5,10 @@ ERROR: 0:40: '##' : unexpected location; end of replacement list
 ERROR: 0:49: '##' : combined tokens are too long 
 ERROR: 0:52: '##' : not supported for these tokens 
 ERROR: 0:69: '##' : combined token is invalid 
-ERROR: 5 compilation errors.  No code generated.
+ERROR: 0:82: 'macro expansion' : Too few args in Macro rec
+ERROR: 0:82: '##' : unexpected location 
+ERROR: 0:82: '##' : unexpected location 
+ERROR: 8 compilation errors.  No code generated.
 
 
 Shader version: 450
index 369b7b8..7aa113a 100644 (file)
@@ -77,3 +77,6 @@ uniform M_OUTER(argPaste);
 #define M_NEST2(q) int q ## suff\r
 #define M_OUTER2(p) M_NEST2(p ## 20)\r
 uniform M_OUTER2(argPaste);\r
+\r
+#define rec(x)##\r
+rec(rec())
\ No newline at end of file
index f9f4ec5..d1bce52 100644 (file)
@@ -2,5 +2,5 @@
 // For the version, it uses the latest git tag followed by the number of commits.
 // For the date, it uses the current date (when then script is run).
 
-#define GLSLANG_REVISION "Overload400-PrecQual.1858"
+#define GLSLANG_REVISION "Overload400-PrecQual.1861"
 #define GLSLANG_DATE "28-Feb-2017"
index ef8e83e..4da1649 100644 (file)
@@ -986,6 +986,8 @@ TPpContext::TokenStream* TPpContext::PrescanMacroArg(TokenStream& arg, TPpToken*
     int token;
     while ((token = scanToken(ppToken)) != tMarkerInput::marker && token != EndOfInput) {
         token = tokenPaste(token, *ppToken);
+        if (token == tMarkerInput::marker || token == EndOfInput)
+            break;
         if (token == PpAtomIdentifier && MacroExpand(ppToken, false, newLineOkay) != 0)
             continue;
         expandedArg->putToken(token, ppToken);