Skip to content

feat(cpp): add ANTLR grammar files for C/C++ support#399

Open
Divyateja2709 wants to merge 5 commits into
cbomkit:mainfrom
Divyateja2709:pr2
Open

feat(cpp): add ANTLR grammar files for C/C++ support#399
Divyateja2709 wants to merge 5 commits into
cbomkit:mainfrom
Divyateja2709:pr2

Conversation

@Divyateja2709
Copy link
Copy Markdown

hi @n1ckl0sk0rtge

This PR adds the initial ANTLR grammar infrastructure required for C/C++ language support in the Sonar Cryptography Plugin.

Changes Included
Added CLexer.g4 for C/C++ lexical analysis
Added CParser.g4 for parsing C/C++ source files
Updated SonarQube plugin configuration to recognize:
.c
.cpp
.h
source files through requiredForLanguages
Purpose

These grammar files provide the parsing foundation required for upcoming:

C/C++ AST generation
OpenSSL detection rules
C/C++ detection engine integration

Signed-off-by: Divyateja Indrakanti <indrakantidivyateja@gmail.com>
Signed-off-by: Divyateja Indrakanti <indrakantidivyateja@gmail.com>
Signed-off-by: Divyateja Indrakanti <indrakantidivyateja@gmail.com>
Signed-off-by: Divyateja Indrakanti <indrakantidivyateja@gmail.com>
Signed-off-by: Divyateja Indrakanti <indrakantidivyateja@gmail.com>
@Divyateja2709 Divyateja2709 requested a review from a team as a code owner May 6, 2026 03:17
@san-zrl
Copy link
Copy Markdown
Contributor

san-zrl commented May 20, 2026

Hi @Divyateja2709,

Thank you for your PR and for the time and effort you put into contributing to CBOMkit—we truly appreciate your interest.

You may have noticed that we are already working on adding C/C++ support to sonar-cryptography in PR #377. This approach differs from yours, as it relies on Sonar’s native C/C++ support package rather than ANTLR, similar to how the existing Java module is implemented.

Work on PR #377 is already quite advanced, and at this stage we are leaning toward this approach because we expect it to provide stronger support for cross-file symbol tracing through the Sonar infrastructure.

That said, we will keep your PR open for now until a final decision is made.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants