Skip to content

Conversation

@juanangp
Copy link
Member

@juanangp juanangp commented Apr 6, 2022

After new changes in the code, it seems that root doesn't like override method, see below:

image

Fix this error by removing override from TRestVolumeHits

@juanangp juanangp requested review from jgalan and lobis April 6, 2022 07:54
@juanangp juanangp merged commit 26aec05 into master Apr 7, 2022
@juanangp juanangp deleted the DrawEventFix branch April 7, 2022 12:04
@nkx111
Copy link
Member

nkx111 commented Apr 8, 2022

You need to use ClassDefOverride instead ClassDef for cint, in case you have override keyword in the header.

This is not ROOT issue, this is a favor of gcc. Once you have one method marked with override, you need to mark override for all. On the other hand, ClassDef macro adds several methods implementing those from TObject, which in this case shall also be marked with override keyword. We can use ClassDefOverride macro to do this.

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

Successfully merging this pull request may close these issues.

5 participants