Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

invMass to energyBranch Name code duplication #45

Open
ldcorpe opened this issue Sep 24, 2015 · 1 comment
Open

invMass to energyBranch Name code duplication #45

ldcorpe opened this issue Sep 24, 2015 · 1 comment

Comments

@ldcorpe
Copy link

ldcorpe commented Sep 24, 2015

This is not urgent, but there is a duplication of nearly identical code which should be factorised, since otherwise if one wants to add a new energy Branch, one needs to change it ins several places instead of just one.

The two places are:

TString ZFit_class::GetEnergyVarName(TString invMass_name){

and:
std::string energyBranchNameFromInvMassName(std::string invMass_var){

@shervin86
Copy link

I agree, maybe we should create a library of functions that can be loaded by both.

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

No branches or pull requests

2 participants