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

copy constr & assignment operator + 100% code coverage #136

Merged
merged 1 commit into from
Jan 19, 2018
Merged

copy constr & assignment operator + 100% code coverage #136

merged 1 commit into from
Jan 19, 2018

Conversation

strasdat
Copy link
Owner

  • bug fixes
  • unit test coverage

@strasdat strasdat force-pushed the copy_constr_and_assignment branch 2 times, most recently from d95fce2 to b645173 Compare January 19, 2018 22:50
@strasdat
copy constr & assignment operator improvements + 100% code coverage
eb1bf80 
 - bug fixes
 - unit test for constr & assignment
 - improve relative test coverage by deleting deprecated functions
 - exclusion of lines using // LCOV_EXCL_START and // LCOV_EXCL_END
@coveralls
Copy link

Coverage Status

Coverage increased (+1.2%) to 99.143% when pulling d95fce2 on copy_constr_and_assignment into 3ee8738 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.9%) to 99.839% when pulling b645173 on copy_constr_and_assignment into 3ee8738 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 98.35% when pulling 9d825d2 on copy_constr_and_assignment into 3ee8738 on master.

@strasdat strasdat changed the title copy constr and assignment operator: copy constr & assignment operator + 100% code coverage Jan 19, 2018
@strasdat strasdat merged commit 7ef9cff into master Jan 19, 2018
@strasdat strasdat deleted the copy_constr_and_assignment branch January 19, 2018 23:33
@strasdat strasdat mentioned this pull request Jan 19, 2018
Closed
@coveralls
Copy link

Coverage Status

Coverage increased (+2.07%) to 100.0% when pulling eb1bf80 on copy_constr_and_assignment into 3ee8738 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@strasdat @coveralls