Skip to content

Debug: protection in ntuple dump in FullLDCTrackingAlg (minor) #237

@qwert2333

Description

@qwert2333

Hi,

There seems a bug in FullLDCTrackingAlg when I disable dumping the ntuples (dumpTime=False):
- In line 246 and 582 if(m_tuple), which I think it's better to write as if(m_dumpTime && m_tuple) to avoid the break.
This is such a minor debug so I did not create a PR, maybe you can fix it sometime convenient.

Cheers,
Fangyi

*Code in git:
https://github.com/cepc/CEPCSW/blob/master/Reconstruction/Tracking/src/FullLDCTracking/FullLDCTrackingAlg.cpp#L246
https://github.com/cepc/CEPCSW/blob/master/Reconstruction/Tracking/src/FullLDCTracking/FullLDCTrackingAlg.cpp#L582

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions