[c2cpg] Recognize more source file extensions #5173
Conversation
To be consistent with latest cmake and CDT (eclipse-cdt/cdt#422).
| @@ -40,9 +40,9 @@ class HeaderAstCreationPassTests extends C2CpgSuite { | |||
| case Seq(bar, foo, m, printf) => | |||
| // note that we don't see bar twice even so it is contained | |||
| // in main.h and included in main.c and we do scan both | |||
| bar.fullName shouldBe "bar" | |||
| bar.fullName shouldBe "bar:void()" | |||
There was a problem hiding this comment.
This looks like it would be a regression for C code, in order to improve C++ code. I guess there just is inherent ambiguity with .h files, whether they contain C or C++ code. But I don't think we can get away with this right now.
Maybe we do header files in a second pass after the regular files, so we can know whether they got included from C or C++ files? Or maybe CDT has some guess-the-file-type magic since the IDE runs into the same problem?
There was a problem hiding this comment.
Or maybe CDT has some guess-the-file-type magic since the IDE runs into the same problem?
Sadly no, they also simply use the C++ parser in all cases.
The two-passes approach also won't work in all cases, as one could e.g. include a C header file in a C and C++ source file.
There was a problem hiding this comment.
Or to phrase it differently:
The behaviour without this PR is definitely wrong as it makes parsing CPP code in .h files impossible.
With this PR will are able to parse such code. "Wrong" fullnames for C method declarations that are never implemented in any source file (because there we will create the correct fullname and de-duplicate correctly) should be no issue or do I miss something there?
There was a problem hiding this comment.
Latest commit to this PR does not handle *.h files as CPP files anymore.
Just want to get the additional source file extensions in.
To be consistent with latest cmake and CDT (eclipse-cdt/cdt#422).