Skip to content

Add special case for std::set template usage - #62

Merged
tkruse merged 2 commits into
cpplint:masterfrom
vinograd47:include-set-false-positive
Aug 12, 2019
Merged

Add special case for std::set template usage#62
tkruse merged 2 commits into
cpplint:masterfrom
vinograd47:include-set-false-positive

Conversation

@vinograd47

@vinograd47 vinograd47 commented Nov 19, 2018

Copy link
Copy Markdown

Avoid false positive for [build/include_what_you_use] in case of foo.set<type> and foo->set<type> usage.

Fixes google/styleguide#410

Avoid false positive for [build/include_what_you_use]
in case of `foo.set<type>` and `foo->set<type>` usage
Avoid false positive for [build/include_what_you_use]
in case of `map` is user defined function
@vinograd47

Copy link
Copy Markdown
Author

Extended the PR with std::map case (map word also can be used in user code).

@tkruse
tkruse merged commit adb3500 into cpplint:master Aug 12, 2019
@tkruse

tkruse commented Aug 12, 2019

Copy link
Copy Markdown
Contributor

thx for also adding map case

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.

False Positive: Add #include <set> for set

2 participants