-
Notifications
You must be signed in to change notification settings - Fork 62
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
Fix issues when compiling against ROOT6 #6
Conversation
These classes are processed by ROOT to create streamers. Unlike rootcint in ROOT5, rootcling seemingly respects the access modifiers and needs all types to be publicly defined. [StTofHitMaker, StJetMaker]
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 see no issues with the proposed changes.
StAutoBrowse conflicts with ROOT6 and seems to be an outdated wrapper around the ROOT TBrowser. Users can invoke TBrowser directly in their interactive ROOT sessions
Regarding changes to StRoot/StEvent/StEmcDetector.cxx I have never understood the purpose of (or history behind) StAutoBrowse. Changes look harmless / touch functionality, not event model. However, this code is in StEvent, and so @ullrich-bnl should review. |
It is written by me. It is automatic Browser, to avoid manual encoding
all included classes.
It was not implemented in ROOT. Later Rene add this functionality in
ROOT.
Now it is possible to remove.
Victor
…On 2021-06-25 13:08, klendathu2k wrote:
Regarding changes to StRoot/StEvent/StEmcDetector.cxx
I have never understood the purpose of (or history behind)
StAutoBrowse. Changes look harmless / touch functionality, not event
model. However, this code is in StEvent, and so @ullrich-bnl [1]
should review.
--
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub [2], or unsubscribe
[3].
Links:
------
[1] https://github.com/ullrich-bnl
[2] #6 (comment)
[3]
https://github.com/notifications/unsubscribe-auth/ANQUL7KZF524EO6543PVGNLTUSZ2NANCNFSM455HPCUA
|
Any more approvals from the code owners? |
If I don't hear any further comments from anyone including @ullrich-bnl I'll merge this PR tomorrow. |
Unlike rootcint, rootcling in ROOT6 seems to be mindful of the access modifiers for nested types used to declare data members. Such types should have public access