bugfix in TOF clusterizer

Hi @p.-a.loizeau, There is probably a bug in the TOF Clusterizer; please have a look.

At the line 3960 of the CbmTofEventClusterizer.cxx:

if (hbin->GetBinContent(hbin->GetNbinsX() > 0))

It should be:

if (hbin->GetBinContent(hbin->GetNbinsX()) > 0)

added BugFix label

requested review from @p.-a.loizeau

assigned to @p.-a.loizeau

Dear @f.uhlig, @v.friese, @p.-a.loizeau, @n.herrmann, @i.deppner,

you have been identified as code owner of at least one file which was changed with this merge request.

Please check the changes and approve them or request changes.

added CodeOwners label

Should be checked also with @n.herrmann. The version of this class in the master branch is not the one used by the TOF team.

requested review from @n.herrmann and removed review request for @p.-a.loizeau

I would also think so, but not 100% sure: the next line accesses the center of NBinsX - 1 so while really strange it may make sense somehow.

The expert here is @n.herrmann I set him as reviewer

I agree, it's a bug. Will be fixed with the pending update once it passes the Merge_realData test...

Superseded by MR !1399 (merged) in commit !1399 (3fb579ac)
=> Closing

closed