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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions