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.

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

Fixed #313: Show CK_NoOp correctly. #314

Merged
merged 1 commit into from
May 23, 2020
Merged

Fixed #313: Show CK_NoOp correctly. #314

merged 1 commit into from
May 23, 2020

Conversation

andreasfertig
Copy link
Owner

CK_NoOp stands for adding const or other qualifiers. Show this
conversion, if --show-all-implicit-casts is active.

This patch enables showing implicit casts for member-function calls
as well.

`CK_NoOp` stands for adding `const` or other qualifiers. Show this
conversion, if `--show-all-implicit-casts` is active.

This patch enables showing implicit casts for member-function calls
as well.
@codecov
Copy link

codecov bot commented May 23, 2020

Codecov Report

Merging #314 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #314   +/-   ##
=======================================
  Coverage   97.64%   97.65%           
=======================================
  Files          23       23           
  Lines        2761     2767    +6     
=======================================
+ Hits         2696     2702    +6     
  Misses         65       65           
Impacted Files Coverage Δ
CodeGenerator.cpp 98.75% <100.00%> (+<0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 44d4c44...bea5681. Read the comment docs.

@andreasfertig andreasfertig merged commit c85cfe0 into master May 23, 2020
@andreasfertig andreasfertig deleted the fixIssue313 branch May 23, 2020 12:15
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.

None yet

1 participant