-
Notifications
You must be signed in to change notification settings - Fork 38
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
Deprecate ResourceBuff
#604
Conversation
Build Status Report - 72ccbab - 2024-05-17 12:12:27 -0500Build
|
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.
Thanks @bennibbelink
@@ -318,7 +318,6 @@ void Storage::ProcessMat_(double cap) { | |||
using cyclus::Material; | |||
using cyclus::ResCast; | |||
using cyclus::toolkit::ResBuf; | |||
using cyclus::toolkit::Manifest; |
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.
While we're here... I don't see any reason for any of these using
statements to be here...?
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.
Great point. I went through storage.cc
and cleaned up some other using
statements. A quick search of the cycamore archetypes shows there are a lot of using
statements throughout the repo. I will make an issue to go through and clean up where it is necessary
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.
Thanks @bennibbelink
A small change to allow for the removal of the
ResourceBuff
class in cyclus