Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SLVS-1668 Include header file language in the VCX command #5861

Open
wants to merge 4 commits into
base: feature/cfamily-migration
Choose a base branch
from
Open

SLVS-1668 Include header file language in the VCX command #5861

wants to merge 4 commits into from

Commits on Dec 4, 2024

  1. Include language information for headers in the command line 

    This helps analyze headers in C projects as C code. This is needed now
that we no longer pass HeaderFileLanguage separately to the analyzer.
    @michael-jabbour-sonarsource
    michael-jabbour-sonarsource committed Dec 4, 2024
    Configuration menu
    Copy the full SHA
    acc0381 View commit details
    Browse the repository at this point in the history

  2. Drop HeaderFileLanguage since it is no longer passed

    @michael-jabbour-sonarsource
    michael-jabbour-sonarsource committed Dec 4, 2024
    Configuration menu
    Copy the full SHA
    8d9037d View commit details
    Browse the repository at this point in the history

  3. Workaround for identifying header files in the CFamily analyzer 

    Normally, the CFamily analyzer deduces that the file being analyzed is a
header when there is no exact match for the file name in the given
compilation database.

This is a temporary workaround to communicate with the analyzer that
the current file is a header file. This helps the analyzer avoid
reporting some FPs.

See https://sonarsource.atlassian.net/browse/CPP-2743
    @michael-jabbour-sonarsource
    michael-jabbour-sonarsource committed Dec 4, 2024
    Configuration menu
    Copy the full SHA
    2fcdbd1 View commit details
    Browse the repository at this point in the history

  4. Revert hack for passing MainFileIsHeader

    @michael-jabbour-sonarsource
    michael-jabbour-sonarsource committed Dec 4, 2024
    Configuration menu
    Copy the full SHA
    d58d1bc View commit details
    Browse the repository at this point in the history