Comments you submit will be routed for moderation. If you have an account, please log in first.
Modify

Ticket #3888 (closed enhancement: invalid)

Opened 3 years ago

Last modified 8 months ago

--branch-probabilities should be optional

Reported by: daniel.neuberger@… Owned by: jdsiiro
Priority: normal Milestone:
Component: gcovr Version:
Keywords: Cc:

Description

Whether or not to use the --branch-probabilities option with gcov should be optional. We've turned it off and are only using line coverage because C++ compilers typically create many extra branches that may not be worth getting coverage for.

Attachments

Change History

comment:1 Changed 10 months ago by wehart

I'm not that I understand this ticket. You want branch coverage, but without the --branch-probabilities option? How would that change coverage statistics generaged by gcovr??

comment:2 Changed 10 months ago by daniel.neuberger@…

Basically, we updated the gcovr script to not pass the "--branch-probabilities" option to gcov, which has the effect of only reporting line coverage and not branch coverage. In other words, we don't want branch coverage. Initially we did, but then we realized that it was confusing with C++ since the compiler adds extra branches. Does that make more sense?

comment:3 Changed 8 months ago by wehart

  • Status changed from new to closed
  • Resolution set to invalid

Moved to github ticket 9

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.