-
Notifications
You must be signed in to change notification settings - Fork 97
Remove unused FairRunOnline::InitContainers() function #1396
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
Remove unused FairRunOnline::InitContainers() function #1396
Conversation
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.
Very good catch!
I have a few notes.
The function is not being invoked, and was not used. Tested with R3BRoot. Adjusted the LOG() messages. Added corresponding CHANGELOG notice. Addresses issue FairRootGroup#1346.
41b5b23
to
47081ea
Compare
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.
I also wonder, whether we should refactor this a bit into some common FairRun feature?
Or should FairRunAna and FairRunOnline have common helper base class?
Just pondering! I haven't thought this through, really!
47081ea
to
98c2cf6
Compare
98c2cf6
to
d5626f0
Compare
Added check for `fStatic` in the `FairRunOnline::EventLoop()`. When `fStatic` is set to true by `SetContainerStatic()`, `ReInit()` will not be called when changed `RunId` detected.
d5626f0
to
d4a8417
Compare
The function is not being invoked, and was not used.
Addresses issue #1346.
Checklist: