-
Notifications
You must be signed in to change notification settings - Fork 68
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
Trajectory #124
Trajectory #124
Conversation
@@ -48,9 +53,10 @@ class NBodySystem { | |||
|
|||
std::unique_ptr<Bodies const> const massive_bodies_; // Never null. | |||
std::unique_ptr<Bodies const> const massless_bodies_; // Never null. | |||
std::unique_ptr<Trajectories const> const trajectories_; // Never null. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider std::reference_wrapper
, but that's a job for another CL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not even sure we need std::reference_wrapper
, but yes, these std::unique_ptr
look rather silly.
No description provided.