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

Jump to bottom

--include flags shouldn't be skipped #1237

Merged
merged 1 commit into from
Dec 8, 2017
Merged

--include flags shouldn't be skipped #1237

merged 1 commit into from
Dec 8, 2017

Conversation

bruntib
Copy link
Contributor

@bruntib bruntib commented Dec 7, 2017

No description provided.

@whisperity whisperity added the analyzer 📈 Related to the analyze commands (analysis driver) label Dec 7, 2017
@whisperity
Copy link
Contributor

Is this flag valid for G++, Clang? Documentations use -I everywhere, with a single hyphen.

@bruntib
Copy link
Contributor Author

bruntib commented Dec 8, 2017

-include and --include flags are given a header file instead of a directory. This file will be included in the translation unit implicitly during the preprocessing phase.

Xazax-hun
Xazax-hun approved these changes Dec 8, 2017
View reviewed changes
Copy link
Contributor

@Xazax-hun Xazax-hun left a comment

Choose a reason for hiding this comment

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

LG!

@Xazax-hun Xazax-hun merged commit 795ee2e into Ericsson:master Dec 8, 2017
dkrupp
dkrupp reviewed Dec 8, 2017
View reviewed changes
Copy link
Member

@dkrupp dkrupp left a comment •
edited
Loading

Choose a reason for hiding this comment

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

what is with the other parameters with double "--" ?
like --imacrose --idirafter etc.

@bruntib bruntib deleted the fix_log_options branch December 8, 2017 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dkrupp dkrupp dkrupp left review comments

@gyorb gyorb gyorb approved these changes

@Xazax-hun Xazax-hun Xazax-hun approved these changes

Assignees
No one assigned
Labels
analyzer 📈 Related to the analyze commands (analysis driver)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@bruntib @whisperity @Xazax-hun @dkrupp @gyorb