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

Partial struct initialization included in a coverage report #1008

Open
zhiburt opened this issue May 14, 2022 · 1 comment
Open

Partial struct initialization included in a coverage report #1008

zhiburt opened this issue May 14, 2022 · 1 comment
Assignees
Labels
enhancement Source-analysis Issues relating to analysis of a projects source code.

Comments

@zhiburt
Copy link
Contributor

zhiburt commented May 14, 2022

Describe the bug

Hi there,

Sometimes a structure initialization included in a coverage report but only partially.
Whereas sometimes it's not included at all.

Screenshot_2022-05-14_14-10-02

Screenshot_2022-05-14_14-04-10

I did tried to reproduce it by a minimal example but didn't managed easily.

Sorry if it's a duplicate.

Expected behavior

I suppose the report must be consistent in approach?

@xd009642
Copy link
Owner

The constants like "" end up being optimised out a lot of the time and don't result in instructions I can catch in the coverage instrumentation. I believe now it should just ignore constants and just simple variable assigns but not ignore function calls or more complicated expressions.

Ah just noticed the second one is a generic function. The stuff to ensure unused generic functions and methods are included in the report is over eagerly adding back in the lines 👀. I'll look into that at some point before the next release

@xd009642 xd009642 added enhancement Source-analysis Issues relating to analysis of a projects source code. labels Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Source-analysis Issues relating to analysis of a projects source code.
Projects
None yet
Development

No branches or pull requests

2 participants