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

Memory leak in ResultInfo::operator =() #1

Closed
philsquared opened this issue Nov 11, 2010 · 3 comments
Closed

Memory leak in ResultInfo::operator =() #1

philsquared opened this issue Nov 11, 2010 · 3 comments

Comments

@philsquared
Copy link
Collaborator

The TestCase pointer member recieves a clone of the incoming, but if it already pointed at something else it leaks.

Op= should be implemented in terms of copy & swap anyway.

Thanks to Sam Saariste for spotting this one.
@ssaariste
Copy link

The issue is in TestCaseInfo::operator=() not in ResultInfo.

@philsquared
Copy link
Collaborator Author

That's what happens when you do this stuff without the code in front of you. Thanks again

@philsquared
Copy link
Collaborator Author

Fixed. Now does copy & swap

@iamleeg iamleeg mentioned this issue May 21, 2012
Closed
nanoant pushed a commit to ujhpc/Catch that referenced this issue Nov 17, 2013
@tanakh
Merge pull request catchorg#1 from yuttie/master
4dfb5d6 
oneof_reader::operator() is searching alt for an input string `s'.
@DavidEGrayson DavidEGrayson mentioned this issue Jun 18, 2015
Closed
@mintyc mintyc mentioned this issue Oct 27, 2015
Closed
@curb1968 curb1968 mentioned this issue Apr 28, 2016
Closed
@chrisdjauntvr chrisdjauntvr mentioned this issue Sep 28, 2016
Closed
@Leandros Leandros mentioned this issue Mar 15, 2018
Closed
alpereira7 added a commit to alpereira7/Catch2 that referenced this issue Jul 9, 2018
@alpereira7
This fixes Multiple-file example link catchorg#2.
34c3fac 
Link catchorg#2 was directed to catchorg#1.
@alpereira7 alpereira7 mentioned this issue Jul 9, 2018
Merged
@floatice floatice mentioned this issue Jul 29, 2019
Closed
@Linshiqi Linshiqi mentioned this issue Mar 21, 2020
Closed
dmsteck pushed a commit to dmsteck/Catch2 that referenced this issue May 3, 2020
Merge pull request catchorg#1 from catchorg/master
c1e4e2b 
Pull latest Catch2 master
@Alan-Jowett Alan-Jowett mentioned this issue Dec 16, 2021
Closed
@LukashonakV LukashonakV mentioned this issue Jul 20, 2023
Closed
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@philsquared @ssaariste