Andrew Yourtchenko

The following diff takes care of the problem: ``` diff diff --git a/lcpp.lua b/lcpp.lua index 4450dcd..e60f225 100755 --- a/lcpp.lua +++ b/lcpp.lua @@ -418,7 +418,7 @@ local function removeComments(input) for k,...

The following diff takes care of it: ``` diff @@ -1404,7 +1413,7 @@ function lcpp.compileFile(filename, predefines, macro_sources, next, _local) if not file then error("file not found: "..filename) end local code...

The following diff takes care of it: ``` diff diff --git a/lcpp.lua b/lcpp.lua index e60f225..be6c34b 100755 --- a/lcpp.lua +++ b/lcpp.lua @@ -462,7 +462,7 @@ local function parseCInteger(input) elseif k ==...

I was not sure how to properly make a testcase, still wrapping my head around the mix of lua and cpp in that part of the code :) I'll do...

I toyed a bit more and see how they work now. I've dug a few more issues. So I will create a commit with the testcases for all, then create...

This diff takes care of the problem (although needs more testing, including here mostly for comments): ``` diff diff --git a/lcpp.lua b/lcpp.lua index 9ac5088..4d1defb 100755 --- a/lcpp.lua +++ b/lcpp.lua @@...

A further reduced testcase for this issue: ``` h #define SOMEMACRO(v) (YYYYY) #define _(a) SOMEMACRO (bi); _SOMEMACRO(ai); _ (foo) ``` The parentheses around YYYYY are essential - if they are...

The problem is within the definition of the function that does the macro substitution - it matches on name that ends with a name of a macro that is already...

Yeah, let me debug bit more, together with other fixes (I think it might add a regression), I will get the fixes for them in once I get lcpp to...

Martin, thanks for the fix! At present, this space (pull requests) is not very well attended, since VPP uses gerrit.fd.io for code, and this is effectively a read-only feed from...