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

Improve memory management using smart pointers #916

Closed
NickDraper opened this issue Dec 6, 2007 · 4 comments
Closed

Improve memory management using smart pointers #916

NickDraper opened this issue Dec 6, 2007 · 4 comments
Assignees
Milestone

Comments

@NickDraper
Copy link
Contributor

Look at every bare pointer we are using and consider if it can be replaced by a scoped_ptr, shared_ptr of a refControl pointer (This may get renamed cow_ptr).

@NickDraper
Copy link
Contributor Author

This issue was originally trac ticket 68

@NickDraper NickDraper self-assigned this Jun 3, 2015
@NickDraper NickDraper added this to the Iteration 4 milestone Jun 3, 2015
@elfring
Copy link

elfring commented Jul 25, 2018

Would you like to wrap any pointers with the class template “std::unique_ptr”?

@NickDraper
Copy link
Contributor Author

We did some profiling several years ago that showed that blindly doing that added a significant performance cost, when added up over the whole system. New development should avoid using bare pointers, but some areas that are highly performance sensitive, and do not show memory management problems such as geometry are not worth addressing, particularly if it affects performance.

@elfring
Copy link

elfring commented Jul 26, 2018

  • Do the run time characteristics look different for the usage of smart pointers in the current software situation?
  • How do you think about to achieve safe resource management together with recent development tools?

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

No branches or pull requests

2 participants