Skip to content

Fix C++ parser relative path - #516

Merged
mcserep merged 1 commit into
Ericsson:masterfrom
bruntib:fix_cpp_parser_relative_path
Feb 23, 2021
Merged

Fix C++ parser relative path#516
mcserep merged 1 commit into
Ericsson:masterfrom
bruntib:fix_cpp_parser_relative_path

Conversation

@bruntib

Copy link
Copy Markdown
Collaborator

C++ parser is working based on a compilation_database.json file.
This file may contain relative file paths in its "file" attributes.
This file is given to C++ parser to build the AST of this file.
However, this works only if the filename is correctly relative to
the current working directory.
In this patch the directory and file components are joined so the
full path of a source file is passed to the parser.

@bruntibbruntib added Kind: Bug ⚠️ Plugin: C++ Issues related to the parsing and presentation of C++ projects. labels Feb 20, 2021
C++ parser is working based on a compilation_database.json file.
This file may contain relative file paths in its "file" attributes.
This file is given to C++ parser to build the AST of this file.
However, this works only if the filename is correctly relative to
the current working directory.
In this patch the directory and file components are joined so the
full path of a source file is passed to the parser.
@bruntib
bruntibforce-pushed the fix_cpp_parser_relative_path branch from e6b5030 to 04826d2CompareFebruary 20, 2021 11:16
@bruntibbruntib added Status: WIP 👷 Issue or PR under development - feel free to review, though! and removed Status: WIP 👷 Issue or PR under development - feel free to review, though! labels Feb 20, 2021

@intjftwintjftw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@mcserep
mcserep merged commit 3b716e1 into Ericsson:masterFeb 23, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Kind: Bug ⚠️Plugin: C++Issues related to the parsing and presentation of C++ projects.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@bruntib@mcserep@intjftw