Skip to content

egammaCaloTools : Let all variables mean all variables and remove a couple of redundat flags

Christos Anastopoulos requested to merge ATLAS-EGamma/athena:egammaCaloTools_redundant_flags into master

@turra brought up that constructs like

m_ExecAllVariables && m_ExecEMThird

can be confusing. We had all these flags true so there is not "real issue" or "expected change" but tried to make so that All means all. aka

m_ExecAllVariables || m_ExecEMThird

etc. Then also also All includes the "other" so there is some redundancy removed.

Edited by Tadej Novak

Merge request reports